[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