[errors] Add error location to value.Place

Issue #1864
oilshell · Mar 15, 2024 · 4ef2d38 · 4ef2d38
1 parent 8774abe
commit 4ef2d38
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/frontend/location.py b/frontend/location.py
@@ -477,6 +477,10 @@ def TokenForExpr(node):
             node = cast(expr.Var, UP_node)
             return node.left
 
+        elif case(expr_e.Place):
+            node = cast(expr.Place, UP_node)
+            return node.blame_tok
+
         elif case(expr_e.CommandSub):
             node = cast(CommandSub, UP_node)
             return node.left_token

diff --git a/spec/ysh-builtin-ctx.test.sh b/spec/ysh-builtin-ctx.test.sh
@@ -157,3 +157,12 @@ json write (spec)
   ]
 }
 ## END
+
+#### ctx with value.Place, not List/Dict (error location bug fix)
+
+ctx push (&p) {
+  true
+}
+## status: 3
+## STDOUT:
+## END