On Wed, Feb 08, 2017 at 11:12:12 +0100, Moritz Barsnick wrote: > Your switch/case no longer has a default. Is that intended? (Code > linters will complain.) AFAICT, avctx->profile will be unset if an > unknown s->profile is given. Is that okay? (It might be - I'm just > wondering. I *do* understand that not using the "profile" option will > default s->profile to FF_PROFILE_H264_HIGH.)
D'uh, sorry about that comment. I just figured out: Your option code doesn't allow anything else (for the time being) than the three named enums. Remark withdrawn, though adding a default *might* be safer. Moritz _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
