[FFmpeg-devel] [PATCH] lavfi documentation nits

Diego Biurrun diego
Mon May 4 22:26:27 CEST 2009


On Mon, May 04, 2009 at 10:07:13PM +0200, Stefano Sabatini wrote:
> On date Monday 2009-05-04 22:01:44 +0200, Diego Biurrun encoded:
> > On Mon, May 04, 2009 at 09:29:34PM +0200, Stefano Sabatini wrote:
> > > On date Monday 2009-05-04 10:10:25 +0200, Diego Biurrun encoded:
> > > > On Sun, May 03, 2009 at 11:12:08PM +0200, Stefano Sabatini wrote:
> > > > > 
> > > > > prefer third person over first person, and use just one space between
> > > > > a sentence and the following one, also removes \p.
> > > > > Consistent with the rest of FFmpeg.
> > > > 
> > > > This contains some whitespace-only changes and variable renamings.
> > > 
> > > Sorry, please check again.
> > 
> > Still contains whitespace-only changes.
> 
> Yes and those changes are intentional (read above). Are you suggesting
> to split whitespace changes apart from the other ones (as for me they
> could also stay in the same patch)?

Michael will rip your still-beating heart out if you don't.

Diego



More information about the ffmpeg-devel mailing list