Correct a rather obvious typo (once Tor Egge pointed it out to me)
in the last change.

PR/7231: Havard Eidnes: Shell quoting/trimming problem

Fix off-by-one error in the starting point to search for an arithmetic
expression.

PR/5577: Craig M. Chase: sh does not build with PARALLEL set.
- Added YHEADER in Makefile, removed arith.h and adjusted the sources.

Patches from Tor Egge (via Havard Eidnes) to fix various bugs in field
splitting and combining.
(Note: Some of this are not strictly bugs, but differences between traditional
Bourne shell and POSIX.)

Be more retentive about use of NOTREACHED and noreturn.

const poisoning.

- change "extern" variables into int's
- remove extern'd variables not actually referenced
- don't use char as an array index

Fix the VSTRIMRIGHT* bugs... The problem was not the string length computation,
but lack of '\0' termination. Factor this segment out as common code too, while
I am there.

off by one error in ${%%}

Previous fix broke $var quoting. Try again differently :-)

Fix bug introduced by EXP_RECORD, where in case there was a variable expansion
involved in the `for' list, the list was recorded twice, leading to incorrect
argument expansion.
Introduce ifsfree() fu

Fix bug introduced by EXP_RECORD, where in case there was a variable expansion
involved in the `for' list, the list was recorded twice, leading to incorrect
argument expansion.
Introduce ifsfree() function that free's the IFS region list, GC'ing duplicated
code.

show more ...

PR/4851: Benjamin Lorenz: In the "for <var> in <args>" construct <args>
was not marked as a region to be handled by ifsbreakup. Add EXP_RECORD
to indicate that the argument string needs to be recorde

PR/4851: Benjamin Lorenz: In the "for <var> in <args>" construct <args>
was not marked as a region to be handled by ifsbreakup. Add EXP_RECORD
to indicate that the argument string needs to be recorded.

show more ...

Unfortunately (as I expected) the previous change broke:

sleep
cmd='set `type "sleep"`; eval echo \$$#'
which=`eval $cmd`
echo $which

because the region did not get recorded at all,

Unfortunately (as I expected) the previous change broke:

sleep
cmd='set `type "sleep"`; eval echo \$$#'
which=`eval $cmd`
echo $which

because the region did not get recorded at all, and it was interpreted as
a single word. I modified the code to keep track when the result of a
backquote expansion has been recorded to avoid recording it twice. I still
feel that this is not the right fix... More to come.

show more ...

PR/4547: Joern Clausen: Incorrect argument expansion in backquote variable
assignment. E.g.
echo ${foo:=`echo 1 2 3 4`}
prints:
1 2 3 1 2 3 4
because when the arquments are not quoted, the backquot

PR/4547: Joern Clausen: Incorrect argument expansion in backquote variable
assignment. E.g.
echo ${foo:=`echo 1 2 3 4`}
prints:
1 2 3 1 2 3 4
because when the arquments are not quoted, the backquote result
gets recorded twice. The fix right now is to comment out the
record_region() call in expbackq(). I hope that it does not break
anything else.

show more ...

Make code agree with man page in processing expansion of "$*".
Fix from PR 2647.

Fix compiler warnings.

PR/3352: From Hiroyuki Ito: ${#1} was not being expanded properly if there
was a need to allocated another stack block.

varisset fixes:
- treat $0 specially since it is not in shellparams
- check the number of parameters instead of walking
the parameters array to avoid checking against the
null terminated elem

varisset fixes:
- treat $0 specially since it is not in shellparams
- check the number of parameters instead of walking
the parameters array to avoid checking against the
null terminated element.

show more ...

PR/3269: Off by one in varisset(), caused variable substitution not to
count the last positional parameter as set.

- varisset(): In positional arguments, take into account VSNUL so that:
set -- ""; echo ${1:-wwww} works.
- when expanding arithmetic, discard previous ifs recorded regions, since we
are doing ou

- varisset(): In positional arguments, take into account VSNUL so that:
set -- ""; echo ${1:-wwww} works.
- when expanding arithmetic, discard previous ifs recorded regions, since we
are doing our own scanning. x=ab; echo $((${#x}+1)) now works.
- in ${var#word} fix two bugs:
* if there was an exact match, there was an off-by-one bug in the
comparison of the words. x=abcd; echo ${x#abcd}
* if there was no match, the stack region was not adjusted and the rest
of the word was getting written in the wrong place. x=123; echo ${x#abc}X

show more ...

kill 'register'

A correction to the previous patch from Todd Miller.

echo ${1:-empty} did not do the substitution; from Todd Miller (OpenBSD)

PR/2808: Fix parsing of $n where n > 9 (from FreeBSD)