Ricardo Mestre wrote:
> phantasia(6) needs some KNF love, but for now here is a patch to
> ansify it:

Did you verify that there's no binary change?

> Index: fight.c
> ===================================================================
> RCS file: /cvs/src/games/phantasia/fight.c,v
> retrieving revision 1.11
> diff -u -p -u -r1.11 fight.c
> --- fight.c   12 Jul 2014 03:41:04 -0000      1.11
> +++ fight.c   2 Dec 2015 19:25:53 -0000
> @@ -38,8 +38,7 @@
>  *************************************************************************/
>  
>  void
> -encounter(particular)
> -     int     particular;
> +encounter(int particular)
>  {
>       int flockcnt = 1;       /* how many time flocked */
>       volatile bool firsthit = Player.p_blessing;     /* set if player gets
> @@ -207,7 +206,7 @@ encounter(particular)
>  *************************************************************************/
>  
>  int
> -pickmonster()
> +pickmonster(void)
>  {
>       if (Player.p_specialtype == SC_VALAR)
>               /* even chance of any monster */
> @@ -264,7 +263,7 @@ pickmonster()
>  *************************************************************************/
>  
>  void
> -playerhits()
> +playerhits(void)
>  {
>       double  inflict;        /* damage inflicted */
>       int     ch;             /* input */
> @@ -435,7 +434,7 @@ playerhits()
>  *************************************************************************/
>  
>  void
> -monsthits()
> +monsthits(void)
>  {
>       double  inflict;        /* damage inflicted */
>       int     ch;             /* input */
> @@ -709,7 +708,7 @@ SPECIALHIT:
>  *************************************************************************/
>  
>  void
> -cancelmonster()
> +cancelmonster(void)
>  {
>      Curmonster.m_energy = 0.0;
>      Curmonster.m_experience = 0.0;
> @@ -743,8 +742,7 @@ cancelmonster()
>  *************************************************************************/
>  
>  void
> -hitmonster(inflict)
> -     double  inflict;
> +hitmonster(double inflict)
>  {
>       mvprintw(Lines++, 0, "You hit %s %.0f times!", Enemyname, inflict);
>       Curmonster.m_energy -= inflict;
> @@ -797,7 +795,7 @@ hitmonster(inflict)
>  *************************************************************************/
>  
>  void
> -throwspell()
> +throwspell(void)
>  {
>       double  inflict;        /* damage inflicted */
>       double  dtemp;          /* for dtemporary calculations */
> @@ -1028,8 +1026,7 @@ throwspell()
>  *************************************************************************/
>  
>  void
> -callmonster(which)
> -     int     which;
> +callmonster(int which)
>  {
>       struct monster Othermonster;    /* to find a name for mimics */
>  
> @@ -1136,7 +1133,7 @@ callmonster(which)
>  *************************************************************************/
>  
>  void
> -awardtreasure()
> +awardtreasure(void)
>  {
>       int     whichtreasure;  /* calculated treasure to grant */
>       int     temp;           /* temporary */
> @@ -1573,7 +1570,7 @@ awardtreasure()
>  *************************************************************************/
>  
>  void
> -cursedtreasure()
> +cursedtreasure(void)
>  {
>       if (Player.p_charms > 0) {
>               addstr("But your charm saved you!\n");
> @@ -1613,7 +1610,7 @@ cursedtreasure()
>  *************************************************************************/
>  
>  void
> -scramblestats()
> +scramblestats(void)
>  {
>       double  dbuf[6];        /* to put statistic in */
>       double  dtemp1, dtemp2; /* for swapping values */
> Index: gamesupport.c
> ===================================================================
> RCS file: /cvs/src/games/phantasia/gamesupport.c,v
> retrieving revision 1.7
> diff -u -p -u -r1.7 gamesupport.c
> --- gamesupport.c     15 Dec 2010 06:40:39 -0000      1.7
> +++ gamesupport.c     2 Dec 2015 19:25:55 -0000
> @@ -54,8 +54,7 @@
>  *************************************************************************/
>  
>  void
> -changestats(ingameflag)
> -     bool    ingameflag;
> +changestats(bool ingameflag)
>  {
>       static char flag[2] =   /* for printing values of bools */
>       {'F', 'T'};
> @@ -509,7 +508,7 @@ BALTER:
>  *************************************************************************/
>  
>  void
> -monstlist()
> +monstlist(void)
>  {
>       int     count = 0;      /* count in file */
>  
> @@ -546,7 +545,7 @@ monstlist()
>  *************************************************************************/
>  
>  void
> -scorelist()
> +scorelist(void)
>  {
>       struct scoreboard sbuf; /* for reading entries */
>       FILE   *fp;             /* to open the file */
> @@ -583,7 +582,7 @@ scorelist()
>  *************************************************************************/
>  
>  void
> -activelist()
> +activelist(void)
>  {
>       fseek(Playersfp, 0L, SEEK_SET);
>       printf("Current characters on file are:\n\n");
> @@ -621,7 +620,7 @@ activelist()
>  *************************************************************************/
>  
>  void
> -purgeoldplayers()
> +purgeoldplayers(void)
>  {
>       int     today;          /* day of year for today */
>       int     daysold;        /* how many days since the character has been
> @@ -678,7 +677,7 @@ purgeoldplayers()
>  *************************************************************************/
>  
>  void
> -enterscore()
> +enterscore(void)
>  {
>       struct scoreboard sbuf; /* buffer to read in scoreboard entries */
>       FILE   *fp;             /* to open scoreboard file */
> Index: interplayer.c
> ===================================================================
> RCS file: /cvs/src/games/phantasia/interplayer.c,v
> retrieving revision 1.6
> diff -u -p -u -r1.6 interplayer.c
> --- interplayer.c     12 Jul 2014 03:41:04 -0000      1.6
> +++ interplayer.c     2 Dec 2015 19:25:57 -0000
> @@ -33,7 +33,7 @@
>  *************************************************************************/
>  
>  void
> -checkbattle()
> +checkbattle(void)
>  {
>       long    foeloc = 0L;    /* location in file of person to fight */
>  
> @@ -110,8 +110,7 @@ checkbattle()
>  *************************************************************************/
>  
>  void
> -battleplayer(foeplace)
> -     long    foeplace;
> +battleplayer(long foeplace)
>  {
>       double  dtemp;          /* for temporary calculations */
>       double  oldhits = 0.0;  /* previous damage inflicted by foe */
> @@ -357,7 +356,7 @@ LEAVE:
>  *************************************************************************/
>  
>  void
> -myturn()
> +myturn(void)
>  {
>       double  dtemp;          /* for temporary calculations */
>       int     ch;             /* input */
> @@ -459,7 +458,7 @@ HIT:
>  *************************************************************************/
>  
>  void
> -checktampered()
> +checktampered(void)
>  {
>       long    loc = 0L;       /* location in energy void file */
>  
> @@ -520,10 +519,7 @@ checktampered()
>  *************************************************************************/
>  
>  void
> -tampered(what, arg1, arg2)
> -     int     what;
> -     double  arg1;
> -     double  arg2;
> +tampered(int what, double arg1, double arg2)
>  {
>       long    loc;            /* location in file of other players */
>  
> @@ -704,8 +700,7 @@ tampered(what, arg1, arg2)
>  *************************************************************************/
>  
>  void
> -userlist(ingameflag)
> -     bool    ingameflag;
> +userlist(bool ingameflag)
>  {
>       int     numusers = 0;   /* number of users on file */
>  
> @@ -795,7 +790,7 @@ userlist(ingameflag)
>  *************************************************************************/
>  
>  void
> -throneroom()
> +throneroom(void)
>  {
>       FILE   *fp;             /* to clear energy voids */
>       long    loc = 0L;       /* location of old king in player file */
> @@ -876,7 +871,7 @@ throneroom()
>  *************************************************************************/
>  
>  void
> -dotampered()
> +dotampered(void)
>  {
>       short   tamper;         /* value for tampering with other players */
>       char   *option;         /* pointer to option description */
> @@ -1108,9 +1103,7 @@ dotampered()
>  *************************************************************************/
>  
>  void
> -writevoid(vp, loc)
> -     struct energyvoid *vp;
> -     long    loc;
> +writevoid(struct energyvoid *vp, long loc)
>  {
>  
>       fseek(Energyvoidfp, loc, SEEK_SET);
> @@ -1145,7 +1138,7 @@ writevoid(vp, loc)
>  *************************************************************************/
>  
>  long
> -allocvoid()
> +allocvoid(void)
>  {
>       long    loc = 0L;       /* location of new energy void */
>  
> Index: io.c
> ===================================================================
> RCS file: /cvs/src/games/phantasia/io.c,v
> retrieving revision 1.6
> diff -u -p -u -r1.6 io.c
> --- io.c      12 Jul 2014 03:41:04 -0000      1.6
> +++ io.c      2 Dec 2015 19:25:58 -0000
> @@ -43,9 +43,7 @@
>  *************************************************************************/
>  
>  void
> -getstring(cp, mx)
> -     char   *cp;
> -     int     mx;
> +getstring(char *cp, int mx)
>  {
>       char   *inptr;          /* pointer into string for next string */
>       int     x, y;           /* original x, y coordinates on screen */
> @@ -119,8 +117,7 @@ getstring(cp, mx)
>  *************************************************************************/
>  
>  void
> -more(where)
> -     int     where;
> +more(int where)
>  {
>       mvaddstr(where, 0, "-- more --");
>       getanswer(" ", FALSE);
> @@ -152,7 +149,7 @@ more(where)
>  *************************************************************************/
>  
>  double
> -infloat()
> +infloat(void)
>  {
>       double  result;         /* return value */
>  
> @@ -190,7 +187,7 @@ infloat()
>  *************************************************************************/
>  
>  int
> -inputoption()
> +inputoption(void)
>  {
>       ++Player.p_age;         /* increase age */
>  
> @@ -233,7 +230,7 @@ inputoption()
>  *************************************************************************/
>  
>  void
> -interrupt()
> +interrupt(void)
>  {
>       char    line[81];       /* a place to store data already on screen */
>       int     loop;           /* counter */
> @@ -307,9 +304,7 @@ interrupt()
>  *************************************************************************/
>  
>  int
> -getanswer(choices, def)
> -     char   *choices;
> -     bool    def;
> +getanswer(char *choices, bool def)
>  {
>                int ch;         /* input */
>       volatile int loop;       /* counter */
> @@ -407,8 +402,7 @@ getanswer(choices, def)
>  *************************************************************************/
>  
>  void
> -catchalarm(dummy)
> -     int dummy;
> +catchalarm(int dummy)
>  {
>       longjmp(Timeoenv, 1);
>  }
> Index: main.c
> ===================================================================
> RCS file: /cvs/src/games/phantasia/main.c,v
> retrieving revision 1.17
> diff -u -p -u -r1.17 main.c
> --- main.c    16 Nov 2014 04:49:48 -0000      1.17
> +++ main.c    2 Dec 2015 19:26:00 -0000
> @@ -95,9 +95,7 @@
>  ****************************************************************************/
>  
>  int
> -main(argc, argv)
> -     int     argc;
> -     char  **argv;
> +main(int argc, char **argv)
>  {
>       bool    noheader = FALSE;       /* set if don't want header */
>       bool    headeronly = FALSE;     /* set if only want header */
> @@ -313,7 +311,7 @@ main(argc, argv)
>  *************************************************************************/
>  
>  void
> -initialstate()
> +initialstate(void)
>  {
>  #ifdef TERMIOS
>      struct termios tty;
> @@ -398,7 +396,7 @@ initialstate()
>  *************************************************************************/
>  
>  long
> -rollnewplayer()
> +rollnewplayer(void)
>  {
>       int     chartype;       /* character type */
>       int     ch;             /* input */
> @@ -515,7 +513,7 @@ rollnewplayer()
>  *************************************************************************/
>  
>  void
> -procmain()
> +procmain(void)
>  {
>       int     ch;             /* input */
>       double  x;              /* desired new x coordinate */
> @@ -749,7 +747,7 @@ procmain()
>  *************************************************************************/
>  
>  void
> -titlelist()
> +titlelist(void)
>  {
>       FILE   *fp;             /* used for opening various files */
>       bool    councilfound = FALSE;   /* set if we find a member of the
> @@ -883,7 +881,7 @@ titlelist()
>  *************************************************************************/
>  
>  long
> -recallplayer()
> +recallplayer(void)
>  {
>       long    loc = 0L;       /* location in player file */
>       int     loop;           /* loop counter */
> @@ -966,7 +964,7 @@ recallplayer()
>  *************************************************************************/
>  
>  void
> -neatstuff()
> +neatstuff(void)
>  {
>       double  temp;           /* for temporary calculations */
>       int     ch;             /* input */
> @@ -1090,8 +1088,7 @@ neatstuff()
>  *************************************************************************/
>  
>  void
> -genchar(type)
> -     int     type;
> +genchar(int type)
>  {
>       int     subscript;      /* used for subscripting into Stattable */
>       struct charstats *statptr;      /* for pointing into Stattable */
> @@ -1151,7 +1148,7 @@ genchar(type)
>  *************************************************************************/
>  
>  void
> -playinit()
> +playinit(void)
>  {
>       initscr();              /* turn on curses */
>       noecho();               /* do not echo input */
> @@ -1188,8 +1185,7 @@ playinit()
>  *************************************************************************/
>  
>  void
> -cleanup(doexit)
> -     int     doexit;
> +cleanup(int doexit)
>  {
>       if (Windows) {
>               move(LINES - 2, 0);
> Index: map.c
> ===================================================================
> RCS file: /cvs/src/games/phantasia/map.c,v
> retrieving revision 1.2
> diff -u -p -u -r1.2 map.c
> --- map.c     28 Jan 2001 23:41:47 -0000      1.2
> +++ map.c     2 Dec 2015 19:26:01 -0000
> @@ -4,7 +4,8 @@
>  #define      minusminus      plusplus
>  #define      minusplus       plusminus
>  
> -main()
> +void
> +main(void)
>  {
>      /* Set up */
>  
> @@ -128,9 +129,8 @@ main()
>      exit(0);
>  }
>  
> -plusplus(s, x, y)    /* draw strings in plus plus quadrant */
> -char *s;
> -int  x, y;
> +void
> +plusplus(char *s, int x, int y)      /* draw strings in plus plus quadrant */
>  {
>  char s1[2];
>  
> @@ -145,9 +145,8 @@ char      s1[2];
>       }
>  }
>  
> -plusminus(s, x, y)   /* draw strings in plus minus quadrant */
> -char *s;
> -int  x, y;
> +void
> +plusminus(char *s, int x, int y)     /* draw strings in plus minus quadrant 
> */
>  {
>  char s1[2];
>  
> Index: misc.c
> ===================================================================
> RCS file: /cvs/src/games/phantasia/misc.c,v
> retrieving revision 1.16
> diff -u -p -u -r1.16 misc.c
> --- misc.c    12 Jul 2014 03:41:04 -0000      1.16
> +++ misc.c    2 Dec 2015 19:26:03 -0000
> @@ -37,7 +37,7 @@
>  *************************************************************************/
>  
>  void
> -movelevel()
> +movelevel(void)
>  {
>       struct charstats *statptr;      /* for pointing into Stattable */
>       double  new;            /* new level */
> @@ -116,9 +116,7 @@ movelevel()
>  *************************************************************************/
>  
>  char   *
> -descrlocation(playerp, shortflag)
> -     struct player *playerp;
> -     bool    shortflag;
> +descrlocation(struct player *playerp, bool shortflag)
>  {
>       double  circle;         /* corresponding circle for coordinates */
>       int     quadrant;       /* quandrant of grid */
> @@ -215,7 +213,7 @@ descrlocation(playerp, shortflag)
>  *************************************************************************/
>  
>  void
> -tradingpost()
> +tradingpost(void)
>  {
>       double  numitems;       /* number of items to purchase */
>       double  cost;           /* cost of purchase */
> @@ -486,7 +484,7 @@ tradingpost()
>  *************************************************************************/
>  
>  void
> -displaystats()
> +displaystats(void)
>  {
>       mvprintw(0, 0, "%s%s\n", Player.p_name, descrlocation(&Player, FALSE));
>       mvprintw(1, 0, "Level :%7.0f   Energy  :%9.0f(%9.0f)  Mana :%9.0f  
> Users:%3d\n",
> @@ -521,7 +519,7 @@ displaystats()
>  *************************************************************************/
>  
>  void
> -allstatslist()
> +allstatslist(void)
>  {
>       static char *flags[] =  /* to print value of some bools */
>       {
> @@ -583,9 +581,7 @@ allstatslist()
>  *************************************************************************/
>  
>  char   *
> -descrtype(playerp, shortflag)
> -     struct player *playerp;
> -     bool    shortflag;
> +descrtype(struct player *playerp, bool shortflag)
>  {
>       int     type;           /* for caluculating result subscript */
>       static char *results[] =/* description table */
> @@ -673,9 +669,7 @@ descrtype(playerp, shortflag)
>  *************************************************************************/
>  
>  long
> -findname(name, playerp)
> -     char   *name;
> -     struct player *playerp;
> +findname(char *name, struct player *playerp)
>  {
>       long    loc = 0;        /* location in the file */
>  
> @@ -717,7 +711,7 @@ findname(name, playerp)
>  *************************************************************************/
>  
>  long
> -allocrecord()
> +allocrecord(void)
>  {
>       long    loc = 0L;       /* location in file */
>  
> @@ -764,9 +758,7 @@ allocrecord()
>  *************************************************************************/
>  
>  void
> -freerecord(playerp, loc)
> -     struct player *playerp;
> -     long    loc;
> +freerecord(struct player *playerp, long loc)
>  {
>       playerp->p_name[0] = CH_MARKDELETE;
>       playerp->p_status = S_NOTUSED;
> @@ -798,7 +790,7 @@ freerecord(playerp, loc)
>  *************************************************************************/
>  
>  void
> -leavegame()
> +leavegame(void)
>  {
>  
>       if (Player.p_level < 1.0)
> @@ -845,8 +837,7 @@ leavegame()
>  *************************************************************************/
>  
>  void
> -death(how)
> -     char   *how;
> +death(char *how)
>  {
>       FILE   *fp;             /* for updating various files */
>       int     ch;             /* input */
> @@ -968,9 +959,7 @@ death(how)
>  *************************************************************************/
>  
>  void
> -writerecord(playerp, place)
> -     struct player *playerp;
> -     long    place;
> +writerecord(struct player *playerp, long place)
>  {
>       fseek(Playersfp, place, SEEK_SET);
>       fwrite(playerp, SZ_PLAYERSTRUCT, 1, Playersfp);
> @@ -1003,8 +992,7 @@ writerecord(playerp, place)
>  *************************************************************************/
>  
>  double
> -explevel(experience)
> -     double  experience;
> +explevel(double experience)
>  {
>       if (experience < 1.1e7)
>               return (floor(pow((experience / 1000.0), 0.4875)));
> @@ -1037,8 +1025,7 @@ explevel(experience)
>  *************************************************************************/
>  
>  void
> -truncstring(string)
> -     char   *string;
> +truncstring(char *string)
>  {
>       int     length;         /* length of string */
>  
> @@ -1075,10 +1062,7 @@ truncstring(string)
>  *************************************************************************/
>  
>  void
> -altercoordinates(xnew, ynew, operation)
> -     double  xnew;
> -     double  ynew;
> -     int     operation;
> +altercoordinates(double xnew, double ynew, int operation)
>  {
>       switch (operation) {
>       case A_FORCED:          /* move with no checks */
> @@ -1151,9 +1135,7 @@ altercoordinates(xnew, ynew, operation)
>  *************************************************************************/
>  
>  void
> -readrecord(playerp, loc)
> -     struct player *playerp;
> -     long    loc;
> +readrecord(struct player *playerp, long loc)
>  {
>       fseek(Playersfp, loc, SEEK_SET);
>       fread(playerp, SZ_PLAYERSTRUCT, 1, Playersfp);
> @@ -1183,7 +1165,7 @@ readrecord(playerp, loc)
>  *************************************************************************/
>  
>  void
> -adjuststats()
> +adjuststats(void)
>  {
>       double  dtemp;          /* for temporary calculations */
>  
> @@ -1298,8 +1280,7 @@ adjuststats()
>  *************************************************************************/
>  
>  void
> -initplayer(playerp)
> -     struct player *playerp;
> +initplayer(struct player *playerp)
>  {
>       playerp->p_experience =
>           playerp->p_level =
> @@ -1381,7 +1362,7 @@ initplayer(playerp)
>  *************************************************************************/
>  
>  void
> -readmessage()
> +readmessage(void)
>  {
>       move(3, 0);
>       clrtoeol();
> @@ -1415,8 +1396,7 @@ readmessage()
>  *************************************************************************/
>  
>  void
> -error(whichfile)
> -     char    *whichfile;
> +error(char *whichfile)
>  {
>  
>       if (Windows)
> @@ -1456,8 +1436,7 @@ error(whichfile)
>  *************************************************************************/
>  
>  double
> -distance(x1, x2, y1, y2)
> -     double  x1, x2, y1, y2;
> +distance(double x1, double x2, double y1, double y2)
>  {
>       double  deltax, deltay;
>  
> @@ -1491,8 +1470,7 @@ distance(x1, x2, y1, y2)
>  *************************************************************************/
>  
>  char *
> -descrstatus(playerp)
> -     struct player *playerp;
> +descrstatus(struct player *playerp)
>  {
>       switch (playerp->p_status) {
>       case S_PLAYING:
> @@ -1551,7 +1529,7 @@ descrstatus(playerp)
>  *************************************************************************/
>  
>  double
> -drandom()
> +drandom(void)
>  {
>       return ((double) arc4random() / (UINT32_MAX + 1.0));
>  }
> @@ -1585,9 +1563,7 @@ drandom()
>  *************************************************************************/
>  
>  void
> -collecttaxes(gold, gems)
> -     double  gold;
> -     double  gems;
> +collecttaxes(double gold, double gems)
>  {
>       FILE   *fp;             /* to update Goldfile */
>       double  dtemp;          /* for temporary calculations */
> Index: setup.c
> ===================================================================
> RCS file: /cvs/src/games/phantasia/setup.c,v
> retrieving revision 1.13
> diff -u -p -u -r1.13 setup.c
> --- setup.c   16 Nov 2014 04:49:48 -0000      1.13
> +++ setup.c   2 Dec 2015 19:26:08 -0000
> @@ -59,9 +59,7 @@ static char *files[] = {            /* all files t
>  char *monsterfile="monsters.asc";
>  
>  int
> -main(argc, argv)
> -     int argc;
> -     char *argv[];
> +main(int argc, char *argv[])
>  {
>       char    **filename;     /* for pointing to file names */
>       int     fd;             /* file descriptor */
> @@ -203,8 +201,7 @@ main(argc, argv)
>  *************************************************************************/
>  
>  void
> -Error(str, file)
> -     char    *str, *file;
> +Error(char *str, char *file)
>  {
>       fprintf(stderr, "Error: ");
>       fprintf(stderr, str, file);
> @@ -236,7 +233,7 @@ Error(str, file)
>  *************************************************************************/
>  
>  double
> -drandom()
> +drandom(void)
>  {
>       return((double) arc4random() / (UINT32_MAX + 1.0));
>  }
> 

Reply via email to