Age | Commit message (Collapse) | Author |
|
avoid compiler silly warnings
sure deraadt@
|
|
OK deraadt
|
|
"warning:" rectification.
|
|
of pointer subtraction without a cast. So cast those expressions
to (int).
Switch one local variable to the same type as the parameter it is
compared to.
ok deraadt@ guenther@ beck@
|
|
ok guenther, krw
|
|
ok beck@
|
|
|
|
ok tom@
|
|
dereference.
Same change made in NetBSD in 1997.
ok tb@ millert@ tom@
|
|
|
|
M_PI is always defined, so we can drop those directives.
OK deraadt@, millert@
|
|
|
|
out of bound accesses of arrays, leading to sefaults or bus errors. Verify
that at most one boardspec with lower case letters is given, or error out.
ok millert
|
|
|
|
ok deraadt
|
|
|
|
their setgid bits got removed. Mark them as such in their manuals.
ok deraadt, help with wording jmc
|
|
random lines are selected on output and that's enough.
From daniel, ok tb
|
|
aren't recognized as such, for example 18446744073709551577 given
in the commit message of factor.c r1.7 from NetBSD. Move the
return type of usqrt() from u_int32_t to u_int64_t.
ok guenther, tom, otto
|
|
nonsensical "if and only iff" with "if and only if" and zap some
trailing whitespace.
|
|
not stanislaw lem (an easy blunder, i'd say); i've also inserted the "J."
into one of these credits (Stanislaw J. Lec) since all the others have them;
credit to antoni grzymala; diff pockled from netbsd
|
|
ok deraadt@
|
|
Pull in <time.h> for time()
ok deraadt@
|
|
ok deraadt@
|
|
and setitimer()
ok deraadt@
|
|
ok deraadt@
|
|
Fix a type mismatch in ftp's "page" command and could make transfers restart
at the wrong position.
ok and a ull->ll tweak from natano@, ok tedu@
|
|
of the Newton method from ping.c. Fixes a rounding issue that caused
failure to factor numbers close to 2^64, e.g. 18446744030759878681.
While there, fix an off by one error that caused 4295360521 to be
reported as a prime. Issues reported by Paul Stoeber and Michael Bozon.
ok tedu, deraadt
|
|
'i' shadows the local loop indexing variable of scr_update(), so
no need to rename it. No binary change on amd64.
ok millert
|
|
can become unusable during game play. Same fix was made in NetBSD's
tetris.c -r1.31 by Christos Zoulas almost exactly a year ago.
ok tedu
|
|
correctly - logically complete that now by removing MLINKS from base;
authors need only to ensure there is an entry in NAME for any function/
util being added. MLINKS will still work, and remain for perl to ease
upgrades;
ok nicm (curses) bcook (ssl)
ok schwarze, who provided a lot of feedback and assistance
ok tb natano jung
|
|
ok millert@
|
|
setting the O_NONBLOCK flag on it with fcntl(F_SETFL) afterwards,
just pass SOCK_NONBLOCK to socket() or accept4() and get it right
to begin with.
ok millert@ krw@ beck@ deraadt@ jca@
|
|
then additionally it calls clearerr(3) deliberately but we want to catch any
error that may happen and this way we couldn't catch it
OK tb@
|
|
At the time when this was commited me and tb@ discussed that it shouldn't be
changed, but still snucked in within a larger diff that we didn't notice.
OK tb@
|
|
do it
- Replace random() >> 5 by only random(): this was discussed a few months ago
and naddy@ said at the time "Those games were originally written with the
rand(3) function. The lower bits returned by rand(3) suffered from notoriously
poor randomness, so this idiom developed where people would use the higher,
more random bits."
OK tb@ after his remarks
|
|
This was prompted by an initial patch sent by Edgar Pettijohn
<edgar ! pettijohn-web.com> but the actual commit is different
|
|
These are wrappers for malloc(3) and free(3) with NULL checks. do_free()
is pointless since free() already checks for NULL. do_malloc() is used
only three times, once asprintf(3) seems more appropriate, and for just
two calls the benefit of a custom wrapper is minimal.
ok millert@
|
|
simplify is_fortfile() accordingly.
The last use of them was guarded by OK_TO_WRITE_DISK, which was
unifdef'ed in revision 1.43 last fall.
tweak + ok mestre@
|
|
|
|
- Remove -? from getopt(3) options, but still keep (or add) -h where applicable
- Replace hardcoded program strings by getprogname(3)
- Specific changes:
- atc(6): this used -? and -u for usage(), remove both from game and manpage
- bcd(6): use __progname instead of getprogname(3), no need to include stdlib.h
- hunt(6): replace fputs(3) by fprintf(3)
OK tb@ after his suggestions
|
|
integer overflow on bogus input and bizarre error messages. Moreover,
fortune files can now be named anything except names matching the regex
^[0-9][0-9.]*%$
ok mestre@
|
|
|
|
- Replace hardcoded program string by getprogname(3)
- Remove '?' from getopt(3)'s switch default case (but still keep 'h')
OK tb@ after his suggestions
|
|
|
|
pjanzen@ correctly pointed out that a majority of games do
this, so no need to deviate here.
|
|
ok mestre@
|
|
- Replace hardcoded string "hangman" by getprogname(3)
OK tb@
|
|
- Remove lint-style comment
- Remove usage() since errors are now more informative (the usage is still
available in the manpage)
With this diff I made it accept 0 as rotation whereas before it didn't, but
alas if you use 0 then it defeats the whole purpose of the game.
Initial diff sent by Peter Williams <peterbw at gmail.com>, tweaked by me and
several hints and OK by tb@. I'm in desperate need of a coffee, thank you tb@
for making me notice that!
|
|
- And while here, fwrite(3) returns size_t whereas nscores is an int, so cast
nscores to u_int. This is a false positive, but silences a compiler warning
with -Wextra -pedantic
tb@ : "Looks ok to me" after suggestion from him
|