⚙ 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 PanelfKeyboard Reference? Differential D20857 Authored by • murrayc on Jun 1 2016, 4:11 AM.Download Raw DiffReviewers aaron.ballmanhokeinalexfh 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 Actionsdocs/clang-tidy/checks/modernize-explicit-operator-bool.rst 7Eugene.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 Actionsclang-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 12Comment ActionsComment ActionsComment Actionsclang-tidy/modernize/OperatorVoidPointerCheck.cpp 20 ↗(On Diff #59346) 21 ↗(On Diff #59346) 31 ↗(On Diff #59346) 35 ↗(On Diff #59346)Comment ActionsComment ActionsComment Actionsclang-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 Actionsetienneb added inline comments.test/clang-tidy/modernize-operator-void-pointer.cpp 39 ↗(On Diff #59367)Comment Actionsdocs/clang-tidy/checks/modernize-explicit-operator-bool.rst 13Comment ActionsComment Actionstest/clang-tidy/modernize-explicit-operator-bool-void-pointer.cpp 7 15Comment Actionstest/clang-tidy/modernize-explicit-operator-bool-void-pointer.cpp 7Comment ActionsComment ActionsComment Actionsclang-tidy/modernize/ExplicitOperatorBoolCheck.cpp 38 55 64This 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 38etienneb removed a reviewer: etienneb.JDevlieghere added a subscriber: JDevlieghere.FilesHistoryCommitsPathSizeclang-tidy/modernize/CMakeLists.txt1 lineExplicitOperatorBoolCheck.h41 linesExplicitOperatorBoolCheck.cpp83 linesModernizeTidyModule.cpp3 linesdocs/ReleaseNotes.rst7 linesclang-tidy/checks/list.rst1 linemodernize-explicit-operator-bool.rst28 linestest/clang-tidy/modernize-explicit-operator-bool-void-pointer.cpp46 linesmodernize-explicit-operator-bool.cpp19 linesDiffIDBaseDescriptionCreatedLintUnitBaseBaseDiff 159196271321Jun 1 2016, 4:11 AM★★Diff 259346271321These are the commits amended with the suggested changes.Jun 2 2016, 1:05 AM★★Diff 359367271321With suggested changes. Ran clang-format (LLVM style). Used voidType() matcher.Jun 2 2016, 5:20 AM★★Diff 459537271321Combined into one check. Also specifies C++11 for the test.Jun 3 2016, 5:41 AM★★Diff 562355274019Same 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.