diff options
| author | John Koleszar <jkoleszar@google.com> | 2011-05-06 11:48:50 -0400 |
|---|---|---|
| committer | John Koleszar <jkoleszar@google.com> | 2011-05-06 11:48:53 -0400 |
| commit | abc9958c52dd4ecbb5f7eef3352672aee815db1f (patch) | |
| tree | b4efc9bc252b562984bc38062b63c895f4f1cfcc /vp8/encoder | |
| parent | db5057c742d57daad5d6e18fb0ab2bd4775f325f (diff) | |
| download | libvpx-abc9958c52dd4ecbb5f7eef3352672aee815db1f.tar libvpx-abc9958c52dd4ecbb5f7eef3352672aee815db1f.tar.gz libvpx-abc9958c52dd4ecbb5f7eef3352672aee815db1f.tar.bz2 libvpx-abc9958c52dd4ecbb5f7eef3352672aee815db1f.zip | |
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
Diffstat (limited to 'vp8/encoder')
| -rw-r--r-- | vp8/encoder/ratectrl.c | 3 |
1 files changed, 2 insertions, 1 deletions
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 { |