commit 61216fd405e9b80ce89ffafbb6518e859679f4ab
author ulfjack <ulfjack@google.com> Wed Dec 18 12:31:03 2019 -0800
committer Copybara-Service <copybara-worker@google.com> Wed Dec 18 12:32:14 2019 -0800
tree ab355c041214d246e825dd6335f523d05dc9cbdc
parent 402d5036995855ac049e7b549e1f3b0a605068e9

Prefer List/NestedSet over Iterable

This is in preparation for making NestedSet not implement Iterable
anymore.

PiperOrigin-RevId: 286243512

src/main/java/com/google/devtools/build/lib/analysis/BuildView.java

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/BuildView.java b/src/main/java/com/google/devtools/build/lib/analysis/BuildView.java
index b77535b..b3f92ab 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/BuildView.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/BuildView.java
@@ -675,7 +675,8 @@
     ImmutableList.Builder<Artifact> artifacts = ImmutableList.builder();
     // Add to 'artifacts' all extra-actions which were registered by aspects which 'topLevel'
     // might have injected.
-    for (Artifact.DerivedArtifact artifact : provider.getTransitiveExtraActionArtifacts()) {
+    for (Artifact.DerivedArtifact artifact :
+        provider.getTransitiveExtraActionArtifacts().toList()) {
       ActionLookupValue.ActionLookupKey owner = artifact.getArtifactOwner();
       if (owner instanceof AspectKey) {
         if (aspectClasses.contains(((AspectKey) owner).getAspectClass())) {