Fixing edge case for open generics

closes #832
belav · Feb 20, 2023 · 4ec90be · 4ec90be
1 parent 7210917
commit 4ec90be
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/Src/CSharpier.Tests/FormattingTests/TestFiles/TypeArgumentLists.cst b/Src/CSharpier.Tests/FormattingTests/TestFiles/TypeArgumentLists.cst
@@ -0,0 +1,4 @@
+typeof(AnExceptionallyLongAndElaborateClassNameToMakeAnExampleRegardingOpenGenerics<,>).MakeGenericType(
+ typeof(string),
+ typeof(int)
+);
diff --git a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/TypeArgumentList.cs b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/TypeArgumentList.cs
@@ -5,7 +5,8 @@ internal static class TypeArgumentList
  public static Doc Print(TypeArgumentListSyntax node, FormattingContext context)
  {
  Doc separator =
- node.Arguments.Count > 1 || node.Arguments.Any(o => o is GenericNameSyntax)
+ node.Arguments.FirstOrDefault() is not OmittedTypeArgumentSyntax
+ && (node.Arguments.Count > 1 || node.Arguments.Any(o => o is GenericNameSyntax))
  ? Doc.SoftLine
  : Doc.Null;