https://github.com/PiotrZSL closed
https://github.com/llvm/llvm-project/pull/74891
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/felix642 approved this pull request.
https://github.com/llvm/llvm-project/pull/74891
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL updated
https://github.com/llvm/llvm-project/pull/74891
>From 8c3b797f1f34d18c1e9211898f7d1a5697251317 Mon Sep 17 00:00:00 2001
From: Piotr Zegar
Date: Fri, 8 Dec 2023 22:01:23 +
Subject: [PATCH] [clang-tidy] Fix missing parentheses in
https://github.com/felix642 approved this pull request.
https://github.com/llvm/llvm-project/pull/74891
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
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
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Piotr Zegar (PiotrZSL)
Changes
Check now more properly add missing parentheses to code like this: 'bool bar =
true ? 1 : 0 != 0;'.
Closes #71867
---
Patch is 31.07 KiB, truncated to 20.00 KiB below, full version:
https://github.com/PiotrZSL created
https://github.com/llvm/llvm-project/pull/74891
Check now more properly add missing parentheses to code like this: 'bool bar =
true ? 1 : 0 != 0;'.
Closes #71867
>From 8c3b797f1f34d18c1e9211898f7d1a5697251317 Mon Sep 17 00:00:00 2001
From: Piotr Zegar