Global cleanup change.

--
PiperOrigin-RevId: 141307853
MOS_MIGRATED_REVID=141307853

5 files changed, 9 insertions, 5 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/packages/AspectDefinition.java b/src/main/java/com/google/devtools/build/lib/packages/AspectDefinition.java
index 0878b2c762..24d28ae5b4 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/AspectDefinition.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/AspectDefinition.java
@@ -22,6 +22,7 @@ import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.ImmutableSetMultimap;
 import com.google.common.collect.LinkedHashMultimap;
 import com.google.common.collect.Multimap;
+import com.google.common.collect.SetMultimap;
 import com.google.devtools.build.lib.cmdline.Label;
 import com.google.devtools.build.lib.concurrent.ThreadSafety.Immutable;
 import com.google.devtools.build.lib.packages.ConfigurationFragmentPolicy.MissingFragmentPolicy;
@@ -170,7 +171,7 @@ public final class AspectDefinition {
   public static ImmutableMultimap<Attribute, Label> visitAspectsIfRequired(
       Rule from, Attribute attribute, boolean canHaveAnyProvider, Set<String> advertisedProviders,
       DependencyFilter dependencyFilter) {
-    LinkedHashMultimap<Attribute, Label> result = LinkedHashMultimap.create();
+    SetMultimap<Attribute, Label> result = LinkedHashMultimap.create();
     for (Aspect candidateClass : attribute.getAspects(from)) {
       // Check if target satisfies condition for this aspect (has to provide all required
       // TransitiveInfoProviders)
diff --git a/src/main/java/com/google/devtools/build/lib/packages/Rule.java b/src/main/java/com/google/devtools/build/lib/packages/Rule.java
index 5d233f0817..34b6ab5fb6 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/Rule.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/Rule.java
@@ -26,6 +26,7 @@ import com.google.common.collect.LinkedListMultimap;
 import com.google.common.collect.ListMultimap;
 import com.google.common.collect.Lists;
 import com.google.common.collect.Multimap;
+import com.google.common.collect.SetMultimap;
 import com.google.devtools.build.lib.cmdline.Label;
 import com.google.devtools.build.lib.cmdline.LabelSyntaxException;
 import com.google.devtools.build.lib.cmdline.RepositoryName;
@@ -695,7 +696,7 @@ public final class Rule implements Target, DependencyFilter.AttributeInfoProvide
    * can require from its direct dependencies.
    */
   public Collection<? extends Label> getAspectLabelsSuperset(DependencyFilter predicate) {
-    LinkedHashMultimap<Attribute, Label> labels = LinkedHashMultimap.create();
+    SetMultimap<Attribute, Label> labels = LinkedHashMultimap.create();
     for (Attribute attribute : this.getAttributes()) {
       for (Aspect candidateClass : attribute.getAspects(this)) {
         AspectDefinition.addAllAttributesOfAspect(Rule.this, labels, candidateClass, predicate);
diff --git a/src/main/java/com/google/devtools/build/lib/query2/ParallelSkyQueryUtils.java b/src/main/java/com/google/devtools/build/lib/query2/ParallelSkyQueryUtils.java
index 086ecfc1b8..30a27f8e63 100644
--- a/src/main/java/com/google/devtools/build/lib/query2/ParallelSkyQueryUtils.java
+++ b/src/main/java/com/google/devtools/build/lib/query2/ParallelSkyQueryUtils.java
@@ -21,6 +21,7 @@ import com.google.common.collect.Collections2;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Iterables;
+import com.google.common.collect.ListMultimap;
 import com.google.common.collect.Maps;
 import com.google.common.collect.Multimap;
 import com.google.common.util.concurrent.ThreadFactoryBuilder;
@@ -314,7 +315,7 @@ class ParallelSkyQueryUtils {
     @Override
     protected Iterable<Task> getVisitTasks(Collection<Pair<SkyKey, SkyKey>> pendingKeysToVisit) {
       // Group pending visits by package.
-      ArrayListMultimap<PackageIdentifier, Pair<SkyKey, SkyKey>> visitsByPackage =
+      ListMultimap<PackageIdentifier, Pair<SkyKey, SkyKey>> visitsByPackage =
           ArrayListMultimap.create();
       for (Pair<SkyKey, SkyKey> visit : pendingKeysToVisit) {
         Label label = SkyQueryEnvironment.SKYKEY_TO_LABEL.apply(visit.second);
diff --git a/src/main/java/com/google/devtools/build/lib/util/OrderedSetMultimap.java b/src/main/java/com/google/devtools/build/lib/util/OrderedSetMultimap.java
index a64ddf6602..79be3d6bb5 100644
--- a/src/main/java/com/google/devtools/build/lib/util/OrderedSetMultimap.java
+++ b/src/main/java/com/google/devtools/build/lib/util/OrderedSetMultimap.java
@@ -26,7 +26,7 @@ import com.google.common.collect.SetMultimap;
  * {@link Multimap} documentation for information common to all multimaps.
  */
 public final class OrderedSetMultimap<K, V> extends ForwardingSetMultimap<K, V> {
-  private final LinkedHashMultimap<K, V> delegate = LinkedHashMultimap.<K, V>create();
+  private final SetMultimap<K, V> delegate = LinkedHashMultimap.<K, V>create();
 
   @Override
   protected SetMultimap delegate() {
diff --git a/src/main/java/com/google/devtools/build/skyframe/CycleDeduper.java b/src/main/java/com/google/devtools/build/skyframe/CycleDeduper.java
index f5ac95482f..d32f559778 100644
--- a/src/main/java/com/google/devtools/build/skyframe/CycleDeduper.java
+++ b/src/main/java/com/google/devtools/build/skyframe/CycleDeduper.java
@@ -16,6 +16,7 @@ package com.google.devtools.build.skyframe;
 import com.google.common.collect.HashMultimap;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableSet;
+import com.google.common.collect.SetMultimap;
 import com.google.devtools.build.lib.util.Preconditions;
 
 /**
@@ -27,7 +28,7 @@ import com.google.devtools.build.lib.util.Preconditions;
  */
 class CycleDeduper<T> {
 
-  private HashMultimap<ImmutableSet<T>, ImmutableList<T>> knownCyclesByMembers =
+  private SetMultimap<ImmutableSet<T>, ImmutableList<T>> knownCyclesByMembers =
       HashMultimap.create();
 
   /**