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

Fix SA1508 #938

Closed
wants to merge 1 commit into from
Closed

Fix SA1508 #938

wants to merge 1 commit into from

Conversation

Maxwe11
Copy link
Member

@Maxwe11 Maxwe11 commented Jul 1, 2015

SA1508 throws ArgumentOutOfRange

@Maxwe11
Fix SA1508
733d988 
SA1508 throws ArgumentOutOfRange
@pdelvo
Copy link
Member

pdelvo commented Jul 1, 2015

Could you include a test for this?

@vweijsters
Copy link
Contributor

Please add a test case that will reproduce the issue.

@Maxwe11
Copy link
Member Author

Maxwe11 commented Jul 1, 2015

Will try, but I'm not sure what's going wrong. Seems it was in moment of typing multiline comment

public void Bar(string s, int i, int i2) /* */
{
    var ints = new[] { 1, 2, 3 };
}

UPD
but now couldn't repro

@Maxwe11 Maxwe11 closed this Jul 1, 2015
@Maxwe11 Maxwe11 deleted the fix-SA1508 branch August 15, 2015 15:19
@sharwell sharwell mentioned this pull request Sep 23, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Maxwe11 @pdelvo @vweijsters