forks/busybox/shell/hush_test/hush-vars, branch master

hush: fix fallout from tweaking ${var:START:LEN} code

2020-12-23T11:38:03+00:00

function old new delta expand_one_var 2344 2353 +9 Signed-off-by: Denys Vlasenko

ash: parser: Only accept single-digit parameter expansion outside of braces

2020-02-17T09:15:35+00:00

Upstream commit: Date: Mon, 27 May 2019 13:39:37 +0800 parser: Only accept single-digit parameter expansion outside of braces This patch should fix the problem. Signed-off-by: Herbert Xu Signed-off-by: Denys Vlasenko

hush: remove test for "echo ${-}" errorring out - now it works

2019-05-19T14:35:56+00:00

Signed-off-by: Denys Vlasenko

hush: fix quoted "${notexist-}" expansion to not disappear

2019-05-15T11:39:19+00:00

function old new delta expand_one_var 2296 2311 +15 Signed-off-by: Denys Vlasenko

ash: expand: Fix ghost fields with unquoted $@/$*

2018-08-05T12:29:58+00:00

Upstream commit: Date: Fri, 23 Mar 2018 18:58:47 +0800 expand: Fix ghost fields with unquoted $@/$* You're right. The proper fix to this is to ensure that nulonly is not set in varvalue for $*. It should only be set for $@ when it's inside double quotes. In fact there is another bug while we're playing with $@/$*. When IFS is set to a non-whitespace character such as :, $* outside quotes won't remove empty fields as it should. This patch fixes both problems. Reported-by: Martijn Dekker Suggested-by: Harald van Dijk Signed-off-by: Herbert Xu function old new delta argstr 1111 1113 +2 evalvar 571 569 -2 varvalue 579 576 -3 ------------------------------------------------------------------------------ (add/remove: 0/0 grow/shrink: 1/2 up/down: 2/-5) Total: -3 bytes Signed-off-by: Denys Vlasenko

ash,hush: properly handle ${v//pattern/repl} if pattern starts with /

2018-08-04T20:25:28+00:00

Closes 2695 function old new delta parse_dollar 762 790 +28 subevalvar 1258 1267 +9 ------------------------------------------------------------------------------ (add/remove: 0/0 grow/shrink: 2/0 up/down: 37/0) Total: 37 bytes Signed-off-by: Denys Vlasenko

hush: IFS fixes

2018-04-11T14:02:58+00:00

$ IFS=": "; x=" "; set x $x; for v; do echo "|$v|"; done |x| $ IFS=": "; x=":"; set x $x; for v; do echo "|$v|"; done |x| || function old new delta run_pipe 1789 1870 +81 expand_on_ifs 310 361 +51 pseudo_exec_argv 588 591 +3 builtin_local 50 53 +3 ------------------------------------------------------------------------------ (add/remove: 0/0 grow/shrink: 4/0 up/down: 138/0) Total: 138 bytes Signed-off-by: Denys Vlasenko

ash: parser: Allow newlines within parameter substitution

2018-04-11T10:39:18+00:00

Upstream commit: Date: Thu, 22 Mar 2018 21:41:24 +0800 parser: Allow newlines within parameter substitution On Fri, Mar 16, 2018 at 11:27:22AM +0800, Herbert Xu wrote: > On Thu, Mar 15, 2018 at 10:49:15PM +0100, Harald van Dijk wrote: > > > > Okay, it can be trivially modified to something that does work in other > > shells (even if it were actually executed), but gets rejected at parse time > > by dash: > > > > if false; then > > : ${$+ > > } > > fi > > That's just a bug in dash's parser with ${} in general, because > it bombs out without the if clause too: > > : ${$+ > } This patch fixes the parsing of newlines with parameter substitution. Signed-off-by: Herbert Xu Signed-off-by: Denys Vlasenko

hush: fix a bug where we don't properly handle f() { a=A; b=B; }; a= f

2018-04-04T22:51:55+00:00

function old new delta unset_local_var 20 274 +254 leave_var_nest_level - 98 +98 set_vars_and_save_old 128 164 +36 enter_var_nest_level - 32 +32 builtin_local 46 50 +4 pseudo_exec_argv 554 544 -10 redirect_and_varexp_helper 77 64 -13 run_pipe 1890 1641 -249 unset_local_var_len 267 - -267 ------------------------------------------------------------------------------ (add/remove: 2/1 grow/shrink: 3/3 up/down: 424/-539) Total: -115 bytes Signed-off-by: Denys Vlasenko

hush: fix for readonly vars in "ro=A ro=B cmd" case

2018-04-04T19:14:28+00:00

Signed-off-by: Denys Vlasenko