diff options
-rw-r--r-- | ChangeLog | 13 | ||||
-rw-r--r-- | eval.c | 23 | ||||
-rw-r--r-- | match.c | 7 |
3 files changed, 34 insertions, 9 deletions
@@ -1,5 +1,18 @@ 2014-02-11 Kaz Kylheku <kaz@kylheku.com> + * eval.c (subst_vars): Bugfix: results of expressions not + treated in the same way as variables: lists not stringified, + causing expansions with parentheses, and sometimes errors + due to unhandled objects. Also, use tostringp instead of + format for stringifying objects. + stringifying object. Bugfix.k + + * match.c (subst_vars): Added comment similar to the one in + the subst_vars of eval.c. Removed superfluous conversion code where the + str variable is already known to be a string. + +2014-02-11 Kaz Kylheku <kaz@kylheku.com> + * eval.c (apply): Fix regression in dwim operator: not handling nil object properly. Since dwim now trivially delegates to apply, apply must recognize nil. @@ -1280,17 +1280,24 @@ static val subst_vars(val forms, val env) val modifiers = fourth(form); val str = eval(expr, env, form); + /* If the object is a list, we let format_field deal with the + conversion to text, because the modifiers influence how + it is done. */ if (!stringp(str) && !listp(str)) - str = format(nil, lit("~a"), str, nao); + str = tostringp(str); - if (pat) + if (pat) { forms = cons(str, cons(pat, rest(forms))); - else if (modifiers) + } else if (modifiers) { forms = cons(format_field(str, modifiers, nil, curry_123_1(func_n3(eval), env, form)), rest(forms)); - else + } else { + if (listp(str)) + str = cat_str(mapcar(func_n1(tostringp), str), lit(" ")); forms = cons(str, rest(forms)); + } + continue; } else if (sym == quasi_s) { val nested = subst_vars(rest(form), env); @@ -1298,8 +1305,12 @@ static val subst_vars(val forms, val env) forms = cdr(forms); continue; } else if (sym == expr_s) { - val result = eval(rest(form), env, form); - forms = cons(format(nil, lit("~a"), result, nao), rest(forms)); + val str = eval(rest(form), env, form); + if (listp(str)) + str = cat_str(mapcar(func_n1(tostringp), str), lit(" ")); + else if (!stringp(str)) + str = tostringp(str); + forms = cons(str, rest(forms)); continue; } else { val nested = subst_vars(form, env); @@ -1359,8 +1359,11 @@ static val subst_vars(val spec, val bindings, val filter) val modifiers = fourth(elem); val str = txeval(spec, expr, bindings); + /* If the object is a list, we let format_field deal with the + conversion to text, because the modifiers influence how + it is done. */ if (!stringp(str) && !listp(str)) - str = format(nil, lit("~a"), str, nao); + str = tostringp(str); if (pat) spec = cons(pat, rest(spec)); @@ -1372,8 +1375,6 @@ static val subst_vars(val spec, val bindings, val filter) } else { if (listp(str)) str = cat_str(mapcar(func_n1(tostringp), str), lit(" ")); - else - str = if3(stringp(str), str, tostringp(str)); spec = cons(filter_string_tree(filter, str), rest(spec)); } |