diff options
author | Jim Bankoski <jimbankoski@google.com> | 2014-02-13 06:19:26 -0800 |
---|---|---|
committer | Gerrit Code Review <gerrit@gerrit.golo.chromium.org> | 2014-02-13 06:19:26 -0800 |
commit | 75ac2913de943dfdeac9dcf98b9cb44a55639109 (patch) | |
tree | f50bb5d5b4aa6b13bcce7a33c51f95069835fe19 /vp9 | |
parent | 8121366fd1c587b7975a416d7d87717010c4eb94 (diff) | |
parent | df86c4497a069eae10f74dd7c9592640a6c9a4d0 (diff) | |
download | libvpx-75ac2913de943dfdeac9dcf98b9cb44a55639109.tar libvpx-75ac2913de943dfdeac9dcf98b9cb44a55639109.tar.gz libvpx-75ac2913de943dfdeac9dcf98b9cb44a55639109.tar.bz2 libvpx-75ac2913de943dfdeac9dcf98b9cb44a55639109.zip |
Merge "clean out static analyzer warnings vp9_encodeframe.c"
Diffstat (limited to 'vp9')
-rw-r--r-- | vp9/encoder/vp9_encodeframe.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/vp9/encoder/vp9_encodeframe.c b/vp9/encoder/vp9_encodeframe.c index 879436aa7..5432c1f45 100644 --- a/vp9/encoder/vp9_encodeframe.c +++ b/vp9/encoder/vp9_encodeframe.c @@ -345,7 +345,6 @@ static void select_in_frame_q_segment(VP9_COMP *cpi, int mi_row, int mi_col, int output_enabled, int projected_rate) { VP9_COMMON *const cm = &cpi->common; - int target_rate = cpi->rc.sb64_target_rate << 8; // convert to bits << 8 const int mi_offset = mi_row * cm->mi_cols + mi_col; const int bw = num_8x8_blocks_wide_lookup[BLOCK_64X64]; @@ -362,7 +361,8 @@ static void select_in_frame_q_segment(VP9_COMP *cpi, } else { // Rate depends on fraction of a SB64 in frame (xmis * ymis / bw * bh). // It is converted to bits * 256 units - target_rate = (cpi->rc.sb64_target_rate * xmis * ymis * 256) / (bw * bh); + const int target_rate = (cpi->rc.sb64_target_rate * xmis * ymis * 256) / + (bw * bh); if (projected_rate < (target_rate / 4)) { segment = 2; @@ -1897,7 +1897,11 @@ static void rd_pick_partition(VP9_COMP *cpi, const TileInfo *const tile, restore_context(cpi, mi_row, mi_col, a, l, sa, sl, bsize); } - + // TODO(jbb): This code added so that we avoid static analysis + // warning related to the fact that best_rd isn't used after this + // point. This code should be refactored so that the duplicate + // checks occur in some sub function and thus are used... + (void) best_rd; *rate = best_rate; *dist = best_dist; |