From 3c58aa0bf51c64eb126be165e7478e70fbb68043 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Thu, 11 May 2017 01:20:26 -0700 Subject: Use clamp() in a couple more places. Change-Id: I9b7a1c101e3c73d0270f9216225e5a13d9937b97 Reviewed-on: https://pdfium-review.googlesource.com/5332 Reviewed-by: Tom Sepez Commit-Queue: Lei Zhang --- core/fpdfapi/render/cpdf_dibsource.cpp | 3 +-- core/fxge/dib/cfx_imagetransformer.cpp | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/core/fpdfapi/render/cpdf_dibsource.cpp b/core/fpdfapi/render/cpdf_dibsource.cpp index c2156a1555..f6009674c5 100644 --- a/core/fpdfapi/render/cpdf_dibsource.cpp +++ b/core/fpdfapi/render/cpdf_dibsource.cpp @@ -1399,8 +1399,7 @@ void CPDF_DIBSource::DownSampleScanline32Bit(int orig_Bpp, m_pCompData[j].m_DecodeStep * component_value) * 255.0f + 0.5f); - extracted_components[j] = - color_value > 255 ? 255 : (color_value < 0 ? 0 : color_value); + extracted_components[j] = pdfium::clamp(color_value, 0, 255); } m_pColorSpace->TranslateImageLine(color, extracted_components, 1, 0, 0, bTransMask); diff --git a/core/fxge/dib/cfx_imagetransformer.cpp b/core/fxge/dib/cfx_imagetransformer.cpp index 4b5fe0b0e0..11f5f56ecb 100644 --- a/core/fxge/dib/cfx_imagetransformer.cpp +++ b/core/fxge/dib/cfx_imagetransformer.cpp @@ -57,7 +57,7 @@ uint8_t bicubic_interpol(const uint8_t* buf, s_result += a_result * v_w[i]; } s_result >>= 16; - return (uint8_t)(s_result < 0 ? 0 : s_result > 255 ? 255 : s_result); + return static_cast(pdfium::clamp(s_result, 0, 255)); } void bicubic_get_pos_weight(int pos_pixel[], -- cgit v1.2.3