From abc9958c52dd4ecbb5f7eef3352672aee815db1f Mon Sep 17 00:00:00 2001 From: John Koleszar Date: Fri, 6 May 2011 11:48:50 -0400 Subject: [PATCH] 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 | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/vp8/encoder/ratectrl.c b/vp8/encoder/ratectrl.c index d87591cb9..313e6455c 100644 --- a/vp8/encoder/ratectrl.c +++ b/vp8/encoder/ratectrl.c @@ -416,7 +416,8 @@ static void calc_iframe_target_size(VP8_COMP *cpi) // 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 {