Took care of a stupid oversight in the expression code. This should fix #13

milochristiansen · milochristiansen · commit da1d0a6492fc · 2018-05-05T12:35:38.000-04:00
diff --git a/compile_expr.go b/compile_expr.go
@@ -259,8 +259,12 @@ func lowerIdentHelper(n *ast.TableAccessor, state *compState, data *identData) {
 		}
 	case *ast.Parens:
 		expr(nObj.Inner, state, data.reg, false).To(false)
+		rk, _ := expr(n.Key, state, data.reg+1, false).RK()
+		state.addInst(createABC(opGetTable, data.reg, data.reg, rk), n.Key.Line())
 	case *ast.FuncCall:
 		expr(nObj, state, data.reg, false).To(false)
+		rk, _ := expr(n.Key, state, data.reg+1, false).RK()
+		state.addInst(createABC(opGetTable, data.reg, data.reg, rk), n.Key.Line())
 	default:
 		panic("IMPOSSIBLE") // I think?
 	}

Original file line number	Diff line number	Diff line change
`@@ -259,8 +259,12 @@ func lowerIdentHelper(n ast.TableAccessor, state compState, data *identData) {`
`259`	`259`	`}`
`260`	`260`	`case *ast.Parens:`
`261`	`261`	`expr(nObj.Inner, state, data.reg, false).To(false)`
	`262`	`+ rk, _ := expr(n.Key, state, data.reg+1, false).RK()`
	`263`	`+ state.addInst(createABC(opGetTable, data.reg, data.reg, rk), n.Key.Line())`
`262`	`264`	`case *ast.FuncCall:`
`263`	`265`	`expr(nObj, state, data.reg, false).To(false)`
	`266`	`+ rk, _ := expr(n.Key, state, data.reg+1, false).RK()`
	`267`	`+ state.addInst(createABC(opGetTable, data.reg, data.reg, rk), n.Key.Line())`
`264`	`268`	`default:`
`265`	`269`	`panic("IMPOSSIBLE") // I think?`
`266`	`270`	`}`