| commit | 393f8994d2aa1a9c66fd69eaf1c9418ae2bb92ee | [log] [tgz] |
|---|---|---|
| author | Dan Sinclair <dsinclair@chromium.org> | Thu Nov 30 15:46:51 2017 +0000 |
| committer | Chromium commit bot <commit-bot@chromium.org> | Thu Nov 30 15:46:51 2017 +0000 |
| tree | 2b9f35c63f3f3761a6cacee099c857d6745deb18 | |
| parent | dddfdad6ebb4e7172ff36f5a0db8a5cbf58d45f3 [diff] |
Run clang-tidy modernize-use-bool-literals on //third_party/pdfium See the bugs and cxx post for justification and details: https://groups.google.com/a/chromium.org/forum/#!topic/cxx/RkOHzIK6Tq8 This change was done using clang-tidy as described here: https://chromium.googlesource.com/chromium/src/+/lkcr/docs/clang_tidy.md Bug: chromium:778955 Change-Id: Ib037ed749539e4380fb336fc6fa85648fc506118 Reviewed-on: https://pdfium-review.googlesource.com/19990 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: dsinclair <dsinclair@chromium.org>
diff --git a/third_party/base/numerics/safe_conversions_impl.h b/third_party/base/numerics/safe_conversions_impl.h index 2735c1e..4077b28 100644 --- a/third_party/base/numerics/safe_conversions_impl.h +++ b/third_party/base/numerics/safe_conversions_impl.h
@@ -143,7 +143,7 @@ public: constexpr RangeCheck(bool is_in_lower_bound, bool is_in_upper_bound) : is_underflow_(!is_in_lower_bound), is_overflow_(!is_in_upper_bound) {} - constexpr RangeCheck() : is_underflow_(0), is_overflow_(0) {} + constexpr RangeCheck() : is_underflow_(false), is_overflow_(false) {} constexpr bool IsValid() const { return !is_overflow_ && !is_underflow_; } constexpr bool IsInvalid() const { return is_overflow_ && is_underflow_; } constexpr bool IsOverflow() const { return is_overflow_ && !is_underflow_; }