Better breaking for generic object creation expressions with no argum…

…ents or initliazer (#589) closes #578 Co-authored-by: Lasath Fernando <devel@lasath.org>
belav · Feb 6, 2022 · c7b1371 · c7b1371
1 parent cf43c06
commit c7b1371
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 10 deletions.
diff --git a/Src/CSharpier.Tests/FormattingTests/TestFiles/FieldDeclarations.cst b/Src/CSharpier.Tests/FormattingTests/TestFiles/FieldDeclarations.cst
@@ -54,18 +54,26 @@ public class ClassName
 
     public System.Text.StringBuilder NamespacedField;
 
-    public static Dictionary<Type, string> PropertiesByType = new Dictionary<
-        SomeLongTypeName,
-        string
-    >();
+    public Dictionary<SomeType__________________, SomeType___________________> Property =
+        new Dictionary<SomeType__________________, SomeType__________________>();
+
+    public Dictionary<SomeType_________, SomeType_________> Property =
+        new Dictionary<SomeType_________, SomeType_________>();
 
     public static Dictionary<
-        SomeLongTypeName,
-        SomeOtherLongTypeName_______________________________
-    > PropertiesByType = new Dictionary<
-        SomeLongTypeName,
+        SomeOtherLongTypeName_______________,
+        SomeOtherLongTypeName_______________
+    > PropertiesByType =
+        new Dictionary<SomeOtherLongTypeName____________, SomeOtherLongTypeName____________>();
+
+    public static Dictionary<
+        SomeOtherLongTypeName_______________________________,
         SomeOtherLongTypeName_______________________________
-    >();
+    > PropertiesByType =
+        new Dictionary<
+            SomeOtherLongTypeName_______________________________,
+            SomeOtherLongTypeName_______________________________
+        >();
 
     private static readonly Func<SyntaxTrivia, bool> s_isVisualBasicCommentTrivia = (
         syntaxTrivia

diff --git a/Src/CSharpier/SyntaxPrinter/RightHandSide.cs b/Src/CSharpier/SyntaxPrinter/RightHandSide.cs
@@ -64,7 +64,17 @@ or ArrowExpressionClauseSyntax
         }
 
         if (
-            !isTail && rightNode is AssignmentExpressionSyntax { Right: AssignmentExpressionSyntax }
+            (
+                !isTail
+                && rightNode is AssignmentExpressionSyntax { Right: AssignmentExpressionSyntax }
+            )
+            || rightNode
+                is ObjectCreationExpressionSyntax
+                {
+                    Type: GenericNameSyntax,
+                    ArgumentList: { Arguments: { Count: 0 } },
+                    Initializer: null
+                }
         )
         {
             return Layout.BreakAfterOperator;