[2.x] Remove redundant conversions (#1673)

Author: xuwei-k

internal/compiler-bridge-test/src/test/scala/sbt/internal/inc/CompilingSpecification.scala

@@ -82,7 +82,7 @@ trait CompilingSpecification extends AbstractBridgeProviderTestkit {
 
   def extractBinaryClassNamesFromSrc(src: String): Set[(String, String)] = {
     val (Seq(tempSrcFile), analysisCallback) = compileSrcs(src)
-    analysisCallback.classNames(tempSrcFile).toSet
+    analysisCallback.classNames(tempSrcFile)
   }
 
   /**

internal/zinc-apiinfo/src/main/scala/sbt/internal/inc/ClassToAPI.scala

@@ -427,7 +427,7 @@ object ClassToAPI {
     val pa = paramAnnots.lazyZip(paramTypes).lazyZip(isVarArg).map {
       case (a, p, v) => parameter(a, p, v)
     }
-    val params = api.ParameterList.of(pa.toArray, false)
+    val params = api.ParameterList.of(pa, false)
     val ret = retType match { case Some(rt) => reference(rt); case None => Empty }
     api.Def.of(
       name,

internal/zinc-scripted/src/test/scala/sbt/internal/inc/IncHandler.scala

@@ -681,7 +681,7 @@ case class ProjectStructure(
   ): Analysis = {
     import i._
     val sources = scalaSources ++ javaSources
-    val vs = sources.toList.map(converter.toVirtualFile)
+    val vs = sources.map(converter.toVirtualFile)
     val entryLookup = new PerClasspathEntryLookupImpl(lookupAnalysis, Locate.definesClass)
     val reporter = new ManagedLoggedReporter(maxErrors, scriptedLog)
     val extra = Array(t2(("key", "value")))

project/Scripted.scala

@@ -25,7 +25,7 @@ object Scripted {
     val pairMap = pairs.groupBy(_._1).mapValues(_.map(_._2).toSet);
 
     val id = charClass(c => !c.isWhitespace && c != '/').+.string
-    val groupP = token(id.examples(pairMap.keySet.toSet)) <~ token('/')
+    val groupP = token(id.examples(pairMap.keySet)) <~ token('/')
 
     // A parser for page definitions
     val pageP: Parser[ScriptedTestPage] = ("*" ~ NatBasic ~ "of" ~ NatBasic) map {