commit 899a3e7ad9853c036bc84b7b6905516df6b7f62b
author Benjamin Peterson <bp@benjamin.pe> Wed Jun 06 14:18:29 2018 -0700
committer Copybara-Service <copybara-piper@google.com> Wed Jun 06 14:19:42 2018 -0700
tree a53a804fa17065e8476338b6bd039b7ebf5a4708
parent af8b772560301f89de251dc494b3c5669451c5df

Remove a pointless nested set expansion.

Change-Id: Icdabf559303bfac459e1a88ff44ca95298574361

Closes #5333.

Change-Id: Icdabf559303bfac459e1a88ff44ca95298574361
PiperOrigin-RevId: 199527673

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

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/Runfiles.java b/src/main/java/com/google/devtools/build/lib/analysis/Runfiles.java
index d14029b..93c3cef 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/Runfiles.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/Runfiles.java
@@ -345,7 +345,7 @@
   @Override
   public NestedSet<Artifact> getArtifacts() {
     NestedSetBuilder<Artifact> allArtifacts = NestedSetBuilder.stableOrder();
-    allArtifacts.addAll(unconditionalArtifacts.toCollection());
+    allArtifacts.addTransitive(unconditionalArtifacts);
     for (PruningManifest manifest : getPruningManifests()) {
       allArtifacts.addTransitive(manifest.getCandidateRunfiles());
     }