Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

SA1119 (Statement should not use unnecessary parenthesis) false positive #3712

Closed
kindermannhubert opened this issue Oct 17, 2023 · 0 comments · Fixed by #3737
Closed

SA1119 (Statement should not use unnecessary parenthesis) false positive #3712

kindermannhubert opened this issue Oct 17, 2023 · 0 comments · Fixed by #3737
Labels
bug fixed
Milestone
1.2.0-beta.553

Comments

@kindermannhubert
Copy link

SA1119 is reported in the following code:

int x = 0, y = 0;
(b ? ref x : ref y) = 1;

But code without parenthesis does not compile.
@sharwell sharwell added the bug label Oct 20, 2023
bjornhellander added a commit to bjornhellander/StyleCopAnalyzers that referenced this issue Nov 26, 2023
@bjornhellander
Update SA1119 to allow parenthesis around a ref ternary conditional e…
b6f0e8e 
…xpression when it is the left-hand side of an assigment

DotNetAnalyzers#3712
@bjornhellander bjornhellander mentioned this issue Nov 26, 2023
Merged
bjornhellander added a commit to bjornhellander/StyleCopAnalyzers that referenced this issue Nov 26, 2023
@bjornhellander
Update SA1119 to allow parenthesis around a ref ternary conditional e…
380924c 
…xpression when it is the left-hand side of an assigment

DotNetAnalyzers#3712
@sharwell sharwell added this to the 1.2-beta.next milestone Nov 28, 2023
@sharwell sharwell added the fixed label Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug fixed
Projects
None yet
Milestone
1.2.0-beta.553
2 participants
@sharwell @kindermannhubert