From 838eae39617d402774b8e22744172b4d723e532f Mon Sep 17 00:00:00 2001 From: Yaowu Xu Date: Mon, 23 Sep 2013 16:07:31 -0700 Subject: [PATCH] Correct 3 step search site initialziation 39c7b01d accidently reverted the row/col initialization, which broke mv clamps, which is dependent on the sites for valid motion vector range. This commit fixed the issue. Change-Id: Ibcce0226e0360b1ef483fe760b2e33f1af4bf494 --- vp9/encoder/vp9_mcomp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/vp9/encoder/vp9_mcomp.c b/vp9/encoder/vp9_mcomp.c index bc20817b0..7dd786904 100644 --- a/vp9/encoder/vp9_mcomp.c +++ b/vp9/encoder/vp9_mcomp.c @@ -144,8 +144,8 @@ void vp9_init3smotion_compensation(MACROBLOCK *x, int stride) { for (len = MAX_FIRST_STEP; len > 0; len /= 2) { // Generate offsets for 8 search sites per step. const MV ss_mvs[8] = { - { 0, -len}, {0, len}, {-len, 0 }, {len, 0 }, - {-len, -len}, {len, -len}, {-len, len}, {len, len} + {-len, 0 }, {len, 0 }, { 0, -len}, {0, len}, + {-len, -len}, {-len, len}, {len, -len}, {len, len} }; int i; for (i = 0; i < 8; ++i) {