[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