| Age | Commit message (Collapse) | Author |
|---|
|
|
|
from pjanzen
|
|
|
|
|
|
double #include.
ok deraadt@ tedu@
|
|
not for a fixed number of iterations. This makes a difference on terminals
not fast enough to update every second, in particular in -s mode.
Inspired by FreeBSD, but implemented differently with less bugs.
Patch from pjanzen@, slightly tweaked by me.
|
|
ok espie@ deraadt@ millert@ tedu@
|
|
|
|
arithmetics and validity constraints fixed by me. OK pjanzen@ tedu@
|
|
|
|
* properly delay scrolling with nanosleep(2), relevant on fast terminals
* change timing to end scrolling on the second instead of starting it
* error out if the terminal is too small instead of garbling the output
* use strtonum(3) rather than strtol(3) and terminate error message with \n
|
|
|
|
and <sys/ttydefaults.h>
Replace MAXPATHLEN with PATH_MAX and MAXLOGNAME with LOGIN_NAME_MAX
Pull in <limits.h> where needed
Prefer sizeof(var) over MAXFOO or FOO_MAX
ok deraadt@
|
|
troff displays these as typographic quotes, but nroff implementations
almost always print them literally, which rarely has the intended effect
with modern fonts, even in stock xterm.
These uses of `` '' can be replaced either with more semantic alternatives
or with Dq, which prints typographic quotes in a UTF-8 locale (but will
automatically fall back to `` '' in an ASCII locale).
improvements and ok schwarze@
|
|
since most of the text is different.
Improve the SYNOPSIS according to suggestions by jmc@, tweaked by me.
Delete a few redundant words, applying a patch from jmc@.
Intentionally not doing any other text changes in this commit;
for further tweaking in the tree.
jmc@, tedu@, and millert@ agree with the general direction, and
deraadt@ suspects we are "all grey beard fanatics" (in all caps).
|
|
|
|
|
|
|
|
|
|
|
|
on any char value.
|
|
don't negatively index into the table for signed chars.
ok pjanzen
|
|
|
|
ok tedu (... other games maintainer absent)
|
|
lopsided, a sharp counter can detect uneven permutations. fix this by
using knuth shuffle.
ok mlarkin pjanzen
|
|
|
|
Avoid potential integer overflow in the size argument of malloc() and
realloc() by using reallocarray() to avoid unchecked multiplication.
ok deraadt@
|
|
Avoid potential integer overflow in the size argument of malloc() and
realloc() by using reallocarray() to avoid unchecked multiplication.
ok deraadt@
|
|
Avoid potential integer overflow in the size argument of malloc() and
realloc() by using reallocarray() to avoid unchecked multiplication.
ok deraadt@
|
|
feature of the English language, so revert that part of the previous.
Requested by jmc@.
|
|
resolve contractions (found with textproc/igor(1)).
Work done by Kaspars at Bankovskis dot net
during the EuroBSDCon 2014 documentation tutorial.
Very minor addition of a few more missing .An macros by myself.
|
|
found with the new mandoc(1) MANDOCERR_AN_MISSING warning;
no text changes
|
|
|
|
caesar instead of a symlink.
agreed by deraadt, sobrado, guenther
|
|
|
|
|
|
|
|
ok mlarkin
|
|
code that catches lots of signals, and then re-enters curses... before
exiting.
|
|
Issue reported by David Crosby <dave at dafyddcrosby dot com> on tech@.
ok miod@
|
|
|
|
|
|
* get_line() always returns the same non-NULL pointer,
so testing the return value is useless.
* Garbage collect the unused variable *line.
* Check for end-of-buffer before parsing the next word instead of after it.
* Skip strlen() when the length has been measured three lines before anyway.
* Do not initialize local variables over and over again.
feedback and ok pjanzen@
|
|
When entering card names, you can use multiple words (like KING OF SPADES).
If you entered more than one consecutive blank character between words,
the function incard() took that as end-of-string and ignored the rest.
Fix this by dropping duplicate blanks up front, in get_line().
Patch simplified by me, ok pjanzen@.
While here, use beep(3) in an adjacent line instead of manually fiddling
with control characters, suggested by pjanzen@.
|
|
zap an old workaround needed to insert a space - it's now useless (was
actually generating a second space);
original diff from pjanzen
|
|
ok cmp(1)
|
|
stopped working because the third letter remained in the buffer,
incard() skipped the NUL and used the old garbage.
Fix this bug reported by pjanzen@, but in a simpler way than he
suggested, by just clearing any trailing garbage from the buffer.
ok pjanzen@
|
|
1. Style: Don't use variables as format strings.
NetBSD rev. 1.26 (May 23, 2011) joerg@NetBSD via maintainer pjanzen@.
2. Style: Avoid needless pointer arithmetics.
NetBSD rev. 1.27 (Oct 13, 2012) dholland@NetBSD via pjanzen@,
who also applied some KNF to the indentation while here.
3. End curses before printing fatal error messages; from pjanzen@.
|
|
NetBSD rev. 1.16 (Oct 13, 2012) by dholland@NetBSD via maintainer pjanzen@.
|
|
While here, add .An (author name) macros.
|
