PlatformCompat: inline fixPathOnNativeWindows

scalameta · Feb 2, 2025 · 5c9e607 · 5c9e607
1 parent bdccbae
commit 5c9e607
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 7 deletions.
diff --git a/scalafmt-cli/shared/src/test/scala/org/scalafmt/cli/CliTest.scala b/scalafmt-cli/shared/src/test/scala/org/scalafmt/cli/CliTest.scala
@@ -726,10 +726,8 @@ trait CliTestBehavior {
         ExitCode.TestError,
         assertOut = out => {
           assertContains(out, "bar.scala")
-          assertContains(
-            out,
-            PlatformCompat.fixPathOnNativeWindows("dir/foo.scala"),
-          )
+          val bslash = PlatformCompat.isNativeOnWindows
+          assertContains(out, if (bslash) "dir\\foo.scala" else "dir/foo.scala")
           assertNotContains(out, "baz.scala")
         },
       )

diff --git a/scalafmt-sysops/jvm/src/main/scala/org/scalafmt/sysops/PlatformCompat.scala b/scalafmt-sysops/jvm/src/main/scala/org/scalafmt/sysops/PlatformCompat.scala
@@ -5,7 +5,6 @@ import scala.concurrent.ExecutionContext
 private[scalafmt] object PlatformCompat {
   def isScalaNative = false
   def prepareCommand(cmd: Seq[String]) = cmd
-  def fixPathOnNativeWindows(path: String) = path
   def isNativeOnWindows = false
 
   implicit def executionContext: ExecutionContext = ExecutionContext.global

diff --git a/scalafmt-sysops/native/src/main/scala/org/scalafmt/sysops/PlatformCompat.scala b/scalafmt-sysops/native/src/main/scala/org/scalafmt/sysops/PlatformCompat.scala
@@ -7,8 +7,6 @@ private[scalafmt] object PlatformCompat {
   def isScalaNative = true
   def prepareCommand(cmd: Seq[String]) =
     if (Platform.isWindows()) cmd.map(arg => s""""$arg"""") else cmd
-  def fixPathOnNativeWindows(path: String) =
-    if (Platform.isWindows()) path.replace('/', '\\') else path
   def isNativeOnWindows = Platform.isWindows()
 
   implicit def executionContext: ExecutionContext = ExecutionContext.global