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