[FFmpeg-devel] [PATCH] lavfi documentation nits

Stefano Sabatini stefano.sabatini-lala
Mon May 4 22:39:56 CEST 2009


On date Monday 2009-05-04 22:26:27 +0200, Diego Biurrun encoded:
> 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.

Eheh. Patch updated, if OK I'll apply as a separate patch the
whitespace changes.

Regards.
-- 
FFmpeg = Faithful and Faboulous Majestic Peaceless Earthshaking Governor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lavfi-docs-nits.patch
Type: text/x-diff
Size: 7911 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090504/ceba6622/attachment.patch>



More information about the ffmpeg-devel mailing list