Fix empty line between try and finally with no catch

closes #454
belav · Oct 13, 2021 · 888ff29 · 888ff29
1 parent 8bfd431
commit 888ff29
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/Src/CSharpier.Tests/FormattingTests/TestFiles/TryStatements.cst b/Src/CSharpier.Tests/FormattingTests/TestFiles/TryStatements.cst
@@ -22,6 +22,9 @@ class ClassName
         try { }
         catch { }
 
+        try { }
+        finally { }
+
         try
         {
             throw null;
@@ -52,5 +55,14 @@ class ClassName
         {
             return;
         }
+
+        try
+        {
+            //something
+        }
+        finally
+        {
+            // do something
+        }
     }
 }
diff --git a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/TryStatement.cs b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/TryStatement.cs
@@ -17,7 +17,7 @@ public static Doc Print(TryStatementSyntax node)
                 AttributeLists.Print(node, node.AttributeLists),
                 Token.Print(node.TryKeyword),
                 Block.Print(node.Block),
-                Doc.HardLine,
+                node.Catches.Any() ? Doc.HardLine : Doc.Null,
                 Doc.Join(Doc.HardLine, node.Catches.Select(CatchClause.Print))
             };
             if (node.Finally != null)