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

Gwenole Beauchesne gbeauchesne
Tue Feb 3 13:26:13 CET 2009


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".

Thanks,
Gwenole.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 004_h264_record_more_fields.patch
Type: text/x-diff
Size: 5785 bytes
Desc: 
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090203/5eb8148f/attachment.patch>



More information about the ffmpeg-devel mailing list