Re: [PATCH] don't record empty IFS scan regions

[Martijn Dekker <martijn@xxxxxxxx>]

Op 22-03-18 om 10:35 schreef Herbert Xu:
> Martijn Dekker <martijn@xxxxxxxx> wrote:
>>
>> evalvar() records empty expansion results (varlen == 0) as string
>> regions that need to be scanned for IFS characters. This is pointless,
>> because there is nothing to split.
>>
>> This patch fixes the bug that, given no positional parameters, unquoted
>> $@ and $* incorrectly generate one empty field (they should generate no
>> fields). Apparently that was a side effect of the above.
> 
> Please attach a reproducer as I cannot reproduce this problem.

Sorry, I forgot to mention it only occurs with set & empty IFS.

It came up earlier in the thread about Harald van Dijk's patch for a
recursive parser.

$ dash-0.5.9.1 -c 'IFS=; set --; set -- $@ $*; echo $#'
2

(expected output: 0)

I'll resend the patch with this reproducer.

- M.
--
To unsubscribe from this list: send the line "unsubscribe dash" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html