summaryrefslogtreecommitdiff
path: root/games
AgeCommit message (Collapse)Author
2017-06-02Fix an infinite loop in fortune(6).Frederic Cambus
When invoking fortune with the -l option (to get long dictums only), the program gets stuck in an infinite loop because fortlen() doesn't return the fortune length correctly. OK mestre@, tb@, deraadt@, schwarze@
2017-05-31late game space battles require tty pledge to clear screen, etc.Ted Unangst
from tomr
2017-05-28remove some escapes that are unnecessary/harmful. ok jmc schwarzeTed Unangst
2017-05-28remove unused macroTed Unangst
2017-05-28there are no bugs; i fixed themTed Unangst
2017-05-28if your dying scream would escape the death field, realign to stay withinTed Unangst
2017-05-28high scores printed too far to the right, move it left a littleTed Unangst
2017-05-26forgot to remove mention of deleted dump/restart.Ted Unangst
also delete ancient todo file: Frankly, I am getting pretty sick of playing this game. Hence, the version which you get may have several bugs
2017-05-26save and restore hasn't worked for some time, not since -fPIE, since itTed Unangst
uses a pointer equality check for save game validity. remove it. discovered by the ever inquisitive mlarkin
2017-05-19Silence a whole slew of clang warnings by doing a no-op in the crazyTheo Buehler
internal obfuscation scheme the other way around, thereby avoiding a signed underflow. ok mestre
2017-04-16According to termcap(3), char PC, *BC, *UP need to be extern. Fixes anTheo Buehler
ld(1) error found by mestre a while ago. fix suggested and ok naddy
2017-04-10Clean up disabled declaration in the text struct.Frederic Cambus
OK deraadt@
2017-04-08snprintf() format string should be literalGleydson Soares
avoid compiler silly warnings sure deraadt@
2017-04-08format string is better be literalGleydson Soares
OK deraadt
2017-01-21Nuke whitespace foolish enough to expose itself during the greatKenneth R Westerback
"warning:" rectification.
2017-01-20In "%.*s" the * takes (int). gcc whines if you try to use the resultKenneth R Westerback
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@
2017-01-20mark the prototypes of tutor() and leave() as __dead as well.Theo Buehler
ok guenther, krw
2017-01-20Mark functions that do not return as __dead to quiet gcc warnings.Kenneth R Westerback
ok beck@
2017-01-20Nuke some excess whitespace.Kenneth R Westerback
2016-12-25gcc says "if you define labels and don't use them, I will whine."Kenneth R Westerback
ok tom@
2016-12-21Eliminate another 'calculated, not used' warning by nuking a spuriousKenneth R Westerback
dereference. Same change made in NetBSD in 1997. ok tb@ millert@ tom@
2016-12-03elements 113, 115, 117, and 118; from pjanzenJason McIntyre
2016-12-01Remove useless #ifndef in atc(6) and pom(6).Frederic Cambus
M_PI is always defined, so we can drop those directives. OK deraadt@, millert@
2016-11-05spelling fix from eric van gyzen, freebsd r308293;Jason McIntyre
2016-09-12If boardspec doesn't consist entirely of lower case letters, there will beTheo Buehler
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
2016-09-12fix init, from randy hartman; ok tb@Otto Moerbeek
2016-09-11Callers of time(3) should #include <time.h>.Theo Buehler
ok deraadt
2016-09-11KNF: Break an overlong line. No binary change.Theo Buehler
2016-09-09These games have been broken for the better part of a year, ever sinceTheo Buehler
their setgid bits got removed. Mark them as such in their manuals. ok deraadt, help with wording jmc
2016-09-02Make build deterministic by not randomizing the datfiles themselves,Theo Buehler
random lines are selected on output and that's enough. From daniel, ok tb
2016-09-01In 32 bits sqrt(val) + 1 can overflow, so some big primes stillTheo Buehler
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
2016-08-31Fix a bogus comment: "factors of" -> "coprime to". Replace theTheo Buehler
nonsensical "if and only iff" with "if and only if" and zap some trailing whitespace.
2016-08-27the quotes in "unkempt thoughts" were authored by stanislaw lec,Jason McIntyre
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
2016-08-27Pull in <stdio.h> for NULLPhilip Guenther
ok deraadt@
2016-08-27Pull in <sys/select.h> for fd_setPhilip Guenther
Pull in <time.h> for time() ok deraadt@
2016-08-27Pull in <sys/select.h> for fd_setPhilip Guenther
ok deraadt@
2016-08-27Pull in <sys/time.h> for struct timespec, gettimeofday(), clock_gettime(),Philip Guenther
and setitimer() ok deraadt@
2016-08-27Pull in <time.h> for time() and perhaps other functionsPhilip Guenther
ok deraadt@
2016-08-14Convert remaining calls to strtoq/strtouq in base with strtoll/strtoull.Philip Guenther
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@
2016-07-11Instead of using the floating point square root, use an integer versionTheo Buehler
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
2016-06-10Clean up gcc -Wshadow warnings: 'numnames' is public in <term.h>.Theo Buehler
'i' shadows the local loop indexing variable of scr_update(), so no need to rename it. No binary change on amd64. ok millert
2016-06-10When eliding a row, clear the invisible row zero, so that no columnsTheo Buehler
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
2016-03-30for some time now mandoc has not required MLINKS to functionJason McIntyre
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
2016-03-28remove unused variablesCharles Longeau
ok millert@
2016-03-21Instead of creating a socket with socket() or accept() and thenPhilip Guenther
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@
2016-03-16Prefer fseek(3) over rewind(3) since the latter although it also calls fseekmestre
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@
2016-03-15execl(3) calls must have the last argument casted to (char *)NULLmestre
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@
2016-03-08- ranf() and franf() are prototyped on trek.h but ranf.c doesn't include it somestre
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
2016-03-08Headers cleanup with and OK tb@mestre
This was prompted by an initial patch sent by Edgar Pettijohn <edgar ! pettijohn-web.com> but the actual commit is different
2016-03-07eliminate do_malloc() and do_free().tb
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@