Update SA1015 for function pointers

Author: sharwell

StyleCop.Analyzers/StyleCop.Analyzers.Test.CSharp9/SpacingRules/SA1015CSharp9UnitTests.cs

@@ -3,9 +3,43 @@
 
 namespace StyleCop.Analyzers.Test.CSharp9.SpacingRules
 {
+    using System.Threading;
+    using System.Threading.Tasks;
     using StyleCop.Analyzers.Test.CSharp8.SpacingRules;
+    using Xunit;
+    using static StyleCop.Analyzers.SpacingRules.SA1015ClosingGenericBracketsMustBeSpacedCorrectly;
+    using static StyleCop.Analyzers.Test.Verifiers.StyleCopCodeFixVerifier<
+        StyleCop.Analyzers.SpacingRules.SA1015ClosingGenericBracketsMustBeSpacedCorrectly,
+        StyleCop.Analyzers.SpacingRules.TokenSpacingCodeFixProvider>;
 
     public partial class SA1015CSharp9UnitTests : SA1015CSharp8UnitTests
     {
+        [Fact]
+        [WorkItem(3970, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/3970")]
+        public async Task TestFunctionPointerClosingBracketSpacingAsync()
+        {
+            var testCode = @"public class TestClass
+{
+    private unsafe delegate*<int, void {|#0:>|}field1;
+    private unsafe delegate*<int, void {|#1:>|} field2;
+}
+";
+
+            var fixedCode = @"public class TestClass
+{
+    private unsafe delegate*<int, void> field1;
+    private unsafe delegate*<int, void> field2;
+}
+";
+
+            var expected = new[]
+            {
+                Diagnostic(DescriptorNotPreceded).WithLocation(0),
+                Diagnostic(DescriptorFollowed).WithLocation(0),
+                Diagnostic(DescriptorNotPreceded).WithLocation(1),
+            };
+
+            await VerifyCSharpFixAsync(testCode, expected, fixedCode, CancellationToken.None).ConfigureAwait(false);
+        }
     }
 }

StyleCop.Analyzers/StyleCop.Analyzers/SpacingRules/SA1015ClosingGenericBracketsMustBeSpacedCorrectly.cs

@@ -87,6 +87,7 @@ private static void HandleGreaterThanToken(SyntaxTreeAnalysisContext context, Sy
             {
             case SyntaxKind.TypeArgumentList:
             case SyntaxKind.TypeParameterList:
+            case SyntaxKindEx.FunctionPointerParameterList:
                 break;
 
             default:

documentation/SA1015.md

@@ -29,6 +29,8 @@ nullable symbol, an end of line or a single whitespace (but not whitespace and a
 contexts, nullable reference type annotations (for example `List<string?>?`) follow the same rules as nullable value
 types.
 
+The `>` token that ends a C# 9 function pointer parameter list follows the same spacing rules, for example `delegate*<int, void>`.
+
 ## How to fix violations
 
 To fix a violation of this rule, ensure the whitespace around the closing generic bracket is correct.