From: Henrik Gramner Date: Thu, 28 Jul 2016 17:33:57 +0000 (+0200) Subject: ratecontrol_new: Simplify an expression in HRD timescale calculation X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=1e4fb55a283ba90fef346033027af851f2a04468;p=libx264 ratecontrol_new: Simplify an expression in HRD timescale calculation Also gets rid of a false positive static analyser integer division warning. --- diff --git a/encoder/ratecontrol.c b/encoder/ratecontrol.c index 99277f1d..301ba0f7 100644 --- a/encoder/ratecontrol.c +++ b/encoder/ratecontrol.c @@ -786,7 +786,7 @@ int x264_ratecontrol_new( x264_t *h ) x264_reduce_fraction64( &num, &denom ); rc->hrd_multiply_denom = 90000 / num; - double bits_required = log2( 90000 / rc->hrd_multiply_denom ) + double bits_required = log2( num ) + log2( h->sps->vui.i_time_scale ) + log2( h->sps->vui.hrd.i_cpb_size_unscaled ); if( bits_required >= 63 )