busybox/shell/ash_test/ash-arith/arith-ternary_nested1.right at master - mirrors/busybox - FSKY Forge

mirrors/busybox

mirror of https://git.busybox.net/busybox synced 2025-05-10 04:33:59 +02:00

Denys Vlasenko 5f56a03882 shell/math: fix parsing of ?: and explain why it's parsed that way

This fixes arith-precedence1.tests.

This breaks arith-ternary2.tests again (we now evaluate variables
on not-taken branches). We need a better logic here anyway:
not only bare variables should not evaluate when not-taken:
	1 ? eval_me : do_not_eval
but any (arbitrarily complex) expressions shouldn't
evaluate as well!
	1 ? var_is_set=1 : ((var_is_not_set=2,var2*=4))

function                                             old     new   delta
evaluate_string                                     1097    1148     +51

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

2023-06-15 10:14:43 +02:00

1 line

4 B

Text

Raw Permalink Blame History

3:3