diff options
author | Ben Wagner <bungeman@google.com> | 2017-10-09 10:47:47 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-10-09 22:01:34 +0000 |
commit | 3f9855293350bd1949855a6002943d3cef97a32f (patch) | |
tree | d63b652f300cc9257f75b4b7ef0c19a688df04b4 /tests/PathOpsDebug.cpp | |
parent | 63fd760a37905c45d26fc3d49cac261fad1b4808 (diff) |
Small pathops cleanup.
Newer clang compiler produces the diagnostic
../../tests/PathOpsDebug.cpp:1349:44:
error: comparison of unsigned enum expression >= 0 is always true
[-Werror,-Wtautological-unsigned-enum-zero-compare]
if (!firstOp && c->operand() && op >= 0) {
The check seems to be defending against bad values of SkPathOp, but
this now seems unnecessary. This also adds 'rdiff' to kPathOpStr for
future debugging.
Change-Id: I08e5ba77c56e519ce4d9ae89491f7ccd5eb7f944
Reviewed-on: https://skia-review.googlesource.com/57104
Reviewed-by: Cary Clark <caryclark@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
Diffstat (limited to 'tests/PathOpsDebug.cpp')
-rw-r--r-- | tests/PathOpsDebug.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/PathOpsDebug.cpp b/tests/PathOpsDebug.cpp index 49f730a30f..8324f37dd4 100644 --- a/tests/PathOpsDebug.cpp +++ b/tests/PathOpsDebug.cpp @@ -1346,7 +1346,7 @@ void SkOpContour::dumpSegments(const char* prefix, SkPathOp op) const { bool firstOp = false; const SkOpContour* c = this; do { - if (!firstOp && c->operand() && op >= 0) { + if (!firstOp && c->operand()) { #if DEBUG_ACTIVE_OP SkDebugf("op %s\n", SkPathOpsDebug::kPathOpStr[op]); #endif |