[FFmpeg-devel] [PATCH 1/2] avcodec/cbs_vp9: fix parsing sRGB samples

James Almer jamrial at gmail.com
Sat Oct 27 22:13:49 EEST 2018


On 10/27/2018 4:11 PM, Mark Thompson wrote:
> On 26/10/18 20:37, James Almer wrote:
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>>  libavcodec/cbs_vp9_syntax_template.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/libavcodec/cbs_vp9_syntax_template.c b/libavcodec/cbs_vp9_syntax_template.c
>> index 0db0f52a6d..b4a7f65e85 100644
>> --- a/libavcodec/cbs_vp9_syntax_template.c
>> +++ b/libavcodec/cbs_vp9_syntax_template.c
>> @@ -65,6 +65,7 @@ static int FUNC(color_config)(CodedBitstreamContext *ctx, RWContext *rw,
>>          if (profile == 1 || profile == 3) {
>>              infer(subsampling_x, 0);
>>              infer(subsampling_y, 0);
>> +            f(1, color_config_reserved_zero);
>>          }
>>      }
>>  
>>
> 
> This and the one above should probably be called "reserved_zero" for trace purposes.  Not sure where the longer name came from.
> 

Yeah, the spec simply says reserved_zero, so i'll change it.

> LGTM in any case.
> 
> Thanks,
> 
> - Mark

Pushed, thanks.


More information about the ffmpeg-devel mailing list