On Mon, 1 Jun 2015 20:59:42 +0200
Michael Niedermayer <michae...@gmx.at> wrote:
> On Mon, Jun 01, 2015 at 08:30:33PM +0200, wm4 wrote:
> > On Mon, 1 Jun 2015 20:27:11 +0200
> > Michael Niedermayer <michae...@gmx.at> wrote:
> >
> > > Signed-off-by: Michael Niedermayer <michae...@gmx.at>
> > > ---
> > > libavcodec/utils.c | 2 ++
> > > 1 file changed, 2 insertions(+)
> > >
> > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c
> > > index 7e3f92a..22e826f 100644
> > > --- a/libavcodec/utils.c
> > > +++ b/libavcodec/utils.c
> > > @@ -1453,6 +1453,8 @@ int attribute_align_arg
> > > avcodec_open2(AVCodecContext *avctx, const AVCodec *code
> > > }
> > > avctx->frame_number = 0;
> > > avctx->codec_descriptor = avcodec_descriptor_get(avctx->codec_id);
> > > + if (!avctx->codec_descriptor)
> > > + av_log(avctx, AV_LOG_WARNING, "No codec descriptor found for
> > > codec_id\n");
> > >
> > > if (avctx->codec->capabilities & CODEC_CAP_EXPERIMENTAL &&
> > > avctx->strict_std_compliance > FF_COMPLIANCE_EXPERIMENTAL) {
> >
> > This shouldn't happen at all. Every CODEC_ID needs a descriptor.
>
> i thought so too at first before writing this patch but
> user applications might want to use their own codecs with their
> own codec_ids and these would not have a descriptor
> also requiring a descriptor breaks the fate-libavcodec-options
> test
But uses can't provide their own codecs.
> ideas welcome of course ...
>
>
> [...]
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
