commit abc9958c52dd4ecbb5f7eef3352672aee815db1f
author John Koleszar <jkoleszar@google.com> Fri May 06 11:48:50 2011 -0400
committer John Koleszar <jkoleszar@google.com> Fri May 06 11:48:53 2011 -0400
tree b4efc9bc252b562984bc38062b63c895f4f1cfcc
parent db5057c742d57daad5d6e18fb0ab2bd4775f325f

Don't override active_worst_quality in 2 pass

Commit db5057c introduced a bug in that the active_worst_quality
selected by the 2 pass rate controller was being overridden for key
frames, causing a severe quality loss.

Change-Id: I4865a6fbe3e94e9b4fb9271c7dd68b455d7b371d

vp8/encoder/ratectrl.c

diff --git a/vp8/encoder/ratectrl.c b/vp8/encoder/ratectrl.c
index d87591c..313e645 100644
--- a/vp8/encoder/ratectrl.c
+++ b/vp8/encoder/ratectrl.c
@@ -416,7 +416,8 @@
 
     // TODO: if we separate rate targeting from Q targetting, move this.
     // Reset the active worst quality to the baseline value for key frames.
-    cpi->active_worst_quality = cpi->worst_quality;
+    if (cpi->pass != 2)
+        cpi->active_worst_quality = cpi->worst_quality;
 
 #if 0
     {