[FFmpeg-devel] [PATCH] lavfi/yadif: add support to named constants

Stefano Sabatini stefasab at gmail.com
Fri Jan 4 11:54:15 CET 2013


On date Thursday 2013-01-03 22:22:05 +0100, Stefano Sabatini encoded:
> On date Thursday 2013-01-03 18:13:51 +0100, Michael Niedermayer encoded:
> > On Thu, Jan 03, 2013 at 09:21:17AM +0100, Stefano Sabatini wrote:
> > [...]
> > > diff --git a/doc/filters.texi b/doc/filters.texi
> > > index e469310..4b59239 100644
> > > --- a/doc/filters.texi
> > > +++ b/doc/filters.texi
> > > @@ -4413,32 +4413,32 @@ Specify the interlacing mode to adopt. Accept one of the following
> > >  values:
> > >  
> > >  @table @option
> > > - at item 0
> > > + at item 0, send_frame
> > >  output 1 frame for each frame
> > > - at item 1
> > > + at item 1, send_field
> > >  output 1 frame for each field
> > > - at item 2
> > > -like 0 but skips spatial interlacing check
> > > - at item 3
> > > -like 1 but skips spatial interlacing check
> > > + at item 2, send_frame_nocheck
> > > +like @code{send_frame} but skip spatial interlacing check
> > > + at item 3, send_field_nocheck
> > 
> > send_*_nospatial seems a better name
> 
> Works fine for me, locally fixed.
>  
> > end of comments from me
> 
> I'll push all the yadif patches tomorrow if I read no more comments,
> thanks.

Applied, thanks all for the reviews.
-- 
FFmpeg = Formidable & Frightening Maxi Problematic Extroverse Game


More information about the ffmpeg-devel mailing list