diff options
author | Mike Klein <mtklein@chromium.org> | 2018-01-25 17:32:51 -0500 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-01-26 15:37:48 +0000 |
commit | 1d746208a1335a192433923b4a818ded3978228d (patch) | |
tree | 08c2925624a1ba9c002faaea63c698e68f91becc /src/core/SkPathRef.cpp | |
parent | 5a907a7cce9a5ca72357f06e7874136d12b867b8 (diff) |
bug fixes from PVS Studio (static analysis)
- Hal, please check out SkPDFCanon... was this unused?
- Cary, please SkOpContour... is it right that allDone isn't necessary?
- Brian, the rest?
Bug: chromium:805881
Change-Id: I7cbbcf44f4460a114f4ed2a59ed3856203049cdc
Reviewed-on: https://skia-review.googlesource.com/99960
Reviewed-by: Brian Salomon <bsalomon@google.com>
Reviewed-by: Hal Canary <halcanary@google.com>
Reviewed-by: Cary Clark <caryclark@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src/core/SkPathRef.cpp')
-rw-r--r-- | src/core/SkPathRef.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/core/SkPathRef.cpp b/src/core/SkPathRef.cpp index 055b63d142..562838b530 100644 --- a/src/core/SkPathRef.cpp +++ b/src/core/SkPathRef.cpp @@ -771,9 +771,6 @@ bool SkPathRef::isValid() const { if (nullptr == fPoints && 0 != fFreeSpace) { return false; } - if (nullptr == fPoints && 0 != fFreeSpace) { - return false; - } if (nullptr == fPoints && fPointCnt) { return false; } |