lang: ast: Expr Param and Poly should not have values

Sometimes a recursive call through ExprVar's Value method would hit one
of these and return (nil, nil) which would throw off things.
This commit is contained in:
James Shubin
2023-12-17 21:16:03 -05:00
parent 6a6546db8d
commit c333cb542c
2 changed files with 14 additions and 2 deletions

View File

@@ -8682,7 +8682,7 @@ func (obj *ExprParam) SetValue(value types.Value) error {
// usually only be valid once the engine has run and values have been produced. // usually only be valid once the engine has run and values have been produced.
// This might get called speculatively (early) during unification to learn more. // This might get called speculatively (early) during unification to learn more.
func (obj *ExprParam) Value() (types.Value, error) { func (obj *ExprParam) Value() (types.Value, error) {
return nil, nil return nil, fmt.Errorf("no value for ExprParam")
} }
// ExprPoly is a polymorphic expression that is a definition that can be used in // ExprPoly is a polymorphic expression that is a definition that can be used in
@@ -8799,7 +8799,7 @@ func (obj *ExprPoly) SetValue(value types.Value) error {
// usually only be valid once the engine has run and values have been produced. // usually only be valid once the engine has run and values have been produced.
// This might get called speculatively (early) during unification to learn more. // This might get called speculatively (early) during unification to learn more.
func (obj *ExprPoly) Value() (types.Value, error) { func (obj *ExprPoly) Value() (types.Value, error) {
return nil, nil return nil, fmt.Errorf("no value for ExprPoly")
} }
// ExprIf represents an if expression which *must* have both branches, and which // ExprIf represents an if expression which *must* have both branches, and which

View File

@@ -0,0 +1,12 @@
-- main.mcl --
import "fmt"
$format = "%d" # should get ignored
$fn = func($format) {
fmt.printf($format, 42)
}
test $fn("%s") {}
# should error at unification if possible, otherwise at runtime
# TODO: I would expect that if the "%s" and "%d" swapped, that speculatively we
# would be able to run this at compile time and know the result statically.
-- OUTPUT --
# err: errStream: func `printf@??????????` stopped before it was loaded: base kind does not match (Str != Int)