[FFmpeg-devel] [PATCH] H.264: Record slice_qs_delta syntax element

[Gwenole Beauchesne]

Le 20 d?c. 09 ? 16:52, Michael Niedermayer a ?crit :

>> diff --git a/libavcodec/h264.h b/libavcodec/h264.h
>> index fa348a2..f624bc3 100644
>> --- a/libavcodec/h264.h
>> +++ b/libavcodec/h264.h
>> @@ -545,6 +545,8 @@ typedef struct H264Context{
>>     // Timestamp stuff
>>     int sei_buffering_period_present;  ///< Buffering period SEI flag
>>     int initial_cpb_removal_delay[32]; ///< Initial timestamps for  
>> CPBs
>> +
>> +    int slice_qs_delta; ///< Initial value of QSy
>
> slice_qs_delta is not the initial value of QSy

I probably have not copied the right comment in the first place (this  
is a local edit of the previous patch). However, the code is correct  
because the HW does expect the bitstream value, i.e. the one we  
get_bits() from.