commit 1159cc2a43c4f6416c11e13b115fc26c63e769c5
author Laurent Le Brun <laurentlb@google.com> Wed Jan 04 12:27:39 2017 +0000
committer John Cater <jcater@google.com> Wed Jan 04 20:38:06 2017 +0000
tree 828f6be988194970ff6de5c4831b2573d4fdf748
parent 9340550de931d952055f3d7b95fd09b96b5bed6c

Minor improvements to error messages.

In case of conflict, show the location of the original rule.

--
PiperOrigin-RevId: 143541281
MOS_MIGRATED_REVID=143541281

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 e3468da..dfbf43a 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
@@ -580,12 +580,13 @@
     if (!(items instanceof Map<?, ?>)) {
       throw new EvalException(
           location,
-          "argument after ** must be a dictionary, not " + EvalUtils.getDataTypeName(items));
+          "argument after ** must be a dictionary, not '" + EvalUtils.getDataTypeName(items) + "'");
     }
     for (Map.Entry<?, ?> entry : ((Map<?, ?>) items).entrySet()) {
       if (!(entry.getKey() instanceof String)) {
         throw new EvalException(
-            location, "keywords must be strings, not " + EvalUtils.getDataTypeName(entry.getKey()));
+            location,
+            "keywords must be strings, not '" + EvalUtils.getDataTypeName(entry.getKey()) + "'");
       }
       addKeywordArg(kwargs, (String) entry.getKey(), entry.getValue(), duplicates);
     }