Prefer immutable type as return type.

meisterT · copybara-github · commit f0ed5ca13d7c · 2023-01-12T01:03:54.000-08:00
PiperOrigin-RevId: 501491397
Change-Id: Ia638878cb78db7b82aaa800ca82bbafeec0f4218
diff --git a/src/main/java/com/google/devtools/build/lib/profiler/Profiler.java b/src/main/java/com/google/devtools/build/lib/profiler/Profiler.java
@@ -279,7 +279,7 @@ void clear() {
     }
 
     // @ThreadSafe
-    Iterable<SlowTask> getSlowestTasks() {
+    ImmutableList<SlowTask> getSlowestTasks() {
       // This is slow, but since it only happens during the end of the invocation, it's OK.
       Extrema<SlowTask> mergedExtrema = Extrema.max(SIZE);
       for (int i = 0; i < SHARDS; i++) {

Original file line number	Diff line number	Diff line change
`@@ -279,7 +279,7 @@ void clear() {`
`279`	`279`	`}`
`280`	`280`
`281`	`281`	`// @ThreadSafe`
`282`		`- Iterable<SlowTask> getSlowestTasks() {`
	`282`	`+ ImmutableList<SlowTask> getSlowestTasks() {`
`283`	`283`	`// This is slow, but since it only happens during the end of the invocation, it's OK.`
`284`	`284`	`Extrema<SlowTask> mergedExtrema = Extrema.max(SIZE);`
`285`	`285`	`for (int i = 0; i < SHARDS; i++) {`