commit c86d2bf91afa4dc7c77906888a31b95f94ace130
author Cheng Chen <chengchen@google.com> Thu Sep 05 15:07:47 2019 -0700
committer Cheng Chen <chengchen@google.com> Wed Sep 11 16:53:51 2019 +0000
tree 5d10079bdfcddb8b5ef08de80a8a3bbe264e7768
parent 78b69cb0d35ee6112122737b67320cc9f1bc77bd

Fix update_arf_stack

The INVALID_IDX should be assigned to the last entry.
However, in current codebase, the input ref_map_index is the last,
so there's no behavior change.

Change-Id: I31ac0388723110b5e130bea0b0a811ddddf7e553

av1/encoder/encode_strategy.c

diff --git a/av1/encoder/encode_strategy.c b/av1/encoder/encode_strategy.c
index 28803a9..793a64b 100644
--- a/av1/encoder/encode_strategy.c
+++ b/av1/encoder/encode_strategy.c
@@ -672,7 +672,8 @@
         for (int idx = i; idx < ref_buffer_stack->lst_stack_size - 1; ++idx)
           ref_buffer_stack->lst_stack[idx] =
               ref_buffer_stack->lst_stack[idx + 1];
-        ref_buffer_stack->lst_stack[i] = INVALID_IDX;
+        ref_buffer_stack->lst_stack[ref_buffer_stack->lst_stack_size - 1] =
+            INVALID_IDX;
         --ref_buffer_stack->lst_stack_size;
       }
     }
@@ -684,7 +685,8 @@
         for (int idx = i; idx < ref_buffer_stack->gld_stack_size - 1; ++idx)
           ref_buffer_stack->gld_stack[idx] =
               ref_buffer_stack->gld_stack[idx + 1];
-        ref_buffer_stack->gld_stack[i] = INVALID_IDX;
+        ref_buffer_stack->gld_stack[ref_buffer_stack->gld_stack_size - 1] =
+            INVALID_IDX;
         --ref_buffer_stack->gld_stack_size;
       }
     }