aboutsummaryrefslogtreecommitdiffhomepage
path: root/tests/InfRectTest.cpp
diff options
context:
space:
mode:
authorGravatar reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>2011-02-07 19:08:59 +0000
committerGravatar reed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>2011-02-07 19:08:59 +0000
commit534240f6058773cc2366b8310ed5452286a6dfff (patch)
treef1b39b3fd8da9086669f79ba25710a1c030e4771 /tests/InfRectTest.cpp
parentff1ec2f1a8d417236f36d1396182538967e68e1a (diff)
hide divide-by-zero from the compiler, to avoid the warning
git-svn-id: http://skia.googlecode.com/svn/trunk@766 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'tests/InfRectTest.cpp')
-rw-r--r--tests/InfRectTest.cpp21
1 files changed, 13 insertions, 8 deletions
diff --git a/tests/InfRectTest.cpp b/tests/InfRectTest.cpp
index fc9c6eba8a..bcdbf4a53c 100644
--- a/tests/InfRectTest.cpp
+++ b/tests/InfRectTest.cpp
@@ -1,35 +1,40 @@
#include "Test.h"
#include "SkRect.h"
+static float make_zero() {
+ return sk_float_sin(0);
+}
+
// Tests that hasValidCoordinates() will reject any rect with +/-inf values
// as one of its coordinates.
static void TestInfRect(skiatest::Reporter* reporter) {
+ float zero = make_zero();
SkRect rect = SkRect::MakeXYWH(10.0f, 10.0f, 100.0f, 100.0f);
REPORTER_ASSERT(reporter, rect.hasValidCoordinates());
- rect = SkRect::MakeXYWH(10.0f, 10.0f, 100.0f, 1.0f/0.0f); // Make 'inf' value without numeric_limits.
+ rect = SkRect::MakeXYWH(10.0f, 10.0f, 100.0f, 1.0f/zero); // Make 'inf' value without numeric_limits.
REPORTER_ASSERT(reporter, !rect.hasValidCoordinates());
- rect = SkRect::MakeXYWH(10.0f, 10.0f, 1.0f/0.0f, 100.0f);
+ rect = SkRect::MakeXYWH(10.0f, 10.0f, 1.0f/zero, 100.0f);
REPORTER_ASSERT(reporter, !rect.hasValidCoordinates());
- rect = SkRect::MakeXYWH(1.0f/0.0f, 10.0f, 100.0f, 100.0f);
+ rect = SkRect::MakeXYWH(1.0f/zero, 10.0f, 100.0f, 100.0f);
REPORTER_ASSERT(reporter, !rect.hasValidCoordinates());
- rect = SkRect::MakeXYWH(10.0f, 1.0f/0.0f, 100.0f, 100.0f);
+ rect = SkRect::MakeXYWH(10.0f, 1.0f/zero, 100.0f, 100.0f);
REPORTER_ASSERT(reporter, !rect.hasValidCoordinates());
- rect = SkRect::MakeXYWH(10.0f, 10.0f, 100.0f, -1.0f/0.0f);
+ rect = SkRect::MakeXYWH(10.0f, 10.0f, 100.0f, -1.0f/zero);
REPORTER_ASSERT(reporter, !rect.hasValidCoordinates());
- rect = SkRect::MakeXYWH(10.0f, 10.0f, -1.0f/0.0f, 100.0f);
+ rect = SkRect::MakeXYWH(10.0f, 10.0f, -1.0f/zero, 100.0f);
REPORTER_ASSERT(reporter, !rect.hasValidCoordinates());
- rect = SkRect::MakeXYWH(-1.0f/0.0f, 10.0f, 100.0f, 100.0f);
+ rect = SkRect::MakeXYWH(-1.0f/zero, 10.0f, 100.0f, 100.0f);
REPORTER_ASSERT(reporter, !rect.hasValidCoordinates());
- rect = SkRect::MakeXYWH(10.0f, -1.0f/0.0f, 100.0f, 100.0f);
+ rect = SkRect::MakeXYWH(10.0f, -1.0f/zero, 100.0f, 100.0f);
REPORTER_ASSERT(reporter, !rect.hasValidCoordinates());
}