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 ...

Pass errret pointer to setupterm() to prevent setupterm()
from calling exit() when given an unknown terminal type.
From Anton Lindqvist, who also upstreamed the fix.

style(9) includes

ok nicm@

calling ioctl() requires include sys/ioctl.h

wrapping an ioctl for TIOCGWINSZ inside #ifdef TIOCGWINSIZE (entirely
different) makes no sense; instead, accept all modern systems have
TIOCGWINSZ.

fsync() on a tty is meaningless; ok deraadt@

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 ...

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@

use STDERR_FILENO; ok cloder

Call fsync() _after_ tcsetattr() and pass tcsetattr the TCSASOFT
flag. Seems to help the occasional problem with messed up terminal
input after suspending less.

another round of missing system includes
ok tedu@

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

some more string shit; krw ok

kill more registers

millert@ ok

$OpenBSD$

use sc_e_keypad, sc_s_keypad, and use ti/te. Undoes previous revs which disabled. Unclear why that was done, must wait for that old bug to resurface and fix it in a different way

Do termios ops on /dev/tty if available, else fd 2.

TCSADRAIN -> TCSANOW. Using TCSADRAIN appears to cause a race condition
whereby the change is done with the wrong tcpgrp. This fixes suspending
less/more when invoked in mail(1) or via sh -c "less

TCSADRAIN -> TCSANOW. Using TCSADRAIN appears to cause a race condition
whereby the change is done with the wrong tcpgrp. This fixes suspending
less/more when invoked in mail(1) or via sh -c "less somefile".

show more ...

No broken push/pop for Theo's less -ewm.

Incorporate less modifications for more and our bug fixes from previous
version.

Import of unmolested less-290.