]> granicus.if.org Git - libvpx/commitdiff
VP9 level targeting: properly handle max_gf_interval
authorhui su <huisu@google.com>
Tue, 20 Jun 2017 23:26:00 +0000 (16:26 -0700)
committerhui su <huisu@google.com>
Tue, 20 Jun 2017 23:29:04 +0000 (16:29 -0700)
Don't overide max_gf_interval if it's not specified. It will
be assigned with a default value in vp9_rc_set_gf_interval_range().

BUG=b/62803416

Change-Id: Ide46ce00279ed076865fc54ce98c55a994f0c798

vp9/vp9_cx_iface.c

index bb6b30bd4678e30bda20f5cf8fc21a71ebd13167..d18457f34e23bdf00785e7586f423b67d0ee8046 100644 (file)
@@ -432,8 +432,12 @@ static void config_target_level(VP9EncoderConfig *oxcf) {
       (int)vp9_level_defs[target_level_index].min_altref_distance) {
     oxcf->min_gf_interval =
         (int)vp9_level_defs[target_level_index].min_altref_distance + 1;
-    oxcf->max_gf_interval =
-        VPXMAX(oxcf->max_gf_interval, oxcf->min_gf_interval);
+    // If oxcf->max_gf_interval == 0, it will be assigned with a default value
+    // in vp9_rc_set_gf_interval_range().
+    if (oxcf->max_gf_interval != 0) {
+      oxcf->max_gf_interval =
+          VPXMAX(oxcf->max_gf_interval, oxcf->min_gf_interval);
+    }
   }
 
   // Adjust maximum column tiles.