[FFmpeg-devel] [PATCH][VAAPI][5/6] Add VC-1 bitstream decoding (take 8)
Benjamin Larsson
banan
Thu Apr 2 16:55:25 CEST 2009
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
More information about the ffmpeg-devel
mailing list