diff options
author | Cary Clark <caryclark@skia.org> | 2018-07-24 12:40:10 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-07-24 17:08:55 +0000 |
commit | 2587f41f2667b2add97f75583b71ebb74bc48af1 (patch) | |
tree | b1e24e0b61631d780d00746f7c6483609c100452 /src/pathops/SkPathOpsCommon.cpp | |
parent | 6a4e60bb8f49e4a031e8fc99c4d8edaa25b3619f (diff) |
handle failing pathop tests
Some tests isolated from GrShapes_arcs do not
fail gracefully, so make sure errors are properly
handled.
TBR=reed@google.com
Bug: skia:
Change-Id: Ia8c9903e64ef755ec11c398df3e5d258ca1f5f8b
Reviewed-on: https://skia-review.googlesource.com/143112
Reviewed-by: Cary Clark <caryclark@skia.org>
Commit-Queue: Cary Clark <caryclark@skia.org>
Auto-Submit: Cary Clark <caryclark@skia.org>
Diffstat (limited to 'src/pathops/SkPathOpsCommon.cpp')
-rw-r--r-- | src/pathops/SkPathOpsCommon.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/pathops/SkPathOpsCommon.cpp b/src/pathops/SkPathOpsCommon.cpp index 19e80b0b67..c1b134fd75 100644 --- a/src/pathops/SkPathOpsCommon.cpp +++ b/src/pathops/SkPathOpsCommon.cpp @@ -156,7 +156,8 @@ SkOpSegment* FindChase(SkTDArray<SkOpSpanBase*>* chase, SkOpSpanBase** startPtr, } // OPTIMIZATION: should this also add to the chase? if (sortable) { - (void) segment->markAngle(maxWinding, sumWinding, angle); + // TODO: add error handling + SkAssertResult(segment->markAngle(maxWinding, sumWinding, angle, nullptr)); } } } |