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

[Benjamin Larsson]

Michael Niedermayer wrote:
> On Thu, Apr 02, 2009 at 10:24:14AM +0200, Gwenole Beauchesne wrote:
>   
>> On Fri, 27 Mar 2009, Gwenole Beauchesne wrote:
>>
>>     
>>> On Thu, 26 Mar 2009, Gwenole Beauchesne wrote:
>>>
>>>       
>>>> On Mon, 23 Mar 2009, Kostya 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.

MvH
Benjamin Larsson