[FFmpeg-devel] [PATCH 3/4] move supported list printing code from cmdutils to lavc

Nicolas George nicolas.george at normalesup.org
Mon Oct 22 13:01:50 CEST 2012


Le primidi 1er brumaire, an CCXXI, Michael Niedermayer a écrit :
> +    if (!type || !strcmp(type, "pixel formats")) {

I must say, I do not like it: it is fragile and offers no build-time error
checking. An enum, possibly a bitmask, seems like a better idea.

Regards,

-- 
  Nicolas George


More information about the ffmpeg-devel mailing list