On Wed, 14 Oct 2015 14:02:23 +0200
Vittorio Giovara <vittorio.giov...@gmail.com> wrote:

> On Wed, Oct 14, 2015 at 11:57 AM, wm4 <nfx...@googlemail.com> wrote:
> > On Wed, 14 Oct 2015 11:33:18 +0200
> > Vittorio Giovara <vittorio.giov...@gmail.com> wrote:
> >
> >>...
> >
> > Patches 01-07,09,10 just do simple replacements of AVPicture usage (in
> > particular avpicture_... to av_image_... functions), and I can't find
> > anything wrong with them.
> 
> yay
> 
> > "[PATCH 08/11] lavc: Add data and linesize to AVSubtitleRect" is
> > slightly controversial. It looks like with that patch applied, the
> > AVPicture fields are just left to 0 when decoding a frame.
> 
> Actually every subtitle decoder and encoder have these additional lines
> 
> +        for (j = 0; j < 4; j++) {
> +            rect->pict.data[j] = rect->data[j];
> +            rect->pict.linesize[j] = rect->linesize[j];
> +        }
> 
> which map the old pict buffers to the new ones, so the old way will still 
> work.

Right, missed this.
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to