diff options
author | caryclark <caryclark@google.com> | 2016-08-24 09:24:18 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-08-24 09:24:18 -0700 |
commit | ef4f32ac858825dc443cfe4739ea878fb0bf550f (patch) | |
tree | 357340c761e3e2c0d82bc53489f0ea7b10511b95 /tests/PathOpsOpTest.cpp | |
parent | ea17dfe40b04fa476008af67984b7a41dc5243f7 (diff) |
remove point aliases
This removes the notion of keeping track of every different t value
that resolves to the same or a similar point. Other fixes make
this concept unnecessary, and removing it simplifies the code.
This removes an allocation, and speeds up paths with many
overlapping curves.
As a bonus, four fuzzer tests that failed before now succeed.
TBR=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2275703003
Review-Url: https://codereview.chromium.org/2275703003
Diffstat (limited to 'tests/PathOpsOpTest.cpp')
-rw-r--r-- | tests/PathOpsOpTest.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/PathOpsOpTest.cpp b/tests/PathOpsOpTest.cpp index 759b3bcae3..d659e8863e 100644 --- a/tests/PathOpsOpTest.cpp +++ b/tests/PathOpsOpTest.cpp @@ -6822,7 +6822,7 @@ path.conicTo(SkBits2Float(0x084b0321), SkBits2Float(0x6ac07b2a), SkBits2Float(0x path.lineTo(SkBits2Float(0x8c558c55), SkBits2Float(0x212a1f2a)); // -1.64512e-31f, 5.76395e-19f SkPath path2(path); - testPathOpFailSkipAssert(reporter, path1, path2, (SkPathOp) 0, filename); + testPathOpSkipAssert(reporter, path1, path2, (SkPathOp) 0, filename); } static void fuzz763_17(skiatest::Reporter* reporter, const char* filename) { @@ -6976,7 +6976,7 @@ path.lineTo(SkBits2Float(0x68395b2d), SkBits2Float(0xf0682955)); // 3.50128e+24 path.close(); SkPath path2(path); - testPathOpFailSkipAssert(reporter, path1, path2, (SkPathOp) 0, filename); + testPathOpSkipAssert(reporter, path1, path2, (SkPathOp) 0, filename); } static void fuzz763_20(skiatest::Reporter* reporter, const char* filename) { @@ -7276,7 +7276,7 @@ path.lineTo(SkBits2Float(0x68385b2d), SkBits2Float(0xf0682955)); // 3.48239e+24 path.close(); SkPath path2(path); - testPathOpFailSkipAssert(reporter, path1, path2, (SkPathOp) 0, filename); + testPathOpSkipAssert(reporter, path1, path2, (SkPathOp) 0, filename); } static void fuzz763_28(skiatest::Reporter* reporter, const char* filename) { @@ -7314,7 +7314,7 @@ path.moveTo(SkBits2Float(0x0321081f), SkBits2Float(0x6a4b7bc0)); // 4.7323e-37f path.conicTo(SkBits2Float(0x212a8ced), SkBits2Float(0x0321081f), SkBits2Float(0x6a3a7bc0), SkBits2Float(0x2147ed7a), SkBits2Float(0x28282a3a)); // 5.77848e-19f, 4.7323e-37f, 5.63611e+25f, 6.77381e-19f, 9.33503e-15f SkPath path2(path); - testPathOpFailSkipAssert(reporter, path1, path2, (SkPathOp) 0, filename); + testPathOpSkipAssert(reporter, path1, path2, (SkPathOp) 0, filename); } static void fuzz763_27(skiatest::Reporter* reporter, const char* filename) { |