Make sure indexer declarations print consistently with other paramete…

…r lists (#1278) closes #1255
belav · Jun 3, 2024 · 7653a6d · 7653a6d
1 parent c6e39f6
commit 7653a6d
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 7 deletions.
diff --git a/Src/CSharpier.Tests/FormattingTests/TestFiles/cs/IndexerDeclarations.test b/Src/CSharpier.Tests/FormattingTests/TestFiles/cs/IndexerDeclarations.test
@@ -1,4 +1,9 @@
 abstract class ClassName
 {
     abstract int this[int index] { get; set; }
+
+    public string this[
+        [SomeAttribute] int a________________________________,
+        [SomeAttribute] int b________________________________
+    ] => someValue;
 }
diff --git a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/BracketedParameterList.cs b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/BracketedParameterList.cs
@@ -4,10 +4,6 @@ internal static class BracketedParameterList
 {
     public static Doc Print(BracketedParameterListSyntax node, FormattingContext context)
     {
-        return Doc.Concat(
-            Token.Print(node.OpenBracketToken, context),
-            SeparatedSyntaxList.Print(node.Parameters, Parameter.Print, " ", context),
-            Token.Print(node.CloseBracketToken, context)
-        );
+        return ParameterList.Print(node, node.OpenBracketToken, node.CloseBracketToken, context);
     }
 }
diff --git a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/ParameterList.cs b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/ParameterList.cs
@@ -3,9 +3,19 @@ namespace CSharpier.SyntaxPrinter.SyntaxNodePrinters;
 internal static class ParameterList
 {
     public static Doc Print(ParameterListSyntax node, FormattingContext context)
+    {
+        return Print(node, node.OpenParenToken, node.CloseParenToken, context);
+    }
+
+    public static Doc Print(
+        BaseParameterListSyntax node,
+        SyntaxToken openToken,
+        SyntaxToken closeToken,
+        FormattingContext context
+    )
     {
         return Doc.Group(
-            Token.Print(node.OpenParenToken, context),
+            Token.Print(openToken, context),
             node.Parameters.Count > 0
                 ? Doc.Concat(
                     Doc.Indent(
@@ -20,7 +30,7 @@ public static Doc Print(ParameterListSyntax node, FormattingContext context)
                     Doc.SoftLine
                 )
                 : Doc.Null,
-            Token.Print(node.CloseParenToken, context)
+            Token.Print(closeToken, context)
         );
     }
 }