From 9315740feb3357d9e5fac5f31e8f04f8036c0648 Mon Sep 17 00:00:00 2001 From: kak Date: Thu, 8 Dec 2016 16:06:09 -0800 Subject: [PATCH] Prefer Multimap interface types over implementation types. This change is required before the migration to MultimapBuilder can be completed. ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=141496601 --- src/com/google/javascript/jscomp/RenameVars.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/com/google/javascript/jscomp/RenameVars.java b/src/com/google/javascript/jscomp/RenameVars.java index 10568787f1f..59448b0551a 100644 --- a/src/com/google/javascript/jscomp/RenameVars.java +++ b/src/com/google/javascript/jscomp/RenameVars.java @@ -21,10 +21,10 @@ import com.google.common.base.Preconditions; import com.google.common.collect.ArrayListMultimap; import com.google.common.collect.ImmutableMap; +import com.google.common.collect.ListMultimap; import com.google.javascript.jscomp.NodeTraversal.AbstractPostOrderCallback; import com.google.javascript.jscomp.NodeTraversal.ScopedCallback; import com.google.javascript.rhino.Node; - import java.util.ArrayList; import java.util.Comparator; import java.util.HashMap; @@ -34,7 +34,6 @@ import java.util.Set; import java.util.SortedSet; import java.util.TreeSet; - import javax.annotation.Nullable; /** @@ -88,7 +87,7 @@ final class RenameVars implements CompilerPass { // Logic for bleeding functions, where the name leaks into the outer // scope on IE but not on other browsers. private final Set localBleedingFunctions = new HashSet<>(); - private final ArrayListMultimap localBleedingFunctionsPerScope = + private final ListMultimap localBleedingFunctionsPerScope = ArrayListMultimap.create(); class Assignment {