Add coverage related comments

Author: bjornhellander

StyleCop.Analyzers/StyleCop.Analyzers.Test/LayoutRules/SA1501UnitTests.cs

@@ -847,6 +847,9 @@ public void Bar(int i)
         [MemberData(nameof(TestStatements))]
         public async Task TestNoSA1503StatementWithBracesAsync(string statementText)
         {
+            // TODO: The statement in this test code does not have braces! Change test name or the code?
+            // There is uncovered code in SA1501 related to blocks when SA1503 is suppressed, so either
+            // change this to do what the the name says or change the name and add another test.
             await new CSharpTest
             {
                 TestCode = this.GenerateFixedTestStatement(statementText),

StyleCop.Analyzers/StyleCop.Analyzers/MaintainabilityRules/SA1400AccessModifierMustBeDeclared.cs

@@ -91,6 +91,8 @@ private static void HandleEventDeclaration(SyntaxNodeAnalysisContext context)
 
             if (syntax.Parent.IsKind(SyntaxKind.InterfaceDeclaration))
             {
+                // NOTE: Events in interfaces seem to be reported as EventFieldDeclaration, so this should not happen.
+                // Leaving it for consistency and possible future changes though.
                 return;
             }