[clang-tools-extra] [clang-tidy] Fix missing parentheses in readability-implicit-bool-conversion fixes (PR #74891)
FĂ©lix-Antoine Constantin via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 15 10:01:38 PST 2024
felix642 wrote:
LGTM !
You'll need to update your branch with master but the PR looks good.
The only thing that seems odd is the changes that you've made to the method `areParensNeededForOverloadedOperator` where the Operators `OO_New`, `OO_Delete`, `OO_Array_New` and `OO_ArrayDelete` now need parenthesis where they did not before.
I'm assuming that you have a used case in mind where parenthesis would be needed and this is why you've changed the behaviour.
https://github.com/llvm/llvm-project/pull/74891
More information about the cfe-commits
mailing list