spelling fixes; from paul tagliamonte
amendments to his diff are noted on tech

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, w

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

show more ...

Cleanup and bugfix:
When looking for uppercase characters, iterate over multibyte
characters with the standard function mbtowc(3) rather than with
the buggy and outdated step_char(), skipping invalid

Cleanup and bugfix:
When looking for uppercase characters, iterate over multibyte
characters with the standard function mbtowc(3) rather than with
the buggy and outdated step_char(), skipping invalid bytes,
and correctly use iswupper(3) instead of the inapplicable isupper(3).
OK stsp@

show more ...

When performing an inverted search in less, make sure to invalidate the match
bounds prior calling regexec(). In this inverted scenario a match is found when
regexec() returns false causing the bound

When performing an inverted search in less, make sure to invalidate the match
bounds prior calling regexec(). In this inverted scenario a match is found when
regexec() returns false causing the bounds to not be updated. This is
problematic since the bounds will then refer to a previous match and future
pointer arithmetic will eventually be off which is manifested in a SIGSEGV.

Issue reported by Larry Hynes on tech@
ok martijn@ tb@

show more ...

little bit more KNF

style(9) includes

ok nicm@

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

ok nicm@

Remove NULL-check before free().

remove fancy null check function in favor of checking for null.
from Ricardo Mestre

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

Remove NULL-checks before free(), a needless comment, and a needless
void* cast.

ok 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

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

show more ...

When invoked as more, make the -i option behave the same as less's
-I to match POSIX. OK and man bits from jmc@

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@

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$

Import of unmolested less-290.