From dcbe0820709c274beb4624588d1aa14b25557d62 Mon Sep 17 00:00:00 2001 From: Wan-Teh Chang Date: Sat, 20 May 2023 16:18:10 -0700 Subject: [PATCH] Save boxwidth - minboxwidth in a local variable Avoid repetitions of the expression boxwidth - minboxwidth. Change-Id: Ib53fb6b06a926b80ff9a64cc5d499aeef0894c99 Reviewed-on: https://chromium-review.googlesource.com/c/libyuv/libyuv/+/4408062 Commit-Queue: Frank Barchard Reviewed-by: Frank Barchard --- source/scale.cc | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/source/scale.cc b/source/scale.cc index 90abaefe8..591a6a938 100644 --- a/source/scale.cc +++ b/source/scale.cc @@ -820,9 +820,10 @@ static void ScaleAddCols2_C(int dst_width, int ix = x >> 16; x += dx; boxwidth = MIN1((x >> 16) - ix); - assert(((boxwidth - minboxwidth) == 0) || ((boxwidth - minboxwidth) == 1)); + int scaletbl_index = boxwidth - minboxwidth; + assert((scaletbl_index == 0) || (scaletbl_index == 1)); *dst_ptr++ = (uint8_t)(SumPixels(boxwidth, src_ptr + ix) * - scaletbl[boxwidth - minboxwidth] >> + scaletbl[scaletbl_index] >> 16); } } @@ -843,10 +844,10 @@ static void ScaleAddCols2_16_C(int dst_width, int ix = x >> 16; x += dx; boxwidth = MIN1((x >> 16) - ix); - assert(((boxwidth - minboxwidth) == 0) || ((boxwidth - minboxwidth) == 1)); - *dst_ptr++ = SumPixels_16(boxwidth, src_ptr + ix) * - scaletbl[boxwidth - minboxwidth] >> - 16; + int scaletbl_index = boxwidth - minboxwidth; + assert((scaletbl_index == 0) || (scaletbl_index == 1)); + *dst_ptr++ = + SumPixels_16(boxwidth, src_ptr + ix) * scaletbl[scaletbl_index] >> 16; } }