Get rid of redundant list copy

vadimsemenov · pocmo · commit e47dcc84fa82 · 2018-10-29T15:14:17.000+01:00
List.filter creates a new List, so
there is no need to call toList() on it's result
diff --git a/components/service/fretboard/src/main/java/mozilla/components/service/fretboard/Fretboard.kt b/components/service/fretboard/src/main/java/mozilla/components/service/fretboard/Fretboard.kt
@@ -104,7 +104,7 @@ class Fretboard(
      * @return active experiments
      */
     fun getActiveExperiments(context: Context): List<Experiment> {
-        return experiments.filter { isInExperiment(context, ExperimentDescriptor(it.name)) }.toList()
+        return experiments.filter { isInExperiment(context, ExperimentDescriptor(it.name)) }
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ class Fretboard(`
`104`	`104`	`* @return active experiments`
`105`	`105`	`*/`
`106`	`106`	`fun getActiveExperiments(context: Context): List<Experiment> {`
`107`		`- return experiments.filter { isInExperiment(context, ExperimentDescriptor(it.name)) }.toList()`
	`107`	`+ return experiments.filter { isInExperiment(context, ExperimentDescriptor(it.name)) }`
`108`	`108`	`}`
`109`	`109`
`110`	`110`	`/**`