On Sun, Oct 16, 2016 at 09:46:20PM +0200, Michael Behrisch wrote: > Hi all, > first of all thanks for providing this great library. > > Today my pull request https://github.com/FFmpeg/FFmpeg/pull/237 has been > rejected which tried to remove the comma at the end of enumerator lists > which triggers a warning when compiling with gcc and -Wpedantic. I still > think it could be a good idea to remove this, especially in the cases > where the enumerator list already ends in some _NB item, which seems to > be the final one "forever", but I am here to learn :-).
The enum with a final _NB (or similar) entry are the only ones where removing the comma is relevant. -- Clément B. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel