[FFmpeg-devel] [PATCH] lavfi documentation nits

Stefano Sabatini stefano.sabatini-lala
Wed May 6 00:02:15 CEST 2009


On date Tuesday 2009-05-05 01:02:06 +0200, Diego Biurrun encoded:
> On Mon, May 04, 2009 at 10:39:56PM +0200, Stefano Sabatini wrote:
[...]
> > Eheh. Patch updated, if OK I'll apply as a separate patch the
> > whitespace changes.
> ..
> 
> > --- libavfilter/avfilter.h	(revision 18726)
> > +++ libavfilter/avfilter.h	(working copy)
> > @@ -112,7 +112,7 @@
> >  
> >  /**
> > - * Remove a reference to a picture.  If this is the last reference to the
> > + * Removes a reference to a picture.  If this is the last reference to the
> 
> There is little point in keeping the two spaces on lines you already
> change...
> 
> > @@ -518,7 +518,7 @@
> >  
> >  /**
> > - * Notify the next filter of the start of a frame.
> > + * Notifies the next filter of the start of a frame.
> 
> notify about

I believe the form "notify of" is correct too, in case of doubt I left
it unchanged.
 
> > --- libavfilter/avfiltergraph.h	(revision 18726)
> > +++ libavfilter/avfiltergraph.h	(working copy)
> > @@ -32,7 +32,7 @@
> >  } AVFilterGraph;
> >  
> >  /**
> > - * Get from \p graph a filter instance with name \p name.
> > + * Gets from graph a filter instance with name name.
> 
> Gets a filter instance with name name from graph.

Fixed.

Updated patch attached.
-- 
FFmpeg = Forgiving Furious MultiPurpose Everlasting God
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lavfi-docs-nits.patch
Type: text/x-diff
Size: 7909 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090506/61a88e29/attachment.patch>



More information about the ffmpeg-devel mailing list