From: Paul Wilkins <paulwilkins@google.com>
Date: Wed, 26 Jan 2011 12:52:31 +0000 (+0000)
Subject: Correction to buffer update for non-viewable frames.
X-Git-Tag: v0.9.6~83^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a3f71ccff6c514a7396a1a329194e5cda18b6825;p=libvpx

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
---

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;