[PATCH] D59859: [clang-tidy] FIXIT for implicit bool conversion now obeys upper case suffixes if enforced.
Patrick Nappa via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 25 22:12:42 PDT 2019
pnappa updated this revision to Diff 196795.
pnappa added a comment.
Apologies for taking a while, I'm afraid I hadn't had a moment of spare time over the past few weeks.
I believe I've applied the requested changes, bar one that I'm unsure about:
@alexfh to be sure, would adding an Option to "readability-implicit-bool-conversion" called say, "EnforceUppercase", suffice? With that option enabled the synthesis of the explicit comparison would be uppercase.
I'll make the appropriate changes to documentation when I get everything finalised.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59859/new/
https://reviews.llvm.org/D59859
Files:
clang-tools-extra/clang-tidy/readability/ImplicitBoolConversionCheck.cpp
clang-tools-extra/clang-tidy/readability/ImplicitBoolConversionCheck.h
clang-tools-extra/test/clang-tidy/readability-implicit-bool-conversion.cpp
clang-tools-extra/test/clang-tidy/readability-uppercase-literal-suffix-implicit-bool-conversion.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59859.196795.patch
Type: text/x-patch
Size: 11304 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190426/ffcb02cf/attachment-0001.bin>
More information about the cfe-commits
mailing list