On Wed, Oct 14, 2015 at 11:57 AM, wm4 <[email protected]> wrote:
> On Wed, 14 Oct 2015 11:33:18 +0200
> Vittorio Giovara <[email protected]> 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.
-- 
Vittorio
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to