⚙ D20857 [clang-tidy] Add modernize-explicit-operator-bool check. (original ) (raw ) This is an archive of the discontinued LLVM Phabricator instance.
Table of Contentst -clang-tidy/modernize/-modernize/-CMakeLists.txt-ExplicitOperatorBoolCheck.h6ExplicitOperatorBoolCheck.cpp-ModernizeTidyModule.cpp-docs/-ReleaseNotes.rst-clang-tidy/checks/-checks/1list.rst3modernize-explicit-operator-bool.rst-test/clang-tidy/-clang-tidy/1/3modernize-explicit-operator-bool-void-pointer.cpp-modernize-explicit-operator-bool.cppHide Panelf Keyboard Reference?
Differential D20857 Authored by • murrayc on Jun 1 2016, 4:11 AM.Download Raw Diff Reviewers aaron.ballman hokein alexfh Event Timeline• murrayc updated this revision to Diff 59196 .• murrayc retitled this revision from to [clang-tidy] Add modernize-explicit-operator-bool check..• murrayc updated this object.Comment Actions docs/clang-tidy/checks/modernize-explicit-operator-bool.rst 7 Eugene.Zelenko added reviewers: alexfh , etienne.bergeron , hokein , aaron.ballman .Eugene.Zelenko set the repository for this revision to rL LLVM .Eugene.Zelenko added a subscriber: cfe-commits .etienneb edited reviewers, added: etienneb ; removed: etienne.bergeron .Comment Actions clang-tidy/modernize/ExplicitOperatorBoolCheck.cpp 22 37 clang-tidy/modernize/OperatorVoidPointerCheck.cpp 30 ↗ (On Diff #59196) 37 ↗ (On Diff #59196) 37 ↗ (On Diff #59196) docs/clang-tidy/checks/list.rst 6–7 docs/clang-tidy/checks/modernize-explicit-operator-bool.rst 12 Comment Actions Comment Actions Comment Actions clang-tidy/modernize/OperatorVoidPointerCheck.cpp 20 ↗ (On Diff #59346) 21 ↗ (On Diff #59346) 31 ↗ (On Diff #59346) 35 ↗ (On Diff #59346)Comment Actions Comment Actions Comment Actions clang-tidy/modernize/OperatorVoidPointerCheck.cpp 27 ↗ (On Diff #59367) 40 ↗ (On Diff #59367)• murrayc added inline comments.clang-tidy/modernize/OperatorVoidPointerCheck.cpp 27 ↗ (On Diff #59367) 40 ↗ (On Diff #59367)Comment Actions etienneb added inline comments.test/clang-tidy/modernize-operator-void-pointer.cpp 39 ↗ (On Diff #59367)Comment Actions docs/clang-tidy/checks/modernize-explicit-operator-bool.rst 13 Comment Actions Comment Actions test/clang-tidy/modernize-explicit-operator-bool-void-pointer.cpp 7 15 Comment Actions test/clang-tidy/modernize-explicit-operator-bool-void-pointer.cpp 7 Comment Actions Comment Actions Comment Actions clang-tidy/modernize/ExplicitOperatorBoolCheck.cpp 38 55 64 This revision now requires changes to proceed.Prazek edited edge metadata.Prazek added a project: Restricted Project.• murrayc added inline comments.clang-tidy/modernize/ExplicitOperatorBoolCheck.cpp 38 etienneb removed a reviewer: etienneb .JDevlieghere added a subscriber: JDevlieghere .Files History Commits PathSizeclang-tidy/modernize/CMakeLists.txt 1 lineExplicitOperatorBoolCheck.h 41 linesExplicitOperatorBoolCheck.cpp 83 linesModernizeTidyModule.cpp 3 linesdocs/ReleaseNotes.rst 7 linesclang-tidy/checks/list.rst 1 linemodernize-explicit-operator-bool.rst 28 linestest/clang-tidy/modernize-explicit-operator-bool-void-pointer.cpp 46 linesmodernize-explicit-operator-bool.cpp 19 linesDiffIDBaseDescriptionCreatedLintUnitBase BaseDiff 159196 271321Jun 1 2016, 4:11 AM★★Diff 259346 271321These are the commits amended with the suggested changes.Jun 2 2016, 1:05 AM★★Diff 359367 271321With suggested changes. Ran clang-format (LLVM style). Used voidType() matcher.Jun 2 2016, 5:20 AM★★Diff 459537 271321Combined into one check. Also specifies C++11 for the test.Jun 3 2016, 5:41 AM★★Diff 562355 274019Same as previous patch, but with a tiny suggested whitespace corretion.Jun 30 2016, 5:53 AM★★CommitTreeParentsAuthorSummaryDatee8d465438579b357c0b1339e8d30be1b1590Murray Cumming[clang-tidy] Add modernize-explicit-operator-bool check. (Show More…) May 16 2016, 10:39 AMThis file was added.This file was added.This file was added.This file was added.This file was added.