Eliminate SkylarkType.toMap() in favor of ImmutableMap.copyOf() and simplify

SkylarkType.castMap() a little.

--
MOS_MIGRATED_REVID=90988088

1 files changed, 1 insertions, 1 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/packages/ImplicitOutputsFunction.java b/src/main/java/com/google/devtools/build/lib/packages/ImplicitOutputsFunction.java
index bd54ca0bc6..1777a7e4ca 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/ImplicitOutputsFunction.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/ImplicitOutputsFunction.java
@@ -98,7 +98,7 @@ public abstract class ImplicitOutputsFunction {
       try {
         ImmutableMap.Builder<String, String> builder = ImmutableMap.builder();
         for (Map.Entry<String, String> entry : castMap(callback.call(attrs),
-            String.class, String.class, "implicit outputs function return value")) {
+            String.class, String.class, "implicit outputs function return value").entrySet()) {
           Iterable<String> substitutions = fromTemplates(entry.getValue()).getImplicitOutputs(map);
           if (!Iterables.isEmpty(substitutions)) {
             builder.put(entry.getKey(), Iterables.getOnlyElement(substitutions));