Re: [PATCH v2] expand: Fix ghost fields with unquoted $@/$*

2018-03-23 Thread Martijn Dekker
Op 23-03-18 om 11:58 schreef Herbert Xu: > Thanks, the problem here is that we need to set c to 0 not just > when quoted is true but also if sep is 0 since both imply that > field splitting is disabled. Here is an second revision which > should fix this by checking (quoted || !sep) instead of just

[PATCH v2] expand: Fix ghost fields with unquoted $@/$*

2018-03-23 Thread Herbert Xu
On Fri, Mar 23, 2018 at 10:48:13AM +0100, Martijn Dekker wrote: > > Unfortunately it also introduces a bug with $*. > > $ src/dash -c 'IFS=:; v=$*; printf "[%s]\n" "$v"' _ abc 'def ghi' jkl > [abcdef ghijkl] > > Expected: > [abc:def ghi:jkl] Thanks, the problem here is that we need to set c to 0