commit 0c8c0fcb4d442071a1232c48b46b7bd0d4cdf3b5
author Yunqing Wang <yunqingwang@google.com> Thu Apr 09 21:52:38 2020 -0700
committer Yunqing Wang <yunqingwang@google.com> Fri Apr 10 18:22:31 2020 +0000
tree 918d551339340d3b35aed0459451549a2ef707c6
parent 2df76f2f41b505c736999e1750699760d5b09829

Modify assigning of second_best_mv

Minor adjustment.

BUG=aomedia:2629

STATS_CHANGED

Change-Id: I22b1b10623141a1dd3616abd1eb017760ee6f2f9

av1/encoder/motion_search_facade.c

diff --git a/av1/encoder/motion_search_facade.c b/av1/encoder/motion_search_facade.c
index abfbad7..0219bc4 100644
--- a/av1/encoder/motion_search_facade.c
+++ b/av1/encoder/motion_search_facade.c
@@ -222,17 +222,16 @@
             smv, &full_ms_params, step_param, cond_cost_list(cpi, cost_list),
             &x->best_mv.as_fullmv, &second_best_mv.as_fullmv);
 
-        // Use first search's second_best_mv
-        // TODO(yunqing): second_best_mv decision will be improved later.
-        if (!m) second_best_mv0 = second_best_mv.as_fullmv;
-
         if (!m || bestsme < cur_bestsme) {
           cur_bestsme = bestsme;
           best_mv = x->best_mv.as_fullmv;
+          second_best_mv0 = second_best_mv.as_fullmv;
         }
 
         sum_weight += cand[m].weight;
         if (m >= 2 || 4 * sum_weight > 3 * total_weight) break;
+        // Reset MVs
+        x->best_mv.as_int = second_best_mv.as_int = INVALID_MV;
       }
       x->best_mv.as_fullmv = best_mv;
       bestsme = cur_bestsme;