[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