wireguard-openbsd/games/snake, branch jd/queueboosts

When system calls indicate an error they return -1, not some arbitrary

2019-06-28T13:32:41Z

value < 0. errno is only updated in this case. Change all (most?) callers of syscalls to follow this better, and let's see if this strictness helps us in the future.

printing the pinball bonus in the corner is confusing when you lose.

2019-01-20T04:14:19Z

just let the bonus happen if it does, players will figure it out.

remove some obsolete comments about deleted code

2019-01-20T04:12:58Z

quick fix to keep the cursor in the corner during space warp.

2019-01-20T04:10:12Z

less distracting this way.

if the snake ran over the money, print the treasure instead of empty.

2019-01-20T04:09:15Z

bug noticed by mlarkin

Remove a few too early pledge(2)s on games/ and apply them a little bit later

2018-08-24T11:14:49Z

but with much reduced permissions ("stdio tty" if ncurses based and "stdio" for the ones that only perform basic operations). There's still a few games that we cannot yet remove their fs access, through pledge(2), since they open files on demand and too late, this might get revisited in the future. OK tb@

Callers of time(3) should #include .

2016-09-11T14:21:17Z

ok deraadt

Prefer fseek(3) over rewind(3) since the latter although it also calls fseek

2016-03-16T15:00:35Z

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@

- General changes:

2016-03-07T12:07:55Z

- 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

- Remove parameter fd from snscore() since it's never used

2016-02-09T13:41:59Z

- 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