[FFmpeg-devel] [PATCH] lavfi documentation nits

Stefano Sabatini stefano.sabatini-lala
Mon May 4 22:07:13 CEST 2009


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)?
 
> You should familiarize yourself with the extra options available for
> 'svn diff', especially -u, -b, -w.

R.
-- 
FFmpeg = Fiendish and Faboulous Meaningful Purposeless Ecstatic God



More information about the ffmpeg-devel mailing list