[FFmpeg-devel] [PATCH] Add hwaccel_data infrastructure (take 3)

Michael Niedermayer michaelni
Fri Mar 6 17:51:36 CET 2009


On Fri, Mar 06, 2009 at 11:24:00AM +0100, Gwenole Beauchesne wrote:
> On Fri, 6 Mar 2009, Benoit Fouet wrote:
>
>>> Besides, I also dropped hwaccel_data, it's not a real gain. I am fine
>>> with AVFrame.data[3].
>>>
>>> New patch attached.
>>
>>> diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c
>>> index 43f2f1f..7739f64 100644
>>> --- a/libavcodec/mpegvideo.c
>>> +++ b/libavcodec/mpegvideo.c
>>> @@ -170,6 +171,11 @@ void ff_copy_picture(Picture *dst, Picture *src){
>>>  static void free_frame_buffer(MpegEncContext *s, Picture *pic)
>>>  {
>>>      s->avctx->release_buffer(s->avctx, (AVFrame*)pic);
>>> +
>>> +    if (pic->hwaccel_data_private) {
>>> +        av_free(pic->hwaccel_data_private);
>>> +        pic->hwaccel_data_private = NULL;
>>> +    }
>>>  }
>>>
>>
>> av_freep(&pic->hwaccel_data_private);
>
> New patch attached. Thanks.

looks ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090306/024f62c2/attachment.pgp>



More information about the ffmpeg-devel mailing list