ref: 0a6e42c241ba96448dbe57665f254c8c24430c8c
parent: 6e6f5881d8f23cee4d21d4ae86d59363713f3bf4
author: Dmitry Kovalev <[email protected]>
date: Thu May 22 08:17:18 EDT 2014
Adding several consts to assign_std_frame_bits(). Change-Id: I6c27c60f7192b1b397f01882ab68a68cdf767534
--- a/vp9/encoder/vp9_firstpass.c
+++ b/vp9/encoder/vp9_firstpass.c
@@ -1769,23 +1769,14 @@
const int max_bits = frame_max_bits(&cpi->rc, oxcf);
// Calculate modified prediction error used in bit allocation.
const double modified_err = calculate_modified_err(twopass, oxcf, this_frame);
- int target_frame_size;
- double err_fraction;
-
- if (twopass->gf_group_error_left > 0)
- // What portion of the remaining GF group error is used by this frame.
- err_fraction = modified_err / twopass->gf_group_error_left;
- else
- err_fraction = 0.0;
-
- // How many of those bits available for allocation should we give it?
- target_frame_size = (int)((double)twopass->gf_group_bits * err_fraction);
-
- // Clip target size to 0 - max_bits (or cpi->twopass.gf_group_bits) at
- // the top end.
- target_frame_size = clamp(target_frame_size, 0,
- MIN(max_bits, (int)twopass->gf_group_bits));
-
+ // What portion of the remaining GF group error is used by this frame.
+ const double err_fraction = twopass->gf_group_error_left > 0 ?
+ modified_err / twopass->gf_group_error_left : 0.0;
+ // How many of those bits available for allocation should we give it? Clip
+ // target size to 0 - max_bits (or cpi->twopass.gf_group_bits) at the top end.
+ const int target_frame_size =
+ clamp((int)(twopass->gf_group_bits * err_fraction),
+ 0, MIN(max_bits, (int)twopass->gf_group_bits));
// Adjust error and bits remaining.
twopass->gf_group_error_left -= (int64_t)modified_err;