less uses a correct raceless signal method of indicating signal events in
a volatile sig_atomic_t variable, and then processing events in the mainloop.
But only one variable was used for 3 signals, with |= bit operations which
are signal interruptable! Rewrite the code to use 3 independent variables
and cleanup how the mainloop observes indications.
ok schwarze

While freeing tag entries, make sure to free the copied strings.
From Anton Lindqvist. OK tobias@ nicm@

little bit more KNF

Remove LINUNUM, a needless alias for off_t. Also, remove a redundant
condition.

ok nicm@

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

Remove support for LESSGLOBALTAGS, we do not have global(1) (standard
ctags(1) support remains). ok tedu

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.

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

Use volatile sig_atomic_t, not int, for the sigs variable. Also
move the call to bell() out of u_interrupt() (a signal handler).
OK and with help from deraadt@

Merge in less 444 plus local changes

ok nicm@

tags.c isn't needed at all, so exclude the entire object instead of
commenting it out top-to-bottom
ok deraadt@

remove some bogus *p tests from charles longeau
ok deraadt millert

Fix an instance of foo[strlen(foo) - 1] = something, which is dangerous
because strlen(foo) could be 0.
OK beck@

typos from Jonathon Gray;

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

kill more registers

millert@ ok

$OpenBSD$

branches: 1.1.1;
Initial revision

While freeing tag entries, make sure to free the copied strings.
From Anton Lindqvist. OK tobias@ nicm@

little bit more KNF

Remove LINUNUM, a needless alias for off_t. Also, remove a redundant
condition.

ok nicm@

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

Remove support for LESSGLOBALTAGS, we do not have global(1) (standard
ctags(1) support remains). ok tedu

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.

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

Use volatile sig_atomic_t, not int, for the sigs variable. Also
move the call to bell() out of u_interrupt() (a signal handler).
OK and with help from deraadt@

Merge in less 444 plus local changes

ok nicm@

tags.c isn't needed at all, so exclude the entire object instead of
commenting it out top-to-bottom
ok deraadt@

remove some bogus *p tests from charles longeau
ok deraadt millert

Fix an instance of foo[strlen(foo) - 1] = something, which is dangerous
because strlen(foo) could be 0.
OK beck@

typos from Jonathon Gray;

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

kill more registers

millert@ ok

$OpenBSD$

branches: 1.1.1;
Initial revision