Router: remove currently unused comma split

Follow-on to scalameta#1722.
kitbellew · Mar 4, 2020 · 3858c65 · 3858c65
1 parent 6b4096c
commit 3858c65
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/scalafmt-core/shared/src/main/scala/org/scalafmt/internal/Router.scala b/scalafmt-core/shared/src/main/scala/org/scalafmt/internal/Router.scala
@@ -926,11 +926,12 @@ class Router(formatOps: FormatOps) {
                   Split(Space, 1, policy = multiLine)
                 )
             }).map(_.onlyFor(SplitTag.OneArgPerLine))
-        def oneLineBody = open.pos.endLine == close.pos.startLine
         Seq(
           Split(Space, 0),
           Split(Newline, 0)
-            .onlyIf(newlines != 0 && oneLineBody)
+            .onlyIf(oneArgPerLineSplits.isEmpty)
+            .onlyIf(newlines != 0)
+            .onlyIf(open.pos.endLine == close.pos.startLine)
             .withOptimalToken(close, killOnFail = true)
             .withPolicy(SingleLineBlock(close))
         ) ++ oneArgPerLineSplits