commit 0b19d721ea2c6e2332387959b639cb4113d703cc
author Remy Foray <remy.foray@allegrodvt.com> Tue Nov 13 17:29:49 2018 +0100
committer Remy Foray <remy.foray@allegrodvt.com> Wed Nov 14 16:06:53 2018 +0000
tree 4678765dceefc04d4f8d98911f6620708627b71f
parent e3f75709b30a787a078e08ea8a0e1a6d7598d1fe

Add missing mv compliance check when skip mode is used

Harmonize mv setting (remove skip particular case), this allows to always
call is_valid_mv.

Change-Id: Idf9c796d43e01e5afb2240ce60deaa356fd15bdb

av1/decoder/decodemv.c

diff --git a/av1/decoder/decodemv.c b/av1/decoder/decodemv.c
index 1cf1ac3..d546936 100644
--- a/av1/decoder/decodemv.c
+++ b/av1/decoder/decodemv.c
@@ -1350,11 +1350,9 @@
     }
   }
 
-  if (mbmi->skip_mode) {
-    assert(mbmi->mode == NEAREST_NEARESTMV);
-    mbmi->mv[0].as_int = nearestmv[0].as_int;
-    mbmi->mv[1].as_int = nearestmv[1].as_int;
-  } else {
+  if (mbmi->skip_mode) assert(mbmi->mode == NEAREST_NEARESTMV);
+
+  {
     int mv_corrupted_flag =
         !assign_mv(cm, xd, mbmi->mode, mbmi->ref_frame, mbmi->mv, ref_mv,
                    nearestmv, nearmv, mi_row, mi_col, is_compound, allow_hp, r);