From da26f3078b23636e7c7597e44e3e73449bf2cb1a Mon Sep 17 00:00:00 2001 From: xuwei-k <6b656e6a69@gmail.com> Date: Sat, 4 Jan 2025 09:10:36 +0900 Subject: [PATCH] use withFilter instead of filter --- .../src/main/scala/sbt/internal/util/appmacro/Cont.scala | 2 +- .../main/scala/sbt/internal/util/appmacro/ContextUtil.scala | 2 +- .../main/scala/sbt/internal/util/complete/ExampleSource.scala | 2 +- .../src/main/scala/lmcoursier/internal/SbtUpdateReport.scala | 2 +- .../main/scala/sbt/internal/librarymanagement/MakePom.scala | 2 +- .../mavenint/PomExtraDependencyAttributes.scala | 2 +- main/src/main/scala/sbt/Defaults.scala | 4 ++-- .../main/scala/sbt/internal/AbstractTaskExecuteProgress.scala | 2 +- .../main/scala/sbt/internal/DefaultBackgroundJobService.scala | 2 +- main/src/main/scala/sbt/internal/PluginDiscovery.scala | 2 +- .../main/scala/sbt/internal/WatchTransitiveDependencies.scala | 2 +- util-cache/src/main/scala/sbt/util/ActionCache.scala | 2 +- util-cache/src/main/scala/sbt/util/ActionCacheStore.scala | 2 +- 13 files changed, 14 insertions(+), 14 deletions(-) diff --git a/core-macros/src/main/scala/sbt/internal/util/appmacro/Cont.scala b/core-macros/src/main/scala/sbt/internal/util/appmacro/Cont.scala index 4adeb0c35..5c9ddbbdb 100644 --- a/core-macros/src/main/scala/sbt/internal/util/appmacro/Cont.scala +++ b/core-macros/src/main/scala/sbt/internal/util/appmacro/Cont.scala @@ -278,7 +278,7 @@ trait Cont: .asExprOf[BuildWideCacheConfiguration] if inputs.exists(_.isCacheInput) then val tags = inputs - .filter(_.isCacheInput) + .withFilter(_.isCacheInput) .map(_.tags.toSet) .reduce(_ & _) .toList diff --git a/core-macros/src/main/scala/sbt/internal/util/appmacro/ContextUtil.scala b/core-macros/src/main/scala/sbt/internal/util/appmacro/ContextUtil.scala index 070ce99f6..1c3c77987 100644 --- a/core-macros/src/main/scala/sbt/internal/util/appmacro/ContextUtil.scala +++ b/core-macros/src/main/scala/sbt/internal/util/appmacro/ContextUtil.scala @@ -72,7 +72,7 @@ trait ContextUtil[C <: Quotes & scala.Singleton](val valStart: Int): override def tupleExpr: Expr[Tuple] = exprOfTupleFromSeq(inputs.map(_.term.asExpr)) override def cacheInputTupleTypeRepr: TypeRepr = - tupleTypeRepr(inputs.filter(_.isCacheInput).map(_.tpe)) + tupleTypeRepr(inputs.withFilter(_.isCacheInput).map(_.tpe)) override def cacheInputExpr(tupleTerm: Term): Expr[Tuple] = exprOfTupleFromSeq(inputs.zipWithIndex.flatMap { case (input, idx) => if input.tags.nonEmpty then diff --git a/internal/util-complete/src/main/scala/sbt/internal/util/complete/ExampleSource.scala b/internal/util-complete/src/main/scala/sbt/internal/util/complete/ExampleSource.scala index bced54930..bf247b52b 100644 --- a/internal/util-complete/src/main/scala/sbt/internal/util/complete/ExampleSource.scala +++ b/internal/util-complete/src/main/scala/sbt/internal/util/complete/ExampleSource.scala @@ -72,7 +72,7 @@ class FileExamples(base: File, prefix: String = "") extends ExampleSource { .map(IO.relativize(base, _).get) .filter(_.startsWith(prefix)) val dirsToRecurseInto = childPaths - .filter(_.isDirectory) + .withFilter(_.isDirectory) .map(IO.relativize(base, _).get) .filter(dirStartsWithPrefix) prefixedDirectChildPaths ++ dirsToRecurseInto.flatMap(dir => files(new File(base, dir))) diff --git a/lm-coursier/src/main/scala/lmcoursier/internal/SbtUpdateReport.scala b/lm-coursier/src/main/scala/lmcoursier/internal/SbtUpdateReport.scala index ea63b100b..cc95a56ab 100644 --- a/lm-coursier/src/main/scala/lmcoursier/internal/SbtUpdateReport.scala +++ b/lm-coursier/src/main/scala/lmcoursier/internal/SbtUpdateReport.scala @@ -206,7 +206,7 @@ private[internal] object SbtUpdateReport { dep -> m.getOrElse(dep, Nil).map { case (_, pub, a, f) => (pub, a, f) } } val fromInterProj = interProjectDependencies - .filter(p => p.module != thisModule._1) + .withFilter(p => p.module != thisModule._1) .map(p => Dependency(p.module, p.version) -> Nil) fromLib ++ fromInterProj } diff --git a/lm-ivy/src/main/scala/sbt/internal/librarymanagement/MakePom.scala b/lm-ivy/src/main/scala/sbt/internal/librarymanagement/MakePom.scala index 3e12d3c71..4347ce3ca 100644 --- a/lm-ivy/src/main/scala/sbt/internal/librarymanagement/MakePom.scala +++ b/lm-ivy/src/main/scala/sbt/internal/librarymanagement/MakePom.scala @@ -393,7 +393,7 @@ class MakePom(val log: Logger) { MavenRepository(m.getName, m.getRoot) :: Nil case _ => Nil } - val repositoryElements = mavenRepositories.filter(filterRepositories).map(mavenRepository) + val repositoryElements = mavenRepositories.withFilter(filterRepositories).map(mavenRepository) if (repositoryElements.isEmpty) repositoryElements else { diff --git a/lm-ivy/src/main/scala/sbt/internal/librarymanagement/mavenint/PomExtraDependencyAttributes.scala b/lm-ivy/src/main/scala/sbt/internal/librarymanagement/mavenint/PomExtraDependencyAttributes.scala index 5b782274c..3f311b3ce 100644 --- a/lm-ivy/src/main/scala/sbt/internal/librarymanagement/mavenint/PomExtraDependencyAttributes.scala +++ b/lm-ivy/src/main/scala/sbt/internal/librarymanagement/mavenint/PomExtraDependencyAttributes.scala @@ -100,7 +100,7 @@ object PomExtraDependencyAttributes { /** parses the sequence of dependencies with extra attribute information, with one dependency per line */ def readDependencyExtra(s: String): Seq[ModuleRevisionId] = ArraySeq.unsafeWrapArray( - LinesP.split(s).map(_.trim).filter(!_.isEmpty).map(ModuleRevisionId.decode) + LinesP.split(s).map(_.trim).withFilter(!_.isEmpty).map(ModuleRevisionId.decode) ) private val LinesP = Pattern.compile("(?m)^") diff --git a/main/src/main/scala/sbt/Defaults.scala b/main/src/main/scala/sbt/Defaults.scala index 17269b066..c09f3abad 100644 --- a/main/src/main/scala/sbt/Defaults.scala +++ b/main/src/main/scala/sbt/Defaults.scala @@ -1598,7 +1598,7 @@ object Defaults extends BuildCommon { val xs = products.value xs .flatMap(Path.allSubpaths) - .filter(_._1.isFile()) + .withFilter(_._1.isFile()) .map { case (p, path) => val vf = converter.toVirtualFile(p.toPath()) (vf: HashedVirtualFileRef) -> path @@ -1612,7 +1612,7 @@ object Defaults extends BuildCommon { Path .allSubpaths(d) .toSeq - .filter(_._1.isFile()) + .withFilter(_._1.isFile()) .map { case (p, path) => val vf = converter.toVirtualFile(p.toPath()) (vf: HashedVirtualFileRef) -> path diff --git a/main/src/main/scala/sbt/internal/AbstractTaskExecuteProgress.scala b/main/src/main/scala/sbt/internal/AbstractTaskExecuteProgress.scala index 0e89228f6..fac7ccd7c 100644 --- a/main/src/main/scala/sbt/internal/AbstractTaskExecuteProgress.scala +++ b/main/src/main/scala/sbt/internal/AbstractTaskExecuteProgress.scala @@ -75,7 +75,7 @@ private[sbt] abstract class AbstractTaskExecuteProgress extends ExecuteProgress ): Unit = { // we need this to infer anonymous task names pendingDeps - .filter { + .withFilter { case t: Task[?] => TaskName.transformNode(t).isEmpty case _ => true } diff --git a/main/src/main/scala/sbt/internal/DefaultBackgroundJobService.scala b/main/src/main/scala/sbt/internal/DefaultBackgroundJobService.scala index 1d9542f3b..936fe9c57 100644 --- a/main/src/main/scala/sbt/internal/DefaultBackgroundJobService.scala +++ b/main/src/main/scala/sbt/internal/DefaultBackgroundJobService.scala @@ -544,7 +544,7 @@ private[sbt] object DefaultBackgroundJobService { backgroundJobServices .values() .forEach(jobService => { - jobService.jobs.filter(_.isAutoCancel).foreach(jobService.stop) + jobService.jobs.withFilter(_.isAutoCancel).foreach(jobService.stop) }) } diff --git a/main/src/main/scala/sbt/internal/PluginDiscovery.scala b/main/src/main/scala/sbt/internal/PluginDiscovery.scala index 1ac817be5..010194238 100644 --- a/main/src/main/scala/sbt/internal/PluginDiscovery.scala +++ b/main/src/main/scala/sbt/internal/PluginDiscovery.scala @@ -137,7 +137,7 @@ object PluginDiscovery: .getResources(resourceName) .asScala .toSeq - .filter(onClasspath(classpath, converter)) + .withFilter(onClasspath(classpath, converter)) .flatMap { u => IO.readLinesURL(u).map(_.trim).filter(!_.isEmpty) } diff --git a/main/src/main/scala/sbt/internal/WatchTransitiveDependencies.scala b/main/src/main/scala/sbt/internal/WatchTransitiveDependencies.scala index 643ceb588..dfe16271e 100644 --- a/main/src/main/scala/sbt/internal/WatchTransitiveDependencies.scala +++ b/main/src/main/scala/sbt/internal/WatchTransitiveDependencies.scala @@ -148,7 +148,7 @@ private[sbt] object WatchTransitiveDependencies { val projects = projectScopes.flatMap(_.project.toOption).distinct.toSet val scopes: Seq[Either[Scope, Seq[Glob]]] = data.scopes.toSeq - .filter(s => s == Scope.Global || s.project.toOption.exists(projects.contains)) + .withFilter(s => s == Scope.Global || s.project.toOption.exists(projects.contains)) .flatMap { s => data.getDirect(ScopedKey(s, Keys.watchSources.key)).map { task => task.work match diff --git a/util-cache/src/main/scala/sbt/util/ActionCache.scala b/util-cache/src/main/scala/sbt/util/ActionCache.scala index 599b6da92..60e0b74c6 100644 --- a/util-cache/src/main/scala/sbt/util/ActionCache.scala +++ b/util-cache/src/main/scala/sbt/util/ActionCache.scala @@ -157,7 +157,7 @@ object ActionCache: val dirPath = conv.toPath(dir) val allPaths = FileTreeView.default .list(dirPath.toGlob / ** / "*") - .filter(!_._2.isDirectory) + .withFilter(!_._2.isDirectory) .map(_._1) .sortBy(_.toString()) // create a manifest of files and their hashes here diff --git a/util-cache/src/main/scala/sbt/util/ActionCacheStore.scala b/util-cache/src/main/scala/sbt/util/ActionCacheStore.scala index e043a969e..000d6fae3 100644 --- a/util-cache/src/main/scala/sbt/util/ActionCacheStore.scala +++ b/util-cache/src/main/scala/sbt/util/ActionCacheStore.scala @@ -318,7 +318,7 @@ class DiskActionCacheStore(base: Path, converter: FileConverter) extends Abstrac val allPaths = mutable.Set( FileTreeView.default .list(dirPath.toGlob / ** / "*") - .filter(!_._2.isDirectory) + .withFilter(!_._2.isDirectory) .map(_._1)* ) def doSync(ref: HashedVirtualFileRef, in: Path): Unit =