On 10/27/2018 4:11 PM, Mark Thompson wrote:
> On 26/10/18 20:37, James Almer wrote:
>> Signed-off-by: James Almer <jamr...@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.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to