Merge pull request #194 from belav/initializer-extra-line

fixing extra line bug with complex initializer
belav · May 15, 2021 · b1560d0 · b1560d0
2 parents 514d9b8 + d241b6d
commit b1560d0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/Src/CSharpier.Tests/TestFiles/InitializerExpression/InitializerExpressions.cst b/Src/CSharpier.Tests/TestFiles/InitializerExpression/InitializerExpressions.cst
@@ -19,6 +19,10 @@ class ClassName
     {
         // should break braces correctly
         { 1, "one" },
-        { 2, "two" }
+        { 2, "two" },
+        {
+            3,
+            "ifThisIsReallyLongItShouldntInsertAnExtraLineAboveItttttttttttttttt"
+        }
     };
 }
diff --git a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/InitializerExpression.cs b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/InitializerExpression.cs
@@ -11,11 +11,9 @@ public static class InitializerExpression
         public static Doc Print(InitializerExpressionSyntax node)
         {
             var result = Doc.Concat(
-                node.Kind() == SyntaxKind.ArrayInitializerExpression
-                    ? string.Empty
-                    : node.Kind() == SyntaxKind.ComplexElementInitializerExpression
-                            ? Doc.SoftLine
-                            : Doc.Line,
+                node.Kind()
+                    is (SyntaxKind.ArrayInitializerExpression
+                        or SyntaxKind.ComplexElementInitializerExpression) ? Doc.Null : Doc.Line,
                 Token.Print(node.OpenBraceToken),
                 Doc.Indent(
                     Doc.Line,