Initialize mb_skip_coeff before picking modes
authorScott LaVarnway <slavarnway@google.com>
Thu, 22 Aug 2013 12:51:04 +0000 (08:51 -0400)
committerScott LaVarnway <slavarnway@google.com>
Thu, 22 Aug 2013 12:51:04 +0000 (08:51 -0400)
It appears that the above/left mb_skip_coeff used during
the pick modes, is left over from the previously
encode frame.  This patch initializes the flag to the default
value of zero.

Change-Id: Ida4684cc99611d6e3e82628db35ed717e28ce550

vp9/encoder/vp9_encodeframe.c

index 51bd447..a65b1d5 100644 (file)
@@ -590,7 +590,11 @@ static void pick_sb_modes(VP9_COMP *cpi, int mi_row, int mi_col,
   set_offsets(cpi, mi_row, mi_col, bsize);
   xd->mode_info_context->mbmi.sb_type = bsize;
 
+  // Set to zero to make sure we do not use the previous encoded frame stats
+  xd->mode_info_context->mbmi.skip_coeff = 0;
+
   x->source_variance = get_sby_perpixel_variance(cpi, x, bsize);
+
   if (cpi->oxcf.tuning == VP8_TUNE_SSIM)
     vp9_activity_masking(cpi, x);