| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
| |
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.
|
|
|
|
| |
From Jake Champlin
|
|
|
|
| |
ok brynet@, tedu@
|
|
|
|
|
|
| |
in for loops.
ok stsp, bcallah
|
|
|
|
| |
ok tb
|
|
|
|
| |
ok jmc@
|
|
|
|
|
|
| |
ld(1) error found by mestre a while ago.
fix suggested and ok naddy
|
|
|
|
|
|
| |
and setitimer()
ok deraadt@
|
|
|
|
| |
ok 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
|
|
|
|
|
|
|
| |
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@
|
|
|
|
|
|
|
|
|
|
|
|
| |
- 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
|
|
|
|
| |
OK tb@
|
|
|
|
|
|
|
|
|
|
| |
- in main() replace exit with return
- drop some /* NOTREACHED */ lint comments along the way.
- make more use of standard CFLAGS, esp. -Wimplicit-function-declaration
- add and sort some headers when needed
- add straightforward pledges to some programs used at compile time
discussed with and ok mestre@
|
|
|
|
|
| |
OK tb@ and he also pointed out that for consistency with rest of the tree we
should include termios.h instead of sys/ttydefaults.h, where applicable
|
|
|
|
|
|
|
|
|
|
|
|
| |
srandom(time()+getpid()), was changed by srandomdev(), but #include <time.h>
lived up until this day so remove it.
Additionally, earlier than that, 18 years ago, random(6) was one of the first
consumers of arc4random(3) family, and was pulling it from dev/rndvar.h but
these days we pull it from stdlib.h, which is already done, so while here
remove dev/rndvar.h also.
"seems comprehensive to me" deraadt@ and OK tb@
|
|
|
|
| |
ok tb@
|
|
|
|
|
|
| |
Found another one in arithmetic(6) which also didn't return, and removed a function from number(6) which is not used anymore.
OK tb@
|
|
|
|
|
|
| |
Use sentence format for the highscore file, too.
Discussed with jmc@
|
|
|
|
|
|
|
| |
see environ(7). Pointed out by millert@. Fall back to getlogin(2)
before defaulting to ???.
looks good to deraadt@
|
|
|
|
| |
From Ricardo Mestre
|
|
|
|
| |
From Ricardo Mestre, thanks!
|
|
|
|
| |
From Ricardo Mestre, thanks!
|
|
|
|
|
|
|
| |
"stdio rpath tty" for game play
"rpath wpath cpath" for maintaining highscores in $USER
ok deraadt@ sthen@
|
|
|
|
|
| |
in turn, move score files into user's home directory.
will let theo buehler continue with a smaller pledge set.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
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@
|
| |
|
|
|
|
| |
ok tedu (... other games maintainer absent)
|
|
|
|
|
| |
found with the new mandoc(1) MANDOCERR_AN_MISSING warning;
no text changes
|
|
|
|
| |
tweaks and ok millert@, ok deraadt@
|
|
|
|
| |
OK nicm@
|
| |
|
|
|
|
|
| |
unmaintainable). these days, people use source.
spotted by schwarze
|
| |
|
|
|
|
| |
ok jmc@
|
| |
|
|
|
|
| |
OK jaredy@
|
|
|
|
| |
OK jaredy@
|
| |
|
| |
|
|
|
|
| |
ok deraadt mcbride millert nick otto pjanzen
|
|
|
|
| |
rescinded 22 July 1999. Proofed by myself and Theo.
|
| |
|
|
|
|
| |
already, but he may have later changes to make still.
|
|
|
|
|
| |
good enough for games but we should at least use a decent seed.
pjanzen@ OK
|
| |
|