adjust a variable name in a comment after the reorg in signal.c rev. 1.18

When system calls indicate an error they return -1, not some arbitrary
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if this strictness
helps us in the future.

style(9) includes

ok nicm@

Add a flag argument to flush() to stop it calling quit() on error, then
use this from quit() to stop less blowing up the stack looping through
quit()/flush() if stderr is closed (for example "less /missing
2</dev/null"). ok millert

NULL_IFILE has no weird use patterns. NULL is sufficient.

error() is not a stdarg function, so there is no reason for NULL_PARG
to be special. Simply use NULL.

do not need wrappers around O_RDONLY and such

replace save() with estrdup() and make estrdup() exit like all the other
ecalloc, etc. functions do.
ok mmcc nicm

Garrett D'Amore has agreed to moving his copyright line up above the
terms with the existing copyright, so it is clear it applies the same
terms.

Remove LSIGNAL which is just the same as lsignal() now.

Replace less with the cleaned-up fork of less 458 maintained by Garrett
D'Amore at https://github.com/gdamore/less-fork. This has significantly
less portability goop, has a tidied up code style, uses terminfo instead
of termcap, and is has stricter POSIX compliance.

Many of our local changes have been accepted upstream: substantial
remaining local changes are code to read help files from /usr/share
rather than compiling them in, man page and help improvements, and some
tweaks to the default options.

Review and testing by millert, ok deraadt

Merge Less 458 plus local patches

ok guenther@, ok jmc@ for man bits

Merge in less 444 plus local changes

ok nicm@

Update to less version 382. This removes some code that is licensed under
a non-BSD license and which was not used on OpenBSD. No functional changes.

From Daniel Dickmann <didickman@gmail.com>, ok itojun, millert, ray, matthieu

use strchr instead of index

Merge in less-381 w/ local changes and remove obsolete files.

more strlcpy/strlcat/snprintf, less strcpy/strcat/sprintf
reviewed by tdeval, millert, dhartmei and others. more bits coming here
from some of them i think

kill more registers

millert@ ok

$OpenBSD$

branches: 1.1.1;
Initial revision

When system calls indicate an error they return -1, not some arbitrary
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if this strictness
helps us in the future.

style(9) includes

ok nicm@

Add a flag argument to flush() to stop it calling quit() on error, then
use this from quit() to stop less blowing up the stack looping through
quit()/flush() if stderr is closed (for example "less /missing
2</dev/null"). ok millert

NULL_IFILE has no weird use patterns. NULL is sufficient.

error() is not a stdarg function, so there is no reason for NULL_PARG
to be special. Simply use NULL.

do not need wrappers around O_RDONLY and such

replace save() with estrdup() and make estrdup() exit like all the other
ecalloc, etc. functions do.
ok mmcc nicm

Garrett D'Amore has agreed to moving his copyright line up above the
terms with the existing copyright, so it is clear it applies the same
terms.

Remove LSIGNAL which is just the same as lsignal() now.

Replace less with the cleaned-up fork of less 458 maintained by Garrett
D'Amore at https://github.com/gdamore/less-fork. This has significantly
less portability goop, has a tidied up code style, uses terminfo instead
of termcap, and is has stricter POSIX compliance.

Many of our local changes have been accepted upstream: substantial
remaining local changes are code to read help files from /usr/share
rather than compiling them in, man page and help improvements, and some
tweaks to the default options.

Review and testing by millert, ok deraadt

Merge Less 458 plus local patches

ok guenther@, ok jmc@ for man bits

Merge in less 444 plus local changes

ok nicm@

Update to less version 382. This removes some code that is licensed under
a non-BSD license and which was not used on OpenBSD. No functional changes.

From Daniel Dickmann <didickman@gmail.com>, ok itojun, millert, ray, matthieu

use strchr instead of index

Merge in less-381 w/ local changes and remove obsolete files.

more strlcpy/strlcat/snprintf, less strcpy/strcat/sprintf
reviewed by tdeval, millert, dhartmei and others. more bits coming here
from some of them i think

kill more registers

millert@ ok

$OpenBSD$

branches: 1.1.1;
Initial revision

style(9) includes

ok nicm@

Add a flag argument to flush() to stop it calling quit() on error, then
use this from quit() to stop less blowing up the stack looping through
quit()/flush() if stderr is closed (for example "less /missing
2</dev/null"). ok millert

NULL_IFILE has no weird use patterns. NULL is sufficient.

error() is not a stdarg function, so there is no reason for NULL_PARG
to be special. Simply use NULL.

do not need wrappers around O_RDONLY and such

replace save() with estrdup() and make estrdup() exit like all the other
ecalloc, etc. functions do.
ok mmcc nicm

Garrett D'Amore has agreed to moving his copyright line up above the
terms with the existing copyright, so it is clear it applies the same
terms.

Remove LSIGNAL which is just the same as lsignal() now.

Replace less with the cleaned-up fork of less 458 maintained by Garrett
D'Amore at https://github.com/gdamore/less-fork. This has significantly
less portability goop, has a tidied up code style, uses terminfo instead
of termcap, and is has stricter POSIX compliance.

Many of our local changes have been accepted upstream: substantial
remaining local changes are code to read help files from /usr/share
rather than compiling them in, man page and help improvements, and some
tweaks to the default options.

Review and testing by millert, ok deraadt

Merge Less 458 plus local patches

ok guenther@, ok jmc@ for man bits

Merge in less 444 plus local changes

ok nicm@

Update to less version 382. This removes some code that is licensed under
a non-BSD license and which was not used on OpenBSD. No functional changes.

From Daniel Dickmann <didickman@gmail.com>, ok itojun, millert, ray, matthieu

use strchr instead of index

Merge in less-381 w/ local changes and remove obsolete files.

more strlcpy/strlcat/snprintf, less strcpy/strcat/sprintf
reviewed by tdeval, millert, dhartmei and others. more bits coming here
from some of them i think

kill more registers

millert@ ok

$OpenBSD$

branches: 1.1.1;
Initial revision