From ef4f32ac858825dc443cfe4739ea878fb0bf550f Mon Sep 17 00:00:00 2001 From: caryclark Date: Wed, 24 Aug 2016 09:24:18 -0700 Subject: 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 --- tests/PathOpsAngleTest.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'tests/PathOpsAngleTest.cpp') diff --git a/tests/PathOpsAngleTest.cpp b/tests/PathOpsAngleTest.cpp index fa8d31469b..12d1d5579a 100644 --- a/tests/PathOpsAngleTest.cpp +++ b/tests/PathOpsAngleTest.cpp @@ -478,9 +478,9 @@ DEF_TEST(PathOpsAngleAfter, reporter) { void SkOpSegment::debugAddAngle(double startT, double endT) { SkOpPtT* startPtT = startT == 0 ? fHead.ptT() : startT == 1 ? fTail.ptT() - : this->addT(startT, kNoAliasMatch, nullptr); + : this->addT(startT, nullptr); SkOpPtT* endPtT = endT == 0 ? fHead.ptT() : endT == 1 ? fTail.ptT() - : this->addT(endT, kNoAliasMatch, nullptr); + : this->addT(endT, nullptr); SkOpAngle* angle = SkOpTAllocator::Allocate(this->globalState()->allocator()); SkOpSpanBase* startSpan = &fHead; while (startSpan->ptT() != startPtT) { -- cgit v1.2.3