[PATCH] D43277: limits: Use `false` instead of `type(0)`.

2018-02-16 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added a comment. In https://reviews.llvm.org/D43277#1009659, @brucem wrote: > Without this, anyone using `clang-tidy` with this check gets 6 system header > warnings whenever they transitively have included `limits`. Only if `-system-headers` is passed to clang-tidy, or if those

[PATCH] D43277: limits: Use `false` instead of `type(0)`.

2018-02-15 Thread Bruce Mitchener via Phabricator via cfe-commits
brucem added a comment. Herald added a subscriber: christof. Without this, anyone using `clang-tidy` with this check gets 6 system header warnings whenever they transitively have included `limits`. Repository: rCXX libc++ https://reviews.llvm.org/D43277

[PATCH] D43277: limits: Use `false` instead of `type(0)`.

2018-02-14 Thread Marshall Clow via Phabricator via cfe-commits
mclow.lists added a comment. I see no benefit to this change. `bool(0)` is `false`. Repository: rCXX libc++ https://reviews.llvm.org/D43277 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D43277: limits: Use `false` instead of `type(0)`.

2018-02-13 Thread Bruce Mitchener via Phabricator via cfe-commits
brucem created this revision. brucem added reviewers: mclow.lists, EricWF. This fixes warnings when using clang-tidy and the `modernize-use-bool-literals` check. Repository: rCXX libc++ https://reviews.llvm.org/D43277 Files: include/limits Index: include/limits