shithub: libvpx

Download patch

ref: 0de8be5a777a76b39d5c201c1b807fae81c9ea9d
parent: ba1a52b9cb81b4dbeccdfbcfca6a821846c61758
author: Jingning Han <jingning@google.com>
date: Mon Aug 20 13:03:07 EDT 2018

Unify set_arf_sign_bias function

Determine if an ARF is on the future side by checking if its
offset meets the gop frame length. This unifies the support to
single- and multiple-layer ARF cases.

Change-Id: I5ab26f54311c345a9b574ffca5ff0a8dbcf4c031

--- a/vp9/encoder/vp9_encoder.c
+++ b/vp9/encoder/vp9_encoder.c
@@ -4517,18 +4517,14 @@
 
 static void set_arf_sign_bias(VP9_COMP *cpi) {
   VP9_COMMON *const cm = &cpi->common;
-  int arf_sign_bias;
+  const GF_GROUP *const gf_group = &cpi->twopass.gf_group;
+  const int gfg_index = gf_group->index;
 
-  if ((cpi->oxcf.pass == 2) && cpi->multi_arf_allowed) {
-    const GF_GROUP *const gf_group = &cpi->twopass.gf_group;
-    arf_sign_bias = cpi->rc.source_alt_ref_active &&
-                    (!cpi->refresh_alt_ref_frame ||
-                     (gf_group->rf_level[gf_group->index] == GF_ARF_LOW));
-  } else {
-    arf_sign_bias =
-        (cpi->rc.source_alt_ref_active && !cpi->refresh_alt_ref_frame);
-  }
-  cm->ref_frame_sign_bias[ALTREF_FRAME] = arf_sign_bias;
+  // If arf_src_offset is less than the GOP length, its arf is on the direction
+  // 1 side.
+  cm->ref_frame_sign_bias[ALTREF_FRAME] =
+      cpi->rc.source_alt_ref_active &&
+      gf_group->arf_src_offset[gfg_index] < cpi->rc.baseline_gf_interval - 1;
 }
 
 static int setup_interp_filter_search_mask(VP9_COMP *cpi) {