commit a4bed750abd6904d54f416b22d2c1aa5337332f6
author Urvang Joshi <urvang@google.com> Fri Jun 12 16:34:22 2020 -0700
committer Urvang Joshi <urvang@google.com> Mon Jun 15 17:18:05 2020 +0000
tree 4f8444694ba2903afca5516541021b825b580c3f
parent 8db5ca98460ae4843e657ef53abd26632d6ce07f

Bugfix for frame type with kf min/max dist of 0

All frames should be keyframes when "--kf-min-dist=0 --kf-max-dist=0" is
given.

But earlier, only the 1st frame was keyframe in this case.

BUG=aomedia:2725

Change-Id: I376eda9be242cc8178a59f023bd00a2219c5a74b

av1/encoder/pass2_strategy.c

diff --git a/av1/encoder/pass2_strategy.c b/av1/encoder/pass2_strategy.c
index 8cae53c..18dd606 100644
--- a/av1/encoder/pass2_strategy.c
+++ b/av1/encoder/pass2_strategy.c
@@ -2646,7 +2646,8 @@
   }
 
   // Keyframe and section processing.
-  if (rc->frames_to_key == 0 || (frame_flags & FRAMEFLAGS_KEY)) {
+  if (rc->frames_to_key <= 0 || (frame_flags & FRAMEFLAGS_KEY)) {
+    assert(rc->frames_to_key >= -1);
     FIRSTPASS_STATS this_frame_copy;
     this_frame_copy = this_frame;
     frame_params->frame_type = KEY_FRAME;