From a34d5071784ff51f68714b61f4100c35f1e4db3a Mon Sep 17 00:00:00 2001
From: Googler <noreply@google.com>
Date: Thu, 5 Mar 2015 13:51:28 +0000
Subject: Skylark: fix auto type conversion between the BUILD language and
 Skylark. SkylarkList#toList() behaves consistently for all implementations.

--
MOS_MIGRATED_REVID=87817550
---
 .../build/lib/syntax/FuncallExpression.java        | 25 ++++++++++++++++------
 1 file changed, 18 insertions(+), 7 deletions(-)

(limited to 'src/main/java/com/google/devtools/build/lib/syntax/FuncallExpression.java')

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/FuncallExpression.java b/src/main/java/com/google/devtools/build/lib/syntax/FuncallExpression.java
index 3ef5973d44..fc228227c5 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/FuncallExpression.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/FuncallExpression.java
@@ -470,11 +470,13 @@ public final class FuncallExpression extends Expression {
     // we'd still have to have a HashMap on the side for the sake of properly handling duplicates.
     Map<String, Object> kwargs = new HashMap<>();
 
+    final Object returnValue;
+    final Function function;
     if (obj != null) { // obj.func(...)
       Object objValue = obj.eval(env);
       // Strings, lists and dictionaries (maps) have functions that we want to use in MethodLibrary.
       // For other classes, we can call the Java methods.
-      Function function =
+      function =
           env.getFunction(EvalUtils.getSkylarkType(objValue.getClass()), func.getName());
       if (function != null) {
         if (!isNamespace(objValue.getClass())) {
@@ -482,8 +484,8 @@ public final class FuncallExpression extends Expression {
           posargs.add(objValue);
         }
         evalArguments(posargs, kwargs, env, function);
-        return EvalUtils.checkNotNull(this,
-            function.call(posargs.build(), ImmutableMap.<String, Object>copyOf(kwargs), this, env));
+        returnValue = function.call(
+            posargs.build(), ImmutableMap.<String, Object>copyOf(kwargs), this, env);
       } else if (env.isSkylarkEnabled()) {
         // Only allow native Java calls when using Skylark
         // When calling a Java method, the name is not in the Environment,
@@ -496,7 +498,8 @@ public final class FuncallExpression extends Expression {
                   func.getName(), objValue, EvalUtils.getDataTypeName(objValue)));
         }
         if (objValue instanceof Class<?>) {
-          // Static Java method call
+          // Static Java method call. We can return the value from here directly because
+          // invokeJavaMethod() has special checks.
           return invokeJavaMethod(null, (Class<?>) objValue, func.getName(), posargs.build());
         } else {
           return invokeJavaMethod(objValue, objValue.getClass(), func.getName(), posargs.build());
@@ -509,16 +512,24 @@ public final class FuncallExpression extends Expression {
     } else { // func(...)
       Object funcValue = func.eval(env);
       if ((funcValue instanceof Function)) {
-        Function function = (Function) funcValue;
+        function = (Function) funcValue;
         evalArguments(posargs, kwargs, env, function);
-        return EvalUtils.checkNotNull(this,
-            function.call(posargs.build(), ImmutableMap.<String, Object>copyOf(kwargs), this, env));
+        returnValue = function.call(
+            posargs.build(), ImmutableMap.<String, Object>copyOf(kwargs), this, env);
       } else {
         throw new EvalException(getLocation(),
             "'" + EvalUtils.getDataTypeName(funcValue)
             + "' object is not callable");
       }
     }
+
+    EvalUtils.checkNotNull(this, returnValue);
+    if (!env.isSkylarkEnabled()) {
+      // The call happens in the BUILD language. Note that accessing "BUILD language" functions in
+      // Skylark should never happen.
+      return SkylarkType.convertFromSkylark(returnValue);
+    }
+    return returnValue;
   }
 
   private ArgConversion getArgConversion(Function function) {
-- 
cgit v1.2.3