[FFmpeg-devel] [PATCH] Add hwaccel_context to AVCodecContext (take 2)

Gwenole Beauchesne gbeauchesne
Mon Mar 16 17:29:36 CET 2009


Hi,

On Thu, 12 Mar 2009, Michael Niedermayer wrote:

> On Wed, Mar 11, 2009 at 11:15:06PM +0100, Gwenole Beauchesne wrote:
>> Hi,
>>
>> This adds an hwaccel_context variable to AVCodecContext. It's used to hold
>> read-only data from an FFmpeg point-of-view. Typically, those are HW
>> accelerator data.
>>
>> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
>> index 1042438..a98b247 100644
>> --- a/libavcodec/avcodec.h
>> +++ b/libavcodec/avcodec.h
>> @@ -2340,6 +2340,13 @@ typedef struct AVCodecContext {
>>       * Set to time_base ticks per frame. Default 1, e.g., H.264/MPEG-2 set it to 2.
>>       */
>>      int ticks_per_frame;
>> +
>> +    /**
>> +     * Hardware accelerator context. This variable holds read-only data from an FFmpeg point of view.
>> +     * - encoding: unused
>> +     * - decoding: Set by user
>> +     */
>> +    const void *hwaccel_context;
>>  } AVCodecContext;
>
> This is not acceptable, this is not specifiying anything at all
> not who sets it not when no details
> just some void * set by the user but set to what? can he set it to
> "foobar" ?

Here is the new patch. It's the same that I posted earlier but I forgot to 
mark it as "take 2", so you probably missed it. That addition is needed 
for "[PATCH][VAAPI][2/6] Add common data structures and helpers (take 
11)".

Thanks,
Gwenole.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg.hwaccel.context.2.patch
Type: text/x-diff
Size: 849 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090316/c285ed29/attachment.patch>



More information about the ffmpeg-devel mailing list