[FFmpeg-devel] [PATCH][H264] Record extra fields for VA API support (take 5)

Michael Niedermayer michaelni
Tue Feb 3 13:41:37 CET 2009


On Tue, Feb 03, 2009 at 01:26:13PM +0100, Gwenole Beauchesne wrote:
> Hi,
>
> On Tue, 3 Feb 2009, Michael Niedermayer wrote:
>
>>> +    for (int i = 0; i < 2; i++) {
>>
>> doesnt compile on gcc-2.95
>
> I am still wondering who could use an outdated compiler that doesn't even 
> know about current standards. ;-)
>
>>> +    int bit_depth_luma_minus8;         ///< 7.4.2.1.1 
>>> bit_depth_luma_minus8
>>> +    int bit_depth_chroma_minus8;       ///< 7.4.2.1.1 
>>> bit_depth_chroma_minus8
>>
>> remove the _minus8 and store the sane value
>
> OK, patch attached. Also applied vertical alignment suggestions though 
> coding style becomes less consistent IMHO: original and surrounding code is 
> mis-aligned too so this new hunk looks "alone".

patch looks ok

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

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- 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/20090203/2eb38764/attachment.pgp>



More information about the ffmpeg-devel mailing list