On Sun, Aug 03, 2014 at 08:12:17PM +0200, Janne Grunau wrote:
> On 2014-08-03 09:02:03 -0700, Diego Biurrun wrote:
> > --- a/libavcodec/mpeg4videodec.c
> > +++ b/libavcodec/mpeg4videodec.c
> > @@ -2539,14 +2539,19 @@ static int
> > mpeg4_update_thread_context(AVCodecContext *dst,
> > {
> > Mpeg4DecContext *s = dst->priv_data;
> > const Mpeg4DecContext *s1 = src->priv_data;
> > + int init = s->m.context_initialized;
> >
> > int ret = ff_mpeg_update_thread_context(dst, src);
> >
> > if (ret < 0)
> > return ret;
> >
> > + if (CONFIG_MPEG4_DECODER && !init && s1->xvid_build >= 0)
>
> the CONFIG_MPEG4_DECODER is kind of superflous since
> mpeg4_update_thread_context is only used from the mpeg4 decoder. might
> make sense to keep it though.
No, it's not, the file is also used by the mpeg4 parser. Without this
condition the mpeg4 parser will not compile standalone.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
