diff options
author | Ben Wagner <bungeman@google.com> | 2018-06-12 16:30:29 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-06-12 21:30:24 +0000 |
commit | cee46e5e99d52a162ec19d7567aeb39083b75ec4 (patch) | |
tree | 6bd1efddd26dbf1a3192007dd2f5c8c39ae1d65b /include/private/SkPathRef.h | |
parent | 75e6490a2952849d72ecb1e59c97e36db8908884 (diff) |
Remove SkBool8.
This typedef was created at a time when compilers often used sizeof(int)
storage for a bool. This is no longer the case and in all compilers
currently supported 'sizeof(bool) == 1'. Removing this also revealed one
field which was actually not a bool but a tri-state enum.
Change-Id: I9240ba457335ee3eff094d6d3f2520c1adf16960
Reviewed-on: https://skia-review.googlesource.com/134420
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Ben Wagner <bungeman@google.com>
Diffstat (limited to 'include/private/SkPathRef.h')
-rw-r--r-- | include/private/SkPathRef.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/include/private/SkPathRef.h b/include/private/SkPathRef.h index 94d5bf13b2..5a5a04118c 100644 --- a/include/private/SkPathRef.h +++ b/include/private/SkPathRef.h @@ -543,13 +543,13 @@ private: SkTDArray<GenIDChangeListener*> fGenIDChangeListeners; // pointers are reffed mutable uint8_t fBoundsIsDirty; - mutable SkBool8 fIsFinite; // only meaningful if bounds are valid + mutable bool fIsFinite; // only meaningful if bounds are valid - SkBool8 fIsOval; - SkBool8 fIsRRect; + bool fIsOval; + bool fIsRRect; // Both the circle and rrect special cases have a notion of direction and starting point // The next two variables store that information for either. - SkBool8 fRRectOrOvalIsCCW; + bool fRRectOrOvalIsCCW; uint8_t fRRectOrOvalStartIdx; uint8_t fSegmentMask; |