From 77cb5c223ee393f2fb7819ce6ed7c614b1f40344 Mon Sep 17 00:00:00 2001
From: Albert Meltzer <7529386+kitbellew@users.noreply.github.com>
Date: Sat, 12 Aug 2023 01:34:13 +0300
Subject: [PATCH] Router: refactor modification before `(`

---
 .../scala/org/scalafmt/internal/Router.scala  | 25 ++++++++-----------
 1 file changed, 11 insertions(+), 14 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
index f27f6f3986..f25a74d069 100644
--- a/scalafmt-core/shared/src/main/scala/org/scalafmt/internal/Router.scala
+++ b/scalafmt-core/shared/src/main/scala/org/scalafmt/internal/Router.scala
@@ -671,24 +671,21 @@ class Router(formatOps: FormatOps) {
               case _ => false
             }
           } =>
-        def modification: Modification = leftOwner match {
-          case _: Mod => Space
+        def modification: Modification = Space(leftOwner match {
+          case _ if left.is[T.Comment] => true
+          case _: Mod => true
           // Add a space between constructor annotations and their parameter lists
           // see:
           // https://github.com/scalameta/scalafmt/pull/1516
           // https://github.com/scalameta/scalafmt/issues/1528
-          case init: Init if init.parent.forall(_.is[Mod.Annot]) => Space
-          case t: Term.Name
-              if style.spaces.afterTripleEquals && t.value == "===" =>
-            Space
-          case name: Term.Name
-              if style.spaces.afterSymbolicDefs &&
-                isSymbolicName(name.value) && name.parent.exists(isDefDef) =>
-            Space
-          case _: Defn.ExtensionGroup if left.is[soft.KwExtension] =>
-            Space(style.spaces.afterKeywordBeforeParen)
-          case _ => Space(left.is[T.Comment])
-        }
+          case t: Init => t.parent.forall(_.is[Mod.Annot])
+          case t @ Term.Name(name) if isSymbolicName(name) =>
+            style.spaces.afterTripleEquals && name == "===" ||
+            style.spaces.afterSymbolicDefs && t.parent.exists(isDefDef)
+          case _: Defn.ExtensionGroup =>
+            style.spaces.afterKeywordBeforeParen && left.is[soft.KwExtension]
+          case _ => false
+        })
         def baseNoSplit(implicit fileLine: FileLine) = Split(modification, 0)
         val defn = isParamClauseSite(rightOwner)
         val defRhs = if (defn) defDefBodyParent(rightOwner) else None