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

SA1312 treats short-hand declared events as variables #1340

Closed
ghost opened this issue Aug 31, 2015 · 1 comment
Closed

SA1312 treats short-hand declared events as variables #1340

ghost opened this issue Aug 31, 2015 · 1 comment
Assignees
@sharwell
Labels
bug fixed
Milestone
1.0.0 Beta 10

Comments

@ghost
Copy link

ghost commented Aug 31, 2015

The following declaration raises SA1312:

public event EventHandler Test;

A full event declaration does not.

public event EventHandler Test
{
    add { }
    remove { }
}
@sharwell
Copy link
Member

😦 Thanks, sorry I missed this.

@sharwell sharwell added this to the 1.0.0 Beta 10 milestone Aug 31, 2015
@sharwell sharwell self-assigned this Aug 31, 2015
sharwell added a commit to sharwell/StyleCopAnalyzers that referenced this issue Aug 31, 2015
@sharwell
Fix SA1312 getting reported for event field declarations
c71608e 
Also added regression tests to related rules SA1306 and SA1313.

Fixes DotNetAnalyzers#1340
@sharwell sharwell mentioned this issue Aug 31, 2015
Merged
@sharwell sharwell mentioned this issue Sep 2, 2015
Closed
@sharwell sharwell removed the blocker label Sep 4, 2015
This was referenced Sep 8, 2015
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sharwell sharwell

Labels
bug fixed
Projects
None yet
Milestone
1.0.0 Beta 10
Development

No branches or pull requests
1 participant
@sharwell