ref: abc9958c52dd4ecbb5f7eef3352672aee815db1f
parent: db5057c742d57daad5d6e18fb0ab2bd4775f325f
author: John Koleszar <jkoleszar@google.com>
date: Fri May 6 07:48:50 EDT 2011
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
--- 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
{