Remove unreachable block in CStretchEnging::Calc().
Having sorted start_i and end_i a few lines above, there isn't a chance
for these to be out-of-order and enter the block removed by this CL. It
is a good thing, too, since there aren't any weights set up despite
setting start and end bounds, which should theoretically lead to a
convolution using uninitialized memory. Our code coverage numbers back
up the claim of this being unreached.
Change-Id: I43fe807d843b21c772704c885dd83bfe27ca70a3
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/82011
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
diff --git a/core/fxge/dib/cstretchengine.cpp b/core/fxge/dib/cstretchengine.cpp
index 86d29ed..e2b9d86 100644
--- a/core/fxge/dib/cstretchengine.cpp
+++ b/core/fxge/dib/cstretchengine.cpp
@@ -115,11 +115,6 @@
int end_i = floor(std::max(src_start, src_end));
start_i = std::max(start_i, src_min);
end_i = std::min(end_i, src_max - 1);
- if (start_i > end_i) {
- start_i = std::min(start_i, src_max - 1);
- pixel_weights.SetStartEnd(start_i, start_i, weight_count);
- continue;
- }
pixel_weights.SetStartEnd(start_i, end_i, weight_count);
for (int j = start_i; j <= end_i; ++j) {
double dest_start = (j - base) / scale;
