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

SA1008 false positive on range expression with first operand in parentheses #3894

Open
rotanov opened this issue Oct 16, 2024 · 0 comments · May be fixed by #3895
Open

SA1008 false positive on range expression with first operand in parentheses #3894

rotanov opened this issue Oct 16, 2024 · 0 comments · May be fixed by #3895

Comments

@rotanov
Copy link

rotanov commented Oct 16, 2024 •
edited
Loading 

// This is fine
var rangeIsFine = 0..(1 + 0);
//                       +-- Warning SA1008: Opening parenthesis should not be preceded by a space.
//                       |
//                       v
var rangeFalsePositive = (0 + 0)..1;

version 1.2.0-beta.556
bjornhellander added a commit to bjornhellander/StyleCopAnalyzers that referenced this issue Oct 17, 2024
@bjornhellander
Update SA1008 to require a space before the opening parenthesis if it…
d34096c 
…'s the left operand of a range expression. Also add test for SA1008 in collection expression.

DotNetAnalyzers#3894
@bjornhellander bjornhellander linked a pull request Oct 17, 2024 that will close this issue
Open
bjornhellander added a commit to bjornhellander/StyleCopAnalyzers that referenced this issue Oct 17, 2024
@bjornhellander
Update SA1008 to require a space before the opening parenthesis if it…
2d7e105 
…'s the left operand of a range expression. Also add test for SA1008 in collection expression.

DotNetAnalyzers#3894
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@rotanov