summaryrefslogtreecommitdiff
path: root/core/fxcrt/fx_coordinates.cpp
diff options
context:
space:
mode:
authorLei Zhang <thestig@chromium.org>2017-12-04 21:24:15 +0000
committerChromium commit bot <commit-bot@chromium.org>2017-12-04 21:24:15 +0000
commit12ec6760afd92b63d185854008a55762fe39f866 (patch)
tree77c68ccfba27109dd6de1dcb01848693272a3790 /core/fxcrt/fx_coordinates.cpp
parentcfc8b87d5242f1493a54ba4f1125f2b8ef1791bd (diff)
downloadpdfium-12ec6760afd92b63d185854008a55762fe39f866.tar.xz
Simplify some CFX_FloatRect methods.chromium/3285
Also add a comment about the various CFX_FloatRect to FX_RECT conversion methods. Change-Id: Ia9984797dc513cdc487fe9972b32c216c9f99ec1 Reviewed-on: https://pdfium-review.googlesource.com/20217 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'core/fxcrt/fx_coordinates.cpp')
-rw-r--r--core/fxcrt/fx_coordinates.cpp29
1 files changed, 13 insertions, 16 deletions
diff --git a/core/fxcrt/fx_coordinates.cpp b/core/fxcrt/fx_coordinates.cpp
index 561f25d4a6..733425ef26 100644
--- a/core/fxcrt/fx_coordinates.cpp
+++ b/core/fxcrt/fx_coordinates.cpp
@@ -17,10 +17,10 @@ void MatchFloatRange(float f1, float f2, int* i1, int* i2) {
int length = static_cast<int>(ceil(f2 - f1));
int i1_1 = static_cast<int>(floor(f1));
int i1_2 = static_cast<int>(ceil(f1));
- float error1 = f1 - i1_1 + (float)fabs(f2 - i1_1 - length);
- float error2 = i1_2 - f1 + (float)fabs(f2 - i1_2 - length);
+ float error1 = f1 - i1_1 + fabsf(f2 - i1_1 - length);
+ float error2 = i1_2 - f1 + fabsf(f2 - i1_2 - length);
- *i1 = (error1 > error2) ? i1_2 : i1_1;
+ *i1 = error1 > error2 ? i1_2 : i1_1;
*i2 = *i1 + length;
}
@@ -108,32 +108,29 @@ void CFX_FloatRect::Union(const CFX_FloatRect& other_rect) {
}
FX_RECT CFX_FloatRect::GetOuterRect() const {
- CFX_FloatRect rect1 = *this;
FX_RECT rect;
- rect.left = static_cast<int>(floor(rect1.left));
- rect.bottom = static_cast<int>(ceil(rect1.top));
- rect.right = static_cast<int>(ceil(rect1.right));
- rect.top = static_cast<int>(floor(rect1.bottom));
+ rect.left = static_cast<int>(floor(left));
+ rect.bottom = static_cast<int>(ceil(top));
+ rect.right = static_cast<int>(ceil(right));
+ rect.top = static_cast<int>(floor(bottom));
rect.Normalize();
return rect;
}
FX_RECT CFX_FloatRect::GetInnerRect() const {
- CFX_FloatRect rect1 = *this;
FX_RECT rect;
- rect.left = static_cast<int>(ceil(rect1.left));
- rect.bottom = static_cast<int>(floor(rect1.top));
- rect.right = static_cast<int>(floor(rect1.right));
- rect.top = static_cast<int>(ceil(rect1.bottom));
+ rect.left = static_cast<int>(ceil(left));
+ rect.bottom = static_cast<int>(floor(top));
+ rect.right = static_cast<int>(floor(right));
+ rect.top = static_cast<int>(ceil(bottom));
rect.Normalize();
return rect;
}
FX_RECT CFX_FloatRect::GetClosestRect() const {
- CFX_FloatRect rect1 = *this;
FX_RECT rect;
- MatchFloatRange(rect1.left, rect1.right, &rect.left, &rect.right);
- MatchFloatRange(rect1.bottom, rect1.top, &rect.top, &rect.bottom);
+ MatchFloatRange(left, right, &rect.left, &rect.right);
+ MatchFloatRange(bottom, top, &rect.top, &rect.bottom);
rect.Normalize();
return rect;
}