[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.


  Nicolas George

More information about the ffmpeg-devel mailing list