From 5c9e607adc681d83b6c1cf6fe7869d6433745b1a Mon Sep 17 00:00:00 2001 From: Albert Meltzer <7529386+kitbellew@users.noreply.github.com> Date: Sun, 2 Feb 2025 07:45:19 -0800 Subject: [PATCH] PlatformCompat: inline `fixPathOnNativeWindows` --- .../shared/src/test/scala/org/scalafmt/cli/CliTest.scala | 6 ++---- .../src/main/scala/org/scalafmt/sysops/PlatformCompat.scala | 1 - .../src/main/scala/org/scalafmt/sysops/PlatformCompat.scala | 2 -- 3 files changed, 2 insertions(+), 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 index 262791ed00..d7466823b3 100644 --- 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 index 009e47d5d8..9aba9c181b 100644 --- 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 index ce8c4c5939..38bd8fe21e 100644 --- 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