shithub: libvpx

Download patch

ref: dcd3a5c0552069ffa728c9fc79abb1dd130551a4
parent: 30d83c415989e67f4efb55277c2901151eb8dcaf
parent: 4ca882f32fb7f45044ccf01db0f4a40c52e5fcca
author: Yunqing Wang <yunqingwang@google.com>
date: Wed Apr 3 10:09:01 EDT 2013 

Merge "Modify vp9_setup_interp_filters function" into experimental


--- a/vp9/common/vp9_reconinter.c
+++ b/vp9/common/vp9_reconinter.c
@@ -206,20 +206,6 @@
 void vp9_setup_interp_filters(MACROBLOCKD *xd,
                               INTERPOLATIONFILTERTYPE mcomp_filter_type,
                               VP9_COMMON *cm) {
-  int i;
-
-  /* Calculate scaling factors for each of the 3 available references */
-  for (i = 0; i < 3; ++i) {
-    if (cm->active_ref_idx[i] >= NUM_YV12_BUFFERS) {
-      memset(&cm->active_ref_scale[i], 0, sizeof(cm->active_ref_scale[i]));
-      continue;
-    }
-
-    vp9_setup_scale_factors_for_frame(&cm->active_ref_scale[i],
-                                      &cm->yv12_fb[cm->active_ref_idx[i]],
-                                      cm->width, cm->height);
-  }
-
   if (xd->mode_info_context) {
     MB_MODE_INFO *mbmi = &xd->mode_info_context->mbmi;
 
@@ -228,7 +214,6 @@
                       mbmi->second_ref_frame - 1,
                       cm->active_ref_scale);
   }
-
 
   switch (mcomp_filter_type) {
     case EIGHTTAP:
--- a/vp9/decoder/vp9_decodframe.c
+++ b/vp9/decoder/vp9_decodframe.c
@@ -1103,13 +1103,6 @@
     vp9_setup_past_independence(pc, xd);
   }
 
-  if (pc->frame_type != KEY_FRAME) {
-    pc->mcomp_filter_type = pc->use_bilinear_mc_filter ? BILINEAR : EIGHTTAP;
-
-    // To enable choice of different interpolation filters
-    vp9_setup_interp_filters(xd, pc->mcomp_filter_type, pc);
-  }
-
   xd->mode_info_context = pc->mi;
   xd->prev_mode_info_context = pc->prev_mi;
   xd->frame_type = pc->frame_type;
@@ -1767,6 +1760,19 @@
 #if CONFIG_COMP_INTERINTRA_PRED
     pc->use_interintra = vp9_read_bit(&header_bc);
 #endif
+
+    /* Calculate scaling factors for each of the 3 available references */
+    for (i = 0; i < 3; ++i) {
+      if (pc->active_ref_idx[i] >= NUM_YV12_BUFFERS) {
+        memset(&pc->active_ref_scale[i], 0, sizeof(pc->active_ref_scale[i]));
+        continue;
+      }
+
+      vp9_setup_scale_factors_for_frame(&pc->active_ref_scale[i],
+                                        &pc->yv12_fb[pc->active_ref_idx[i]],
+                                        pc->width, pc->height);
+    }
+
     // To enable choice of different interploation filters
     vp9_setup_interp_filters(xd, pc->mcomp_filter_type, pc);
   }
--- a/vp9/encoder/vp9_onyx_if.c
+++ b/vp9/encoder/vp9_onyx_if.c
@@ -3640,6 +3640,7 @@
   VP9_COMMON *cm = &cpi->common;
   struct vpx_usec_timer  cmptimer;
   YV12_BUFFER_CONFIG    *force_src_buffer = NULL;
+  int i;
 
   if (!cpi)
     return -1;
@@ -3784,7 +3785,20 @@
                                 cm->width, cm->height,
                                 VP9BORDERINPIXELS);
 
+  /* Calculate scaling factors for each of the 3 available references */
+  for (i = 0; i < 3; ++i) {
+    if (cm->active_ref_idx[i] >= NUM_YV12_BUFFERS) {
+      memset(&cm->active_ref_scale[i], 0, sizeof(cm->active_ref_scale[i]));
+      continue;
+    }
+
+    vp9_setup_scale_factors_for_frame(&cm->active_ref_scale[i],
+                                      &cm->yv12_fb[cm->active_ref_idx[i]],
+                                      cm->width, cm->height);
+  }
+
   vp9_setup_interp_filters(&cpi->mb.e_mbd, DEFAULT_INTERP_FILTER, cm);
+
   if (cpi->pass == 1) {
     Pass1Encode(cpi, size, dest, frame_flags);
   } else if (cpi->pass == 2) {