From 94bfbaa84ea956e733d4eb9602d589f9d3577b49 Mon Sep 17 00:00:00 2001
From: Scott LaVarnway <slavarnway@google.com>
Date: Thu, 22 Aug 2013 08:51:04 -0400
Subject: [PATCH] Initialize mb_skip_coeff before picking modes

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 | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/vp9/encoder/vp9_encodeframe.c b/vp9/encoder/vp9_encodeframe.c
index 51bd4477de..a65b1d5c26 100644
--- a/vp9/encoder/vp9_encodeframe.c
+++ b/vp9/encoder/vp9_encodeframe.c
@@ -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);
 
-- 
GitLab