diff options
author | caryclark <caryclark@google.com> | 2016-09-23 05:47:20 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-09-23 05:47:20 -0700 |
commit | 27c015dfcf4e2b8fb1abe327cc40204e2a4f452a (patch) | |
tree | 1fab83dc8286a4f1900871c69e80e624203e7e3e /src/pathops/SkOpCoincidence.cpp | |
parent | 5a9c2f110e4f1a78d9bfedcf708168909706d7fd (diff) |
split tight quads and conics
Tight quads and conics may nearly fold over on themselves, confusing
coincidence against other curves. Split them at their max curvature
early on to avoid complicating later logic.
TBR=reed@google.com
BUG=skia:5131
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2357353002
Review-Url: https://codereview.chromium.org/2357353002
Diffstat (limited to 'src/pathops/SkOpCoincidence.cpp')
-rwxr-xr-x | src/pathops/SkOpCoincidence.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/pathops/SkOpCoincidence.cpp b/src/pathops/SkOpCoincidence.cpp index 1bbb82a9e0..d2874c3c5e 100755 --- a/src/pathops/SkOpCoincidence.cpp +++ b/src/pathops/SkOpCoincidence.cpp @@ -334,6 +334,9 @@ bool SkOpCoincidence::addEndMovedSpans(const SkOpSpan* base, const SkOpSpanBase* } SkOpSegment* writableSeg = const_cast<SkOpSegment*>(testSeg); SkOpPtT* oppStart = writableSeg->addT(t); + if (oppStart == testPtT) { + continue; + } SkOpSpan* writableBase = const_cast<SkOpSpan*>(base); oppStart->span()->addOpp(writableBase); if (oppStart->deleted()) { |