From: John Koleszar <jkoleszar@google.com>
Date: Tue, 21 Sep 2010 15:54:36 +0000 (-0400)
Subject: Don't reset mb clamping state during splitmv decoding
X-Git-Tag: 1.0_branch~910^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4d391e8ed281c58bfa36e864050091f2934a7097;p=profile%2Fivi%2Flibvpx.git

Don't reset mb clamping state during splitmv decoding

The MV decoding changes in c5fb0eb introduced a bug where the
macroblock clamping state was reset for each partition, so if an
earlier partition needed clamping but a subsequent one didn't,
the MB wouldn't receive clamping. Instead, the state is only
set during splitmv decoding, never cleared.

Change-Id: I224fe258493405ee0f6a04596acdb622c475e845
---

diff --git a/vp8/decoder/decodemv.c b/vp8/decoder/decodemv.c
index e9281f7..106dbde 100755
--- a/vp8/decoder/decodemv.c
+++ b/vp8/decoder/decodemv.c
@@ -394,7 +394,7 @@ void vp8_read_mb_modes_mv(VP8D_COMP *pbi, MODE_INFO *mi, MB_MODE_INFO *mbmi,
                     break;
                 }
 
-                mbmi->need_to_clamp_mvs = (mv->col < mb_to_left_edge) ? 1 : 0;
+                mbmi->need_to_clamp_mvs |= (mv->col < mb_to_left_edge) ? 1 : 0;
                 mbmi->need_to_clamp_mvs |= (mv->col > mb_to_right_edge) ? 1 : 0;
                 mbmi->need_to_clamp_mvs |= (mv->row < mb_to_top_edge) ? 1 : 0;
                 mbmi->need_to_clamp_mvs |= (mv->row > mb_to_bottom_edge) ? 1 : 0;