Merge pull request #2579 from vweijsters/fix-2564

Fixed false positive in SA1011 for array index followed by pointer de…

Author: vweijsters

StyleCop.Analyzers/StyleCop.Analyzers.Test/SpacingRules/SA1011UnitTests.cs

@@ -312,6 +312,26 @@ void Method()
             await this.VerifyCSharpDiagnosticAsync(testCode, expected, CancellationToken.None).ConfigureAwait(false);
         }
 
+        [Fact]
+        [WorkItem(2564, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/2564")]
+        public async Task TestArrayElementPointerDeferenceAsync()
+        {
+            var testCode = @"public class TestClass
+{
+    public unsafe string TestMethod(int[] a)
+    {
+        fixed (int* p = a)
+        {
+            int*[] xx = new[] { p };
+            return xx[0]->ToString(""n2"");
+        }
+    }
+}
+";
+
+            await this.VerifyCSharpDiagnosticAsync(testCode, EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
+        }
+
         /// <inheritdoc/>
         protected override IEnumerable<DiagnosticAnalyzer> GetCSharpDiagnosticAnalyzers()
         {

StyleCop.Analyzers/StyleCop.Analyzers/SpacingRules/SA1011ClosingSquareBracketsMustBeSpacedCorrectly.cs

@@ -103,6 +103,7 @@ private static void HandleCloseBracketToken(SyntaxTreeAnalysisContext context, S
                 case SyntaxKind.DotToken:
                 case SyntaxKind.OpenBracketToken:
                 case SyntaxKind.CloseParenToken:
+                case SyntaxKind.MinusGreaterThanToken:
                     precedesSpecialCharacter = true;
                     break;
                 case SyntaxKind.PlusPlusToken: