> > > if (width != s->avctx->width || ((width+15)/16 != s->mb_width ||
> > > (height+15)/16 != s->mb_height) && s->macroblocks_base ||
> > > height != s->avctx->height) {
> > > ...
> > > ff_set_dimensions(...);
> > > }
> > >
> > > The condition for updating dimensions is simple in VP9:
> >
> > Now I didn't test VP9 decoder in this case
>
> Yes, this patch is for VP8 decoder only. What I mean is that
> ff_set_dimensions()
> is called too when {width, height} are changed without allocated
> macroblocks_base. So dim_reset should be 1 instead of
> (s->macroblocks_base !=
> NULL) in the patch.
Agree dim-reset should be 1, and this comment should be addressed.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
