[FFmpeg-devel] [PATCH] lavfi/tinterlace: use symbolic names for the parameter

Stefano Sabatini stefasab at gmail.com
Mon Apr 30 23:12:14 CEST 2012


On date Sunday 2012-04-29 21:44:02 +0200, Nicolas George encoded:
> Le primidi 11 floréal, an CCXX, Stefano Sabatini a écrit :
> > +static const char *tinterlace_mode_str[] = {
> > +    "merge",
> > +    "drop_even",
> > +    "drop_odd",
> > +    "pad",
> > +    "interleave_top",
> > +    "interleave_bottom",
> > +    "interlacex2",
> > +    NULL
> > +};
> 
> Maybe drop the final NULL and use FF_ARRAY_ELEMS?
> 
> Otherwise, I see no obvious flaws and it looks like a good idea.

Applied.
-- 
FFmpeg = Foolish & Fabulous Martial Practical Enchanting God


More information about the ffmpeg-devel mailing list