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

Michael Niedermayer michaelni
Thu Apr 2 16:13:27 CEST 2009


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

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

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- 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/20090402/83a1475e/attachment.pgp>



More information about the ffmpeg-devel mailing list