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/PathOpsAngleTest.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/PathOpsAngleTest.cpp')
-rw-r--r-- | tests/PathOpsAngleTest.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
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<SkOpAngle>::Allocate(this->globalState()->allocator()); SkOpSpanBase* startSpan = &fHead; while (startSpan->ptT() != startPtT) { |