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.

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

SA1514 incorrectly reported when preceded by preprocessor directive with line comment #1223

Closed
sharwell opened this issue Aug 16, 2015 · 2 comments
Assignees
Milestone

Comments

@sharwell
Copy link
Member

The following code reports SA1514, but it should not.

#pragma warning disable RS1012 // Any comment.

/// <summary>
/// Summary
/// </summary>
static void MyMethod()
{
}
@sharwell sharwell added the bug label Aug 16, 2015
@sharwell sharwell added this to the 1.0.0 Beta 8 milestone Aug 16, 2015
@sharwell sharwell changed the title SA1514 incorrectly reported when preceded by preprocessor directive SA1514 incorrectly reported when preceded by preprocessor directive with line comment Aug 16, 2015
@alaendle
Copy link

Same problem exists for regions - e.g.

    #region Test

    /// <summary>
    /// Initializes a new instance of the <see cref="Form1"/> class.
    /// </summary>
    public Form1()
    {
      this.InitializeComponent();
    }

    #endregion

@sharwell
Copy link
Member Author

@alaendle That one was reported in #1231.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants