diff options
| author | Philip Guenther <guenther@cvs.openbsd.org> | 2015-09-27 05:13:12 +0000 |
|---|---|---|
| committer | Philip Guenther <guenther@cvs.openbsd.org> | 2015-09-27 05:13:12 +0000 |
| commit | 09d35c95ce0ff1594c9bf50b447c1886edffe109 (patch) | |
| tree | cf9cd951b0ed3f02fa6c13f7ad8cb3981d04a14a /games/hack/hack.timeout.c | |
| parent | 4b3f84711e25bb946b5c023faf89a43fbb194bd7 (diff) | |
Annotate funcs with __attribute__((printf(...))) and clean up the fallout:
* lots of foo(str) --> foo("%s", str) transformations
* one totally insane foo(fmt, ap) --> vfoo(fmt, ap) conversion: how did
this ever work?
* prefer const char[] over char* for static format strings, as it lets
gcc check the format and eliminates an unnecessary pointer
ok beck@
Diffstat (limited to 'games/hack/hack.timeout.c')
| -rw-r--r-- | games/hack/hack.timeout.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/games/hack/hack.timeout.c b/games/hack/hack.timeout.c index 481a360442a..e456c999ad3 100644 --- a/games/hack/hack.timeout.c +++ b/games/hack/hack.timeout.c @@ -1,4 +1,4 @@ -/* $OpenBSD: hack.timeout.c,v 1.5 2009/10/27 23:59:25 deraadt Exp $ */ +/* $OpenBSD: hack.timeout.c,v 1.6 2015/09/27 05:13:11 guenther Exp $ */ /* * Copyright (c) 1985, Stichting Centrum voor Wiskunde en Informatica, @@ -120,7 +120,7 @@ stoned_dialogue() long i = (Stoned & TIMEOUT); if(i > 0 && i <= SIZE(stoned_texts)) - pline(stoned_texts[SIZE(stoned_texts) - i]); + pline("%s", stoned_texts[SIZE(stoned_texts) - i]); if(i == 5) Fast = 0; if(i == 3) |