path ops stream-lining
authorcaryclark <caryclark@google.com>
Thu, 25 Aug 2016 18:27:17 +0000 (11:27 -0700)
committerCommit bot <commit-bot@chromium.org>
Thu, 25 Aug 2016 18:27:17 +0000 (11:27 -0700)
commit29b2563afb1677515739f1d24fb27733626eca92
tree8c447f8969688bf3e1187d1b0f1461a3e777c3cd
parentf412f0988424cb5fad48a6097c8b29b5b9c56275
path ops stream-lining

The addT() function is a workhorse of pathops.
Make it simpler, removing branches and parameters.

Separate addOpp() into const and modify parts.

Add more debugging that asserts if the function
fails and the data is not extreme (e.g., fuzzer
generated).

TBR=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2273293004

Review-Url: https://codereview.chromium.org/2273293004
src/pathops/SkAddIntersections.cpp
src/pathops/SkOpCoincidence.cpp
src/pathops/SkOpSegment.cpp
src/pathops/SkOpSegment.h
src/pathops/SkOpSpan.cpp
src/pathops/SkOpSpan.h
src/pathops/SkPathOpsDebug.cpp
src/pathops/SkPathOpsDebug.h
src/pathops/SkPathOpsTypes.h
tests/PathOpsAngleTest.cpp