tree f84e4bcf892b4abb29b57812630d3e08ce0c1a24
parent 83684e11569193dd1d4b411ed32d9ce9ab12104c
author Urvang Joshi <urvang@google.com> 1553030917 -0700
committer Urvang Joshi <urvang@google.com> 1553098112 +0000

Bugfix for gf-max-pyr-height option.

oxcf->gf_max_pyr_height was being set to the appropriate value *after*
av1_rc_init() was already called. So, rc->max_gf_length was always 16
earlier, irrespective of gf_max_pyr_height value passed in by the user.

This inconsistency was causing the creation of an invalid bitstream.

Now, we set rc->max_gf_length independently of oxcf->gf_max_pyr_height.
And, in define_gf_group() call, when both are set to their final values,
we pick a max_gf_length that adheres to both the constraints.

Also, for max height = 1 in particular, we make sure that extra arfs are
not allowed.

With this, max height of 1, 2 and 3 work correctly. (4 was working fine
already).

No change in stats for default option (gf_max_pyr_height = 4)

BUG=aomedia:2345

Change-Id: I5abfc8f98e93f4f4c4af6b9d969763b4f48b7196