Add __attribute__((__noreturn__)) to various function prototypes in games/.
authorSascha Wildner <saw@online.de>
Thu, 28 Dec 2017 21:57:41 +0000 (22:57 +0100)
committerSascha Wildner <saw@online.de>
Thu, 28 Dec 2017 21:58:05 +0000 (22:58 +0100)
This general cleanup is useful in various ways. It helps the compiler
optimize better and it also helps later GCCs detect better when to give
-Wimplicit-fallthrough warnings and when not.

Found-with: -Wsuggest-attribute=noreturn

28 files changed:
games/adventure/hdr.h
games/arithmetic/arithmetic.c
games/atc/extern.h
games/backgammon/common_source/back.h
games/backgammon/teachgammon/tutor.h
games/battlestar/externs.h
games/cribbage/cribbage.h
games/hack/hack.h
games/hangman/hangman.h
games/hunt/hunt/client.h
games/hunt/huntd/server.h
games/larn/header.h
games/mille/mille.c
games/mille/mille.h
games/number/number.c
games/phantasia/io.c
games/phantasia/setup.c
games/pig/pig.c
games/quiz/quiz.c
games/robots/robots.h
games/rogue/rogue.h
games/sail/externs.h
games/tetris/tetris.c
games/tetris/tetris.h
games/trek/trek.h
games/worm/worm.c
games/worms/worms.c
games/wump/wump.c

index 9f2f964..740141f 100644 (file)
@@ -50,7 +50,6 @@
  * <URL:ftp://ftp.gmd.de/if-archive/games/source/advent-original.tar.gz>.
  *
  * $FreeBSD: src/games/adventure/hdr.h,v 1.5.2.1 2001/03/05 11:43:11 kris Exp $
- * $DragonFly: src/games/adventure/hdr.h,v 1.3 2004/09/12 17:19:58 dillon Exp $
  */
 
 /* hdr.h: included by c advent files */
@@ -157,7 +156,7 @@ int abbnum, bonus, clock1, clock2, closed,  /* various flags and counters */
 int demo, limit;
 
 int at(int objj);
-int bug(int n);
+int bug(int n) __dead2;
 void carry(int, int);
 void caveclose(void);
 void checkhints(void);
@@ -169,7 +168,7 @@ int dark(void);
 void datime(int *d, int *t);
 char *decr(const char *, const char *, const char *, const char *, const char *);
 void die(int entry);
-void done(int entry);
+void done(int entry) __dead2;
 void drop(int object, int where);
 void dstroy(int);
 int fdwarf(void);
index 3089ed7..5eba4cc 100644 (file)
 #include <unistd.h>
 
 static int getrandom(int, int, int);
-static void intr(int);
+static void intr(int) __dead2;
 static int opnum(int);
 static void penalise(int, int, int);
 static int problem(void);
 static void showstats(void);
-static void usage(void);
+static void usage(void) __dead2;
 
 static const char keylist[] = "+-x/";
 static const char defaultkeys[] = "+-";
index 6f0909b..246f499 100644 (file)
@@ -67,7 +67,7 @@ extern void   ioclrtobot(void);
 extern void    ioclrtoeol(int);
 extern void    ioerror(int, int, const char *);
 extern void    iomove(int);
-extern void    loser(const PLANE *, const char *);
+extern void    loser(const PLANE *, const char *) __dead2;
 extern void    planewin(void);
 extern void    redraw(void);
 extern void    setup_screen(const C_SCREEN *);
@@ -81,7 +81,7 @@ extern void   delete(LIST *, PLANE *);
 extern PLANE   *newplane(void);
 /* log.c */
 extern int     log_score(int);
-extern void    log_score_quit(int);
+extern void    log_score_quit(int) __dead2;
 extern void    open_score_file(void);
 /* update.c */
 extern int     addplane(void);
index 3b64c59..84c8744 100644 (file)
@@ -134,7 +134,7 @@ int count(void);
 void   curmove(int, int);
 void   dble(void);
 int    dblgood(void);
-void   errexit(const char *);
+void   errexit(const char *) __dead2;
 void   fancyc(char);
 void   fboard(void);
 void   fixtty(int);
@@ -142,7 +142,7 @@ int freemen(int);
 void   getarg(int, char **);
 int    getcaps(const char *);
 void   getmove(void);
-void   getout(int);            /* function to exit backgammon cleanly */
+void   getout(int) __dead2;    /* function to exit backgammon cleanly */
 void   gwrite(void);
 void   init(void);
 int    makmove(int);
index 2fb0b47..1912204 100644 (file)
  *     @(#)tutor.h     8.1 (Berkeley) 5/31/93
  */
 
-void   leave(void);
-void   tutor(void);
+#include <sys/cdefs.h>
+
+void   leave(void) __dead2;
+void   tutor(void) __dead2;
 
 struct situatn  {
        int     brd[26];
index aed0b4a..a994d8a 100644 (file)
@@ -29,7 +29,6 @@
  *     @(#)externs.h   8.1 (Berkeley) 5/31/93
  *
  *     $FreeBSD: src/games/battlestar/externs.h,v 1.9.2.1 2001/03/05 11:45:36 kris Exp $
- *     $DragonFly: src/games/battlestar/externs.h,v 1.4 2006/08/08 16:47:20 pavalos Exp $
  */
 
 #include <errno.h>
@@ -313,7 +312,7 @@ int card(const char *, int);
 void chime(void);
 void crash(void);
 int cypher(void);
-void die(int);
+void die(int) __dead2;
 void dig(void);
 int draw(void);
 void drink(void);
@@ -331,7 +330,7 @@ void kiss(void);
 int land(void);
 int launch(void);
 void light(void);
-void live(void);
+void live(void) __dead2;
 void love(void);
 void murder(void);
 void news(void);
index 44b97b1..2b5682b 100644 (file)
@@ -27,7 +27,6 @@
  * SUCH DAMAGE.
  *
  *     @(#)cribbage.h  8.1 (Berkeley) 5/31/93
- * $DragonFly: src/games/cribbage/cribbage.h,v 1.3 2005/08/03 13:31:00 eirikn Exp $
  */
 
 
@@ -71,7 +70,7 @@ char  *get_line(void);
 int     getuchar(void);
 int     infrom(CARD [], int, const char *);
 void    instructions(void);
-void    intr(int);
+void    intr(int) __dead2;
 bool    isone(CARD, CARD [], int);
 void    makedeck(CARD []);
 void    makeknown(CARD [], int);
index f875387..7358dfc 100644 (file)
@@ -1,6 +1,5 @@
 /* Copyright (c) Stichting Mathematisch Centrum, Amsterdam, 1985. */
 /* hack.h - version 1.0.3 */
-/* $DragonFly: src/games/hack/hack.h,v 1.4 2006/08/21 19:45:32 pavalos Exp $ */
 
 #include "config.h"
 #include <fcntl.h>
@@ -272,7 +271,7 @@ void         done_in_by(struct monst *);
 void    done(const char *);
 void    clearlocks(void);
 #ifdef NOSAVEONHANGUP
-void    hangup(int);
+void    hangup(int) __dead2;
 #endif
 char   *eos(char *);
 void    charcat(char *, char);
@@ -461,7 +460,7 @@ void        atl(int, int, char);
 void   on_scr(int, int);
 void   tmp_at(schar, schar);
 void   Tmp_at(schar, schar);
-void   setclipped(void);
+void   setclipped(void) __dead2;
 void   at(xchar, xchar, char);
 void   prme(void);
 int    doredraw(void);
@@ -505,7 +504,7 @@ void        outrumor(void);
 /* hack.save.c */
 int             dosave(void);
 #ifndef NOSAVEONHANGUP
-void            hangup(int);
+void            hangup(int) __dead2;
 #endif
 bool            dorecover(int);
 struct obj     *restobjchn(int);
@@ -616,14 +615,14 @@ void               drown(void);
 void    gettty(void);
 void    settty(const char *);
 void    setftty(void);
-void    error(const char *, ...) __printflike(1, 2);
+void    error(const char *, ...) __printflike(1, 2) __dead2;
 void    getlin(char *);
 void    getret(void);
 void    cgetret(const char *);
 void    xwaitforspace(const char *);
 char   *parse(void);
 char    readchar(void);
-void    end_of_input(void);
+void    end_of_input(void) __dead2;
 
 /* hack.u_init.c */
 void   u_init(void);
index 4aba72c..680862a 100644 (file)
@@ -91,7 +91,7 @@ extern int syms;
 extern int symfd;
 extern off_t symoffs, symsize;
 
-void   die(int);
+void   die(int) __dead2;
 void   endgame(void);
 void   getguess(void);
 void   getword(void);
index c8abf70..695806a 100644 (file)
@@ -29,7 +29,6 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $OpenBSD: client.h,v 1.4 2003/06/17 00:44:21 pjanzen Exp $
- * $DragonFly: src/games/hunt/hunt/client.h,v 1.1 2008/09/02 21:50:20 dillon Exp $
  */
 
 /* hunt.c */
@@ -39,8 +38,8 @@ extern char * Send_message;           /* -w message */
 extern int     Socket;                 /* connection to server */
 extern char     map_key[256];          /* HUNT envvar */
 
-void   bad_con(void);
-void   bad_ver(void);
+void   bad_con(void) __dead2;
+void   bad_ver(void) __dead2;
 void   intr(int);
 
 /* connect.c */
index 785606e..7732226 100644 (file)
@@ -30,7 +30,6 @@
  *
  * $OpenBSD: server.h,v 1.9 2004/01/16 00:13:19 espie Exp $
  * $NetBSD: hunt.h,v 1.5 1998/09/13 15:27:28 hubertf Exp $
- * $DragonFly: src/games/hunt/huntd/server.h,v 1.1 2008/09/02 21:50:21 dillon Exp $
  */
 
 
@@ -233,7 +232,7 @@ void        message(PLAYER *, const char *);
 /* driver.c */
 int    rand_num(int);
 void   checkdam(PLAYER *, PLAYER *, IDENT *, int, char);
-void   cleanup(int);
+void   cleanup(int) __dead2;
 
 /* execute.c */
 void   mon_execute(PLAYER *);
index 5516aa1..153713b 100644 (file)
@@ -1,5 +1,4 @@
 /*     header.h                Larn is copyrighted 1986 by Noah Morgan. */
-/* $DragonFly: src/games/larn/header.h,v 1.5 2006/08/26 17:05:05 pavalos Exp $ */
 
 #include <fcntl.h>
 #include <stdio.h>
@@ -357,7 +356,7 @@ extern struct _itm itm_[];
 extern const char *potionhide[], *potionname[], *scrollhide[], *scrollname[];
 
 /* bill.c */
-void   mailbill(void);
+void   mailbill(void) __dead2;
 
 /* create.c */
 void   makeplayer(void);
index ce936a0..2fb4e95 100644 (file)
@@ -27,7 +27,6 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/games/mille/mille.c,v 1.10 1999/12/12 06:17:24 billf Exp $
- * $DragonFly: src/games/mille/mille.c,v 1.5 2006/08/27 17:17:23 pavalos Exp $
  *
  * @(#) Copyright (c) 1982, 1993 The Regents of the University of California.  All rights reserved.
  * @(#)mille.c 8.1 (Berkeley) 5/31/93
@@ -42,7 +41,7 @@
  * @(#)mille.c 1.3 (Berkeley) 5/10/83
  */
 
-static void usage (void);
+static void usage (void) __dead2;
 
 int
 main(int ac, char *av[])
index 4e7ffcb..9e83a28 100644 (file)
@@ -29,7 +29,6 @@
  *     @(#)mille.h     8.1 (Berkeley) 5/31/93
  *
  * $FreeBSD: src/games/mille/mille.h,v 1.7 1999/12/12 06:17:24 billf Exp $
- * $DragonFly: src/games/mille/mille.h,v 1.6 2006/08/27 17:17:23 pavalos Exp $
  */
 
 #include <sys/types.h>
@@ -220,7 +219,7 @@ void        calcmove(void);
 bool   canplay(PLAY *, PLAY *, CARD);
 bool   check_ext(bool);
 void   check_more(void);
-void   die(int);
+void   die(int) __dead2;
 void   domove(void);
 bool   error(const char *, ...);
 #ifdef EXTRAP
index 14fcb7d..9674911 100644 (file)
@@ -68,7 +68,7 @@ static void   convert(char *);
 static int     number(char *, int);
 static void    pfract(int);
 static int     unit(int, char *);
-static void    usage(void);
+static void    usage(void) __dead2;
 
 static int lflag;
 
index 8dfd7d4..780d534 100644 (file)
@@ -14,7 +14,7 @@ extern        void    leavegame(void);
 /* phantglobs.c */
 extern double  drandom(void);
 
-void   catchalarm(int);
+void   catchalarm(int) __dead2;
 int    getanswer(const char *, bool);
 void   getstring(char *, int);
 double infloat(void);
index e80744d..9da19de 100644 (file)
@@ -2,7 +2,6 @@
  * setup.c - set up all files for Phantasia
  *
  * $FreeBSD: src/games/phantasia/setup.c,v 1.11 1999/11/16 02:57:34 billf Exp $
- * $DragonFly: src/games/phantasia/setup.c,v 1.3 2005/05/31 00:06:26 swildner Exp $
  */
 #include "include.h"
 #include <sys/types.h>
@@ -13,7 +12,7 @@
 /* phantglobs.c */
 extern double  drandom(void);
 
-void   Error(const char *, const char *);
+void   Error(const char *, const char *) __dead2;
 
 static const char *files[] = {         /* all files to create */
        _SPATH_MONST,
index 6499cb0..6e99e64 100644 (file)
@@ -40,7 +40,7 @@
 #include <unistd.h>
 
 static void pigout(char *, int);
-static void usage(void);
+static void usage(void) __dead2;
 
 int
 main(int argc, char **argv)
index 0636a1b..10ef998 100644 (file)
@@ -54,14 +54,14 @@ static u_int qsize;
 
 char   *appdstr (char *, char *, size_t);
 void    downcase (char *);
-void    err (const char *, ...) __printflike(1, 2);
+void    err (const char *, ...) __printflike(1, 2) __dead2;
 void    get_cats (char *, char *);
 void    get_file (const char *);
 char   *next_cat (char *);
 void    quiz (void);
 void    score (u_int, u_int, u_int);
 void    show_index (void);
-void    usage (void);
+void    usage (void) __dead2;
 
 int
 main(int argc, char **argv)
index 1118513..28e4757 100644 (file)
@@ -104,7 +104,7 @@ void make_level(void);
 void move_robots(int);
 void play_level(void);
 int query(const char *);
-void quit(int);
+void quit(int) __dead2;
 void reset_count(void);
 COORD *rnd_pos(void);
 void score(void);
index 4fea6d1..2f2c06d 100644 (file)
@@ -459,12 +459,12 @@ void      s_con_mon(object *);
 
 /* init.c */
 boolean        init(int, char**);
-void   clean_up(const char *);
+void   clean_up(const char *) __dead2;
 void   start_window(void);
 void   stop_window(void);
 void   byebye(int);
 void   onintr(int);
-void   error_save(int);
+void   error_save(int) __dead2;
 
 /* inventory.c */
 void   inventory(const object *, unsigned short);
@@ -503,7 +503,7 @@ const char  *md_gln(void);
 void   md_sleep(int);
 char   *md_getenv(const char *);
 char   *md_malloc(int);
-void   md_exit(int);
+void   md_exit(int) __dead2;
 void   md_lock(boolean);
 void   md_shell(const char *);
 #endif
@@ -627,10 +627,10 @@ void      save_into_file(const char *);
 void   restore(const char *);
 
 /* score.c */
-void   killed_by(const object *, short);
-void   win(void);
+void   killed_by(const object *, short) __dead2;
+void   win(void) __dead2;
 void   quit(boolean);
-void   put_scores(const object *, short);
+void   put_scores(const object *, short) __dead2;
 boolean        is_vowel(short);
 void   xxxx(char *, short);
 long   xxx(boolean);
index ab039af..2ae1c77 100644 (file)
@@ -353,12 +353,12 @@ bool boarding(struct ship *, char);
 void unboard(struct ship *, struct ship *, char);
 
 /* pl_1.c */
-void leave(int);
-void choke(int);
+void leave(int) __dead2;
+void choke(int) __dead2;
 void child(int);
 
 /* pl_2.c */
-void play(void);
+void play(void) __dead2;
 
 /* pl_3.c */
 void acceptcombat(void);
@@ -400,7 +400,7 @@ void leftview(void);
 void rightview(void);
 
 /* pl_main.c */
-void pl_main(void);
+void pl_main(void) __dead2;
 
 /* sync.c */
 void fmtship(char *, size_t, const char *, struct ship *);
index 5d5079f..e217654 100644 (file)
@@ -62,7 +62,7 @@ char  key_msg[100];
 int    showpreview, classic;
 
 static void elide(void);
-void onintr(int __unused);
+__dead2 void onintr(int __unused);
 const struct shape *randshape(void);
 static void setup_board(void);
 __dead2 void usage(void);
index e3d2507..66d54ef 100644 (file)
@@ -174,4 +174,4 @@ extern int  classic;
 
 int fits_in(const struct shape *, int);
 void place(const struct shape *, int, int);
-void stop(const char *);
+void stop(const char *) __dead2;
index c5d977c..8b7c07f 100644 (file)
@@ -408,13 +408,13 @@ void      killb(int, int);
 void   kills(int, int, int);
 void   killd(int, int, int);
 void   klmove(int);
-void   lose(int);
+void   lose(int) __dead2;
 void   lrscan(int);
 double move(int, int, double, double);
 void   nova(int, int);
 void   out(int);
 void   phaser(int);
-void   play(void);
+void   play(void) __dead2;
 void   ram(int, int);
 int    ranf(int);
 double franf(void);
@@ -434,8 +434,8 @@ const char  *systemname(struct quad *);
 void   torped(int);
 char   *bmove(const void *, void *, size_t);
 bool   sequal(const char *, const char *);
-void   syserr(const char *, ...) __printflike(1, 2);
+void   syserr(const char *, ...) __dead2 __printflike(1, 2);
 void   visual(int);
 void   warp(int, int, double);
 void   dowarp(int);
-void   win(void);
+void   win(void) __dead2;
index 181684e..9760133 100644 (file)
@@ -66,9 +66,9 @@ static int start_len = LENGTH;
 static char lastch;
 static char outbuf[BUFSIZ];
 
-static void crash(void);
+static void crash(void) __dead2;
 static void display(struct body *, char);
-static void leave(int);
+static void leave(int) __dead2;
 static void life(void);
 static void newpos(struct body *);
 static void prize(void);
index 6f57484..9990320 100644 (file)
@@ -166,7 +166,7 @@ static struct       worm {
 
 static volatile sig_atomic_t sig_caught = 0;
 
-static void nomem(void);
+static void nomem(void) __dead2;
 static void onsig(int);
 
 int
index 1676d37..838fd75 100644 (file)
@@ -118,7 +118,7 @@ static void pit_survive(void);
 static int shoot(char *);
 static void shoot_self(void);
 static int take_action(void);
-static void usage(void);
+static void usage(void) __dead2;
 static void wump_kill(void);
 static int wump_nearby(void);