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

[Nicolas George]

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.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120429/d5e51d09/attachment.asc>