[arith] Make ? a right associative operator.

Fixes bug #712.
oils-for-unix · Apr 17, 2020 · d7834ef · d7834ef
1 parent d688d3f
commit d7834ef
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/osh/arith_parse.py b/osh/arith_parse.py
@@ -160,7 +160,7 @@ def MakeShellSpec():
     spec.Left(9, tdop.LeftBinaryOp, [Id.Arith_DAmp])
     spec.Left(7, tdop.LeftBinaryOp, [Id.Arith_DPipe])
 
-    spec.Left(5, LeftTernary, [Id.Arith_QMark])
+    spec.LeftRightAssoc(5, LeftTernary, [Id.Arith_QMark])
 
     # Right associative: a = b = 2 is a = (b = 2)
     spec.LeftRightAssoc(3, tdop.LeftAssign, [

diff --git a/spec/arith.test.sh b/spec/arith.test.sh
@@ -571,3 +571,10 @@ echo $(( a2 ))
 0
 0
 ## END
+
+#### nested ternary (bug fix)
+echo $((1?2?3:4:5))
+## STDOUT:
+3
+## END
+