Make lputs use psl instead of expecting it to be null-terminated.
This allows us to enable the commandl1 and commandl2 regress tests.

Original analysis from seL4 <at> disroot <dot> org
OK millert@

Make sed's -i flag more compatible with what gsed does.
- Reset the hold-space in between files
- quit the editor as soon as a 'q' command is found
- Make sure the temp-file is written back to the original file if we quit
the editor

temp-file not written back issue found by Time Chase.
Lots of feedback from millert@ and schwarze@
OK millert@

Fix sign compare warnings; OK martijn@

Remove unused-with-dead-store variable oldpsanl.

From Daniel Cegielka - thanks

ok jca@

Nuke more whitespace caught in the headlights of "warning:"
rectification.

Split error() into error() and warning() so error() can be marked __dead to
appease gcc.

ok procter@ deraadt@

Fix an off by one error when no matches were found in a substitute.
By pfg@ from FreeBSD

OK millert@ and otto@

Make sed use the new REG_STARTEND | REG_NOTBOL combination. This fixes a begin
of word mismatch as reported by jsg@.

Discussed with otto@ and others early on, earlier version tested in ports build
by aja@

OK millert@
Few readability tweaks and OK schwarze@

Rename err() to error() to prevent confusion with the stdlib function.

Discussed with jca@.

fix eyesore whitespace

figure out the linewidth at initialization, rather than late
ok tedu miod

add -i flag to sed to do in-place editing; mostly based on freebsd

feedback/ok deraadt@ millert@

Convert many atoi() calls to strtonum(), adding range checks and failure
handling along the way.
Reviews by Brendan MacDonell, Jeremy Devenport, florian, doug, millert

correct multiplication idiom during xreallocarray, and expand appendnum
to size_t to avoid overflow after allocation success
ok guenther doug

Rework the pointer swap in the 'P' command to make the intent
clearer and avoid a crash on 'g;P' found by Sebastien Marie with
the afl fuzzer.

sed has a xreallocarray(), but two n*s cases were not replaced

unsigned char for ctype
ok krw okan

Rewrite the main loop of the "sed s/..." command, to fix multiple
issues regarding the replacement of zero-length strings.

This commit brings back rev. 1.16, but without the regression that
forced the backout: No NUL bytes will be output now, not even when
the input file lacks a trailing newline character and there is a
zero-length match at the end.

OK otto@ deraadt@;
and naddy@ (who originally found the regression) checked that
the regression is indeed fixed.

Backout previous, naddy@ found the following regression:
When the input does not end in a trailing newline character
and there is an empty match at the end, the new code adds
a spurious '\0' character.
I have a fix, but otto@ prefers backout and full re-evaluation
after release.

Rewrite the main loop of the "sed s/..." command, shortening it by ten
lines and simplifying it by removing the switch statement implementing
/g, /1, and /2 separately and repetitively.
The idea to make the loop control variable slen, i.e. the length of the
string remaining to be processed, signed, and stay in the loop even when
slen == 0 (i.e. at the end of the string), lifted from FreeBSD by otto@.
On i386, process.o shrinks by 440 bytes, and the sed binary by 23 bytes.

This fixes multiple aspects of the replacement of multiple (/g) or
specific (e.g. /2) instances of zero-length matches, both with BREs and
EREs, both with and without a trailing newline character on the input.

Feedback and OK otto@.

rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and
unmaintainable). these days, people use source. these id's do not provide
any benefit, and do hurt the small install media
(the 33,000 line diff is essentially mechanical)
ok with the idea millert, ok dms

The 'l' command should not encode newlines. Also document that
backspace is escaped as \b. OK phessler@ and jmc@

pretty. the comma operator should not be used to cram two statements into
an if without braces, or worse, into a return.

prevent overflow, see freebsd r1.32. also, do nicer reallocs.
ok millert@ otto@

mostly ansi cleanup; pval ok

Remove the advertising clause in the UCB license which Berkeley
rescinded 22 July 1999. Proofed by myself and Theo.

kill err(3) newlines; ok miod@, deraadt@

Fix sed(1) in the case where a last line is specified and hold space
is not specified, and then the first part of the pattern space is
deleted, when there are two or more input lines, as this results
in subtraction of one from an unsigned integral value of '0'. That
bogus value is used in one case for a loop (that will run far too
many times in this case) and a function to search for a value within
a specified range of memory, however now the range of memory is
obscenely large and a segmentation (or memory) fault will occur.
This is fixed by checking for and appropriately handling a nil
pattern space as if the specified search in memory failed, as indeed
it obviously will with nil pattern space.

From Tim J. Robbins by way of FreeBSD

Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.

kill more registers

millert@ ok

make G newline if hold is empty

make y command 8 bit clean; Elmar.Bartel@informatik.tu-muenchen.de

extra ()

rcsid

branches: 1.1.1;
Initial revision

Make sed's -i flag more compatible with what gsed does.
- Reset the hold-space in between files
- quit the editor as soon as a 'q' command is found
- Make sure the temp-file is written back to the original file if we quit
the editor

temp-file not written back issue found by Time Chase.
Lots of feedback from millert@ and schwarze@
OK millert@

Fix sign compare warnings; OK martijn@

Remove unused-with-dead-store variable oldpsanl.

From Daniel Cegielka - thanks

ok jca@

Nuke more whitespace caught in the headlights of "warning:"
rectification.

Split error() into error() and warning() so error() can be marked __dead to
appease gcc.

ok procter@ deraadt@

Fix an off by one error when no matches were found in a substitute.
By pfg@ from FreeBSD

OK millert@ and otto@

Make sed use the new REG_STARTEND | REG_NOTBOL combination. This fixes a begin
of word mismatch as reported by jsg@.

Discussed with otto@ and others early on, earlier version tested in ports build
by aja@

OK millert@
Few readability tweaks and OK schwarze@

Rename err() to error() to prevent confusion with the stdlib function.

Discussed with jca@.

fix eyesore whitespace

figure out the linewidth at initialization, rather than late
ok tedu miod

add -i flag to sed to do in-place editing; mostly based on freebsd

feedback/ok deraadt@ millert@

Convert many atoi() calls to strtonum(), adding range checks and failure
handling along the way.
Reviews by Brendan MacDonell, Jeremy Devenport, florian, doug, millert

correct multiplication idiom during xreallocarray, and expand appendnum
to size_t to avoid overflow after allocation success
ok guenther doug

Rework the pointer swap in the 'P' command to make the intent
clearer and avoid a crash on 'g;P' found by Sebastien Marie with
the afl fuzzer.

sed has a xreallocarray(), but two n*s cases were not replaced

unsigned char for ctype
ok krw okan

Rewrite the main loop of the "sed s/..." command, to fix multiple
issues regarding the replacement of zero-length strings.

This commit brings back rev. 1.16, but without the regression that
forced the backout: No NUL bytes will be output now, not even when
the input file lacks a trailing newline character and there is a
zero-length match at the end.

OK otto@ deraadt@;
and naddy@ (who originally found the regression) checked that
the regression is indeed fixed.

Backout previous, naddy@ found the following regression:
When the input does not end in a trailing newline character
and there is an empty match at the end, the new code adds
a spurious '\0' character.
I have a fix, but otto@ prefers backout and full re-evaluation
after release.

Rewrite the main loop of the "sed s/..." command, shortening it by ten
lines and simplifying it by removing the switch statement implementing
/g, /1, and /2 separately and repetitively.
The idea to make the loop control variable slen, i.e. the length of the
string remaining to be processed, signed, and stay in the loop even when
slen == 0 (i.e. at the end of the string), lifted from FreeBSD by otto@.
On i386, process.o shrinks by 440 bytes, and the sed binary by 23 bytes.

This fixes multiple aspects of the replacement of multiple (/g) or
specific (e.g. /2) instances of zero-length matches, both with BREs and
EREs, both with and without a trailing newline character on the input.

Feedback and OK otto@.

rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and
unmaintainable). these days, people use source. these id's do not provide
any benefit, and do hurt the small install media
(the 33,000 line diff is essentially mechanical)
ok with the idea millert, ok dms

The 'l' command should not encode newlines. Also document that
backspace is escaped as \b. OK phessler@ and jmc@

pretty. the comma operator should not be used to cram two statements into
an if without braces, or worse, into a return.

prevent overflow, see freebsd r1.32. also, do nicer reallocs.
ok millert@ otto@

mostly ansi cleanup; pval ok

Remove the advertising clause in the UCB license which Berkeley
rescinded 22 July 1999. Proofed by myself and Theo.

kill err(3) newlines; ok miod@, deraadt@

Fix sed(1) in the case where a last line is specified and hold space
is not specified, and then the first part of the pattern space is
deleted, when there are two or more input lines, as this results
in subtraction of one from an unsigned integral value of '0'. That
bogus value is used in one case for a loop (that will run far too
many times in this case) and a function to search for a value within
a specified range of memory, however now the range of memory is
obscenely large and a segmentation (or memory) fault will occur.
This is fixed by checking for and appropriately handling a nil
pattern space as if the specified search in memory failed, as indeed
it obviously will with nil pattern space.

From Tim J. Robbins by way of FreeBSD

Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.

kill more registers

millert@ ok

make G newline if hold is empty

make y command 8 bit clean; Elmar.Bartel@informatik.tu-muenchen.de

extra ()

rcsid

branches: 1.1.1;
Initial revision

Fix sign compare warnings; OK martijn@

Remove unused-with-dead-store variable oldpsanl.

From Daniel Cegielka - thanks

ok jca@

Nuke more whitespace caught in the headlights of "warning:"
rectification.

Split error() into error() and warning() so error() can be marked __dead to
appease gcc.

ok procter@ deraadt@

Fix an off by one error when no matches were found in a substitute.
By pfg@ from FreeBSD

OK millert@ and otto@

Make sed use the new REG_STARTEND | REG_NOTBOL combination. This fixes a begin
of word mismatch as reported by jsg@.

Discussed with otto@ and others early on, earlier version tested in ports build
by aja@

OK millert@
Few readability tweaks and OK schwarze@

Rename err() to error() to prevent confusion with the stdlib function.

Discussed with jca@.

fix eyesore whitespace

figure out the linewidth at initialization, rather than late
ok tedu miod

add -i flag to sed to do in-place editing; mostly based on freebsd

feedback/ok deraadt@ millert@

Convert many atoi() calls to strtonum(), adding range checks and failure
handling along the way.
Reviews by Brendan MacDonell, Jeremy Devenport, florian, doug, millert

correct multiplication idiom during xreallocarray, and expand appendnum
to size_t to avoid overflow after allocation success
ok guenther doug

Rework the pointer swap in the 'P' command to make the intent
clearer and avoid a crash on 'g;P' found by Sebastien Marie with
the afl fuzzer.

sed has a xreallocarray(), but two n*s cases were not replaced

unsigned char for ctype
ok krw okan

Rewrite the main loop of the "sed s/..." command, to fix multiple
issues regarding the replacement of zero-length strings.

This commit brings back rev. 1.16, but without the regression that
forced the backout: No NUL bytes will be output now, not even when
the input file lacks a trailing newline character and there is a
zero-length match at the end.

OK otto@ deraadt@;
and naddy@ (who originally found the regression) checked that
the regression is indeed fixed.

Backout previous, naddy@ found the following regression:
When the input does not end in a trailing newline character
and there is an empty match at the end, the new code adds
a spurious '\0' character.
I have a fix, but otto@ prefers backout and full re-evaluation
after release.

Rewrite the main loop of the "sed s/..." command, shortening it by ten
lines and simplifying it by removing the switch statement implementing
/g, /1, and /2 separately and repetitively.
The idea to make the loop control variable slen, i.e. the length of the
string remaining to be processed, signed, and stay in the loop even when
slen == 0 (i.e. at the end of the string), lifted from FreeBSD by otto@.
On i386, process.o shrinks by 440 bytes, and the sed binary by 23 bytes.

This fixes multiple aspects of the replacement of multiple (/g) or
specific (e.g. /2) instances of zero-length matches, both with BREs and
EREs, both with and without a trailing newline character on the input.

Feedback and OK otto@.

rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and
unmaintainable). these days, people use source. these id's do not provide
any benefit, and do hurt the small install media
(the 33,000 line diff is essentially mechanical)
ok with the idea millert, ok dms

The 'l' command should not encode newlines. Also document that
backspace is escaped as \b. OK phessler@ and jmc@

pretty. the comma operator should not be used to cram two statements into
an if without braces, or worse, into a return.

prevent overflow, see freebsd r1.32. also, do nicer reallocs.
ok millert@ otto@

mostly ansi cleanup; pval ok

Remove the advertising clause in the UCB license which Berkeley
rescinded 22 July 1999. Proofed by myself and Theo.

kill err(3) newlines; ok miod@, deraadt@

Fix sed(1) in the case where a last line is specified and hold space
is not specified, and then the first part of the pattern space is
deleted, when there are two or more input lines, as this results
in subtraction of one from an unsigned integral value of '0'. That
bogus value is used in one case for a loop (that will run far too
many times in this case) and a function to search for a value within
a specified range of memory, however now the range of memory is
obscenely large and a segmentation (or memory) fault will occur.
This is fixed by checking for and appropriately handling a nil
pattern space as if the specified search in memory failed, as indeed
it obviously will with nil pattern space.

From Tim J. Robbins by way of FreeBSD

Part one of userland __P removal. Done with a simple regexp with some minor hand editing to make comments line up correctly. Another pass is forthcoming that handles the cases that could not be done automatically.

kill more registers

millert@ ok

make G newline if hold is empty

make y command 8 bit clean; Elmar.Bartel@informatik.tu-muenchen.de

extra ()

rcsid

branches: 1.1.1;
Initial revision