From a3f71ccff6c514a7396a1a329194e5cda18b6825 Mon Sep 17 00:00:00 2001 From: Paul Wilkins Date: Wed, 26 Jan 2011 12:52:31 +0000 Subject: [PATCH] Correction to buffer update for non-viewable frames. The code previously tested cpi->common.refresh_alt_ref_frame but there are situations where this flag may be set for viewable frames. The correct test should be !cm->show_frame. Change-Id: Ia1a600622992a4a68fe1d38ac23bf6b34b133688 --- vp8/encoder/onyx_if.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/vp8/encoder/onyx_if.c b/vp8/encoder/onyx_if.c index 900785364..d60bf8b52 100644 --- a/vp8/encoder/onyx_if.c +++ b/vp8/encoder/onyx_if.c @@ -4614,7 +4614,8 @@ static void encode_frame_to_data_rate } // Update the buffer level variable. - if (cpi->common.refresh_alt_ref_frame) + // Non-viewable frames are a special case and are treated as pure overhead. + if ( !cm->show_frame ) cpi->bits_off_target -= cpi->projected_frame_size; else cpi->bits_off_target += cpi->av_per_frame_bandwidth - cpi->projected_frame_size;