[PATCH] D115249: Clang-Tidy implicit bool conversion check use upercase suffixes

2022-01-05 Thread Nathan James via Phabricator via cfe-commits
njames93 added a comment. Something stylistic like this should likely be configurable. Lowercase literals are a part of the language and there is likely a codebase out there that explicitly prefers them Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org

[PATCH] D115249: Clang-Tidy implicit bool conversion check use upercase suffixes

2021-12-08 Thread Kilian Traub via Phabricator via cfe-commits
kili updated this revision to Diff 392678. kili added a comment. Fix tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D115249/new/ https://reviews.llvm.org/D115249 Files: clang-tools-extra/clang-tidy/readability/ImplicitBoolConversionCheck.cpp

[PATCH] D115249: Clang-Tidy implicit bool conversion check use upercase suffixes

2021-12-07 Thread Kilian Traub via Phabricator via cfe-commits
kili created this revision. Herald added a subscriber: carlosgalvezp. kili requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. Make the type suffix of the implicit bool conversion check upercase. So that the result does not fail th