From 939f0e7444374fbd4aca2f11b47ae48fde930562 Mon Sep 17 00:00:00 2001 From: Marco Paniconi Date: Tue, 1 Apr 2014 17:50:14 -0700 Subject: [PATCH] Remove the q division by 3 for first frame. This code/setting was actually not used (since speed features were not set on first frame, until a recent change) and should be removed. In CBR mode, the q value for the first frame can be controlled by setting the target size via the parameters rc_buf_initial_sz (and max_intra_size_pct). Change-Id: I65afc64972b36c449bd5a8c25800e65da5389066 --- vp9/encoder/vp9_ratectrl.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/vp9/encoder/vp9_ratectrl.c b/vp9/encoder/vp9_ratectrl.c index 3c8baad79..78cf29e86 100644 --- a/vp9/encoder/vp9_ratectrl.c +++ b/vp9/encoder/vp9_ratectrl.c @@ -977,12 +977,9 @@ int vp9_rc_pick_q_and_bounds(const VP9_COMP *cpi, q = rc_pick_q_and_bounds_two_pass(cpi, bottom_index, top_index); } - // JBB : This is realtime mode. In real time mode the first frame - // should be larger. Q of 0 is disabled because we force tx size to be + // Q of 0 is disabled because we force tx size to be // 16x16... if (cpi->sf.use_nonrd_pick_mode) { - if (cpi->common.current_video_frame == 0) - q /= 3; if (q == 0) q++; if (cpi->sf.partition_check == 1) -- 2.40.0