On Sun, Feb 28, 2016 at 12:17 AM, Michael Niedermayer <mich...@niedermayer.cc> wrote: > This would require listing supported formats for rawvideo and wrapped_avframe > > Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> > --- > libavcodec/utils.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index 2690d0d..267c973 100644 > --- a/libavcodec/utils.c > +++ b/libavcodec/utils.c > @@ -188,6 +188,11 @@ av_cold void avcodec_register(AVCodec *codec) > > if (codec->init_static_data) > codec->init_static_data(codec); > + > + if (av_codec_is_encoder(codec) && codec->type == AVMEDIA_TYPE_VIDEO && > !codec->pix_fmts) > + av_log(NULL, AV_LOG_WARNING, "Encoder %s is missing the pix_fmts > field\n", codec->name); > + if (av_codec_is_encoder(codec) && codec->type == AVMEDIA_TYPE_AUDIO && > !codec->sample_fmts) > + av_log(NULL, AV_LOG_WARNING, "Encoder %s is missing the sample_fmts > field\n", codec->name); > }
It seems kinda silly to write code to check if other code is correct. Apparently all encoders today are correct, so lets just pay attention in the future when adding new ones? - Hendrik _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
