[FFmpeg-devel] [PATCH][VAAPI][5/6] Add VC-1 bitstream decoding (take 8)

Benjamin Larsson banan
Thu Apr 2 17:07:06 CEST 2009


Gwenole Beauchesne wrote:
> On Thu, 2 Apr 2009, Benjamin Larsson wrote:
>
>>>>>>>> New patch attached. Also made sure B/BI-types were correctly 
>>>>>>>> checked for.
>>>>>>>>
>>>>>>> looks ok but you may also need to get an approval from Michael 
>>>>>>> since this
>>>>>>> is a new file and not the stuff maintained by me
>>>>>>>
>>>>>> Ping. Here is the same patch minus some assert()'s.
>>>>>>
>>>>> Ping. Here is the same (functionally) patch minus single line 
>>>>> functions and some vertical alignment "compression".
>>>>>
>>>> Ping. Here is a new patch to cope with new VA API buffers 
>>>> allocation logic and a cosmetics change (use a temp var to retrieve 
>>>> the MpegEncContext in a cast-less way).
>>>>
>>>> Michael? Kostya already approved the initial patch:
>>>>
>>>
>>> i leave this review entirely to kostya
>>>
>>> [...]
>>
>> Just a note, I think this review will take a little bit of time. Be 
>> patient.
>
> What do you mean?


What I mean is that any input from Kostya regarding anything might take 
a while.

MvH
Benjamin Larsson





More information about the ffmpeg-devel mailing list