Don't declare things in more than one place. Move some declarations

between headers.  Code is now fully prototyped and compiles cleanly
with gcc -Wstrict-prototypes -Wmissing-prototypes -Wnested-externs
-Wredundant-decls.  Closes #723788.

diff --git a/include/combat.h b/include/combat.h
index 5b8286aa70866f2910d4c513b8fb64fbc0052b98..2c9590ca66ca6f022be85dd4bda89a3163ff979c 100644 (file)
--- a/include/combat.h
+++ b/include/combat.h
@@ -97,7 +97,6 @@ extern int att_fight(int, struct combat *, struct emp_qelem *, double,
 extern int att_free_lists(struct emp_qelem *, struct emp_qelem *);
 
 
-extern double combat_mob;
 extern s_char *att_mode[];
 
 #endif /* _COMBAT_H_ */

diff --git a/include/land.h b/include/land.h
index 8906d40b7dfa1632ab820a8c812106dc57961234..47e6aafa19d7d4ed23c0d8642160ad2d86b37af9 100644 (file)
--- a/include/land.h
+++ b/include/land.h
@@ -228,7 +228,4 @@ extern int lnd_support(natid, natid, coord, coord);
 extern int lnd_can_attack(struct lndstr *);
 extern int lnd_fortify (struct lndstr *lp, int hard_amt);
 
-void landdamage(struct lndstr *lp, int dam);
-s_char *prland(struct lndstr *lp);
-
 #endif /* _LAND_H_ */

diff --git a/include/nsc.h b/include/nsc.h
index 6c381975756e92a1eeb16e9964292966c63059ed..d9301905aa4086444bd7038ed3d83000892a99ff 100644 (file)
--- a/include/nsc.h
+++ b/include/nsc.h
@@ -116,8 +116,8 @@ struct nstr_item {
 #define NSC_USHORT     (4<<18) /* pointer to unsigned short */
 #define NSC_INT                (5<<18) /* pointer to int */
 #define        NSC_LONG        (6<<18) /* pointer to long */
-#define NSC_XCOORD     (7<<18) /* pointer to short that needs x conversion */
-#define NSC_YCOORD     (8<<18) /* pointer to short that needs y conversion */
+#define NSC_XCOORD     (7<<18) /* pointer to coord that needs x conversion */
+#define NSC_YCOORD     (8<<18) /* pointer to coord that needs y conversion */
 #define NSC_FLOAT      (9<<18) /* pointer to float */
 #define NSC_CHARP      (10<<18)        /* pointer to string pointer */
 #define NSC_TIME        (11<<18)       /* pointer to time_t */
@@ -161,6 +161,8 @@ extern s_char *nstr_comp(struct nscstr *, int *, int, s_char *);
 extern int encode(register s_char *, long *, int);
 
 
-extern s_char *decodep(register long code, register void *addr);
+extern s_char *decodep(long, void *);
+extern int decode(natid, long, void *, int);
+extern int nstr_exec(struct nscstr *, register int, void *, int);
 
 #endif /* _NSC_H_ */

diff --git a/include/optlist.h b/include/optlist.h
index bc722685c0ce5d1d1b37c986506bafd7a3859ba5..5f164e29eb4b192a11f85d5051a8f72f181317f3 100644 (file)
--- a/include/optlist.h
+++ b/include/optlist.h
@@ -205,10 +205,6 @@ extern long last_demand_update;
 
 /* Game parameters that cannot be set in econfig (yet) */
 extern int melt_item_denom[];
-extern float start_education;
-extern float start_happiness;
-extern float start_technology;
-extern float start_research;
 #ifdef START_UNITS
 int start_unit_type[];
 #endif
@@ -217,13 +213,13 @@ int start_unit_type[];
 extern s_char *upfil;
 extern s_char *downfil;
 extern s_char *disablefil;
-extern s_char *telfil;
-extern s_char *annfil;
-extern s_char *commfil;
 extern s_char *banfil;
 extern s_char *authfil;
+extern s_char *commfil;
+extern s_char *annfil;
 extern s_char *timestampfil;
-/* teldir, telfil, annfil are in tel.h */
+extern s_char *teldir;
+extern s_char *telfil;
 
 struct keymatch {
     s_char *km_key;            /* the key */

diff --git a/include/plane.h b/include/plane.h
index a9db76fc0d078f2a78e81aeec7611e6401056448..7ad79013dbcb352e433bd7829ee2f90e4d380907 100644 (file)
--- a/include/plane.h
+++ b/include/plane.h
@@ -165,10 +165,6 @@ struct shiplook {
 #define PLN_LOAD(b, t) (t ? (b * (logx((double)t, (double)50.0) < 1.0 ? 1.0 : \
                                  logx((double)t, (double)50.0))) : b)
 
-double logx(double d, double base);
-
-s_char *prplane(struct plnstr *pp);
-
 
 /* src/lib/subs/aircombat.c */
 extern void ac_planedamage(struct plist *, natid, int, natid, int,

diff --git a/include/prototypes.h b/include/prototypes.h
index 3a479d6ec76dbe3d8715cc2a9ef0fbbe95e8a103..f9eabc671baccaada7af9fd9ab0a6c3ffc914eb0 100644 (file)
--- a/include/prototypes.h
+++ b/include/prototypes.h
@@ -56,234 +56,32 @@ extern void shutdwn(int sig);
  *****************************************************************************/
 
 /* src/lib/empthread/lwp.c */
-#ifdef _EMPTH_LWP
-extern int empth_init(char **ctx, int flags);
-extern empth_t *empth_create(int prio, void (*entry)(void *), int size,
-                            int flags, char *name, char *desc, void *ud);
-#if 0
-extern void empth_setctx(void *ctx);
-#endif
-extern empth_t *empth_self(void);
-extern void empth_exit(void);
-extern void empth_yield(void);
-extern void empth_terminate(empth_t *a);
-extern void empth_select(int fd, int flags);
-extern void empth_wakeup(empth_t *a);
-extern void empth_sleep(long int until);
-extern empth_sem_t *empth_sem_create(char *name, int cnt);
-extern void empth_sem_signal(empth_sem_t *sm);
-extern void empth_sem_wait(empth_sem_t *sm);
-extern void empth_alarm(int sig);
-#endif /* _EMPTH_LWP */
-
-/* src/lib/empthread/pthread.c */
-#ifdef _EMPTH_POSIX
-extern int empth_init(char **ctx_ptr, int flags);
-extern empth_t *empth_create(int prio, void (*entry)(void *),
-                            int size, int flags,
-                            char *name, char *desc, void *ud);
-extern empth_t *empth_self(void);
-extern void empth_exit(void);
-extern void empth_yield(void);
-extern void empth_terminate(empth_t *a);
-extern void empth_select(int fd, int flags);
-extern void empth_alarm(int sig);
-extern void empth_wakeup(empth_t *a);
-extern void empth_sleep(long until);
-extern empth_sem_t *empth_sem_create(char *name, int cnt);
-extern void empth_sem_signal(empth_sem_t *sm);
-extern void empth_sem_wait(empth_sem_t *sm);
-#endif /* _EMPTH_POSIX */
+/* in empthread.h */
 
 /*
  * src/lib/commands/ *.c 
  */
-extern int acce(void);
-extern int add(void);
-extern int anti(void);
-extern int arm(void);
-extern int army(void);
-extern int assa(void);
-extern int atta(void);
-extern int bdes(void);
-extern int best(void);
-extern int boar(void);
-extern int bomb(void);
-extern int brea(void);
-extern int budg(void);
-extern int buil(void);
-extern int buy(void);
 extern int check_market(void);
-extern int capi(void);
-extern int carg(void);
-extern int lcarg(void);
-extern int cede(void);
-extern int cens(void);
 extern void set_coastal(struct sctstr *);
-extern int chan(void);
-extern int coas(void);
-extern int coll(void);
-extern int comm(void);
-extern int cons(void);
-extern int conv(void);
-extern int coun(void);
-extern int cuto(void);
-extern int decl(void);
-extern int deli(void);
-extern int demo(void);
-extern int desi(void);
-extern int disa(void);
-extern int diss(void);
-extern int dist(void);
-extern int drop(void);
-extern int dump(void);
-extern int echo(void);
-extern int edit(void);
-extern int enab(void);
-extern int enli(void);
-extern int explore(void);
-extern int fina(void);
-extern int flash(void);
 extern int sendmessage(struct natstr *, struct natstr *, char *, int);
-extern int wall(void);
-extern int flee(void);
-extern int fly(void);
-extern int foll(void);
-extern int force(void);
-extern int fort(void);
-extern int fuel(void);
-extern int give(void);
-extern int grin(void);
-extern int hard(void);
-extern int head(void);
-extern int hidd(void);
-extern int improve(void);
-extern int info(void);
-extern int apro(void);
-extern int land(void);
-extern int laun(void);
-extern int ldump(void);
-extern int ledg(void);
-extern int leve(void);
 extern void gift(int, int, s_char *, int, s_char *);
-extern int load(void);
-extern int lload(void);
-extern int look(void);
-extern int llook(void);
-extern int lost(void);
-extern int lsta(void);
-extern int ltend(void);
-extern int map(void);
-extern int march(void);
-extern int mark(void);
 extern int display_mark(s_char *);
-extern int multifire(void);
-extern int mine(void);
-extern int landmine(void);
-extern int mission(void);
-extern int mobq(void);
-extern int mobupdate(void);
-extern int morale(void);
-extern int move(void);
 extern int want_to_abandon(struct sctstr *, int, int, struct lndstr *);
 extern int would_abandon(struct sctstr *, int, int, struct lndstr *);
-extern int mult(void);
-extern int name(void);
-extern int nati(void);
-extern int navi(void);
 extern int nav_map(int, int, int);
-extern int ndump(void);
-extern int new(void);
-extern int newe(void);
-extern int news(void);
-extern int nuke(void);
-extern int offe(void);
-extern int offs(void);
-extern int orde(void);
-extern int qorde(void);
-extern int sorde(void);
-extern int orig(void);
-extern int para(void);
-extern int path(void);
-extern int pdump(void);
-extern int plan(void);
-extern int play(void);
-extern int powe(void);
 extern int count_pop(register int);
-extern int prod(void);
-extern int pstat(void);
-extern int rada(void);
-extern int range(void);
-extern int lrange(void);
-extern int rea(void);
-extern int real(void);
-extern int reco(void);
-extern int reje(void);
-extern int rela(void);
-extern int repa(void);
-extern int repo(void);
-extern int rese(void);
-extern int reso(void);
-extern int retr(void);
-extern int lretr(void);
-extern int rout(void);
-extern int sail(void);
-extern int sate(void);
-extern int scra(void);
 extern int scuttle_tradeship(struct shpstr *, int);
-extern int scut(void);
 extern void scuttle_ship(struct shpstr *);
-extern int sdump(void);
-extern int sct(void);
-extern int sell(void);
-extern int set(void);
-extern int setres(void);
-extern int setsector(void);
 extern void resnoise(struct sctstr *sptr, int public_amt, s_char *name,
                     int old, int new);
-extern int shar(void);
-extern int shark(void);
-extern int shi(void);
-extern int shoo(void);
-extern int show(void);
-extern int shut(void);
-extern int sinfra(void);
-extern int skyw(void);
 extern int line_of_sight(s_char **rad, int ax, int ay, int bx, int by);
-extern int sona(void);
 extern void plane_sona(struct emp_qelem *, int, int, struct shiplook *);
-extern int spy(void);
-extern int sstat(void);
-extern int start(void);
-extern int stop(void);
-extern int stre(void);
-extern int starve(void);
-extern int supp(void);
-extern int surv(void);
-extern int swaps(void);
-extern int tele(void);
-extern int tend(void);
 extern int tend_nxtitem(struct nstr_item *, caddr_t);
-extern int terr(void);
-extern int thre(void);
-extern int togg(void);
 extern s_char *prsub(struct shpstr *);
-extern int torp(void);
-extern int trad(void);
 extern int check_trade(void);
 extern int ontradingblock(int, int *);
 extern void trdswitchown(int, int *, int);
-extern int tran(void);
-extern int trea(void);
-extern int turn(void);
-extern int upda(void);
-extern int upgr(void);
-extern int vers(void);
-extern int wai(void);
-extern int wing(void);
-extern int wipe(void);
-extern int work(void);
-extern int zdon(void);
+/* more in commands.h */
 
 /*
  * src/lib/common/ *.c 
@@ -340,11 +138,9 @@ extern int unit_map(int, int, struct nstr_sect *, s_char *);
 extern int bmaps_intersect(natid, natid);
 extern int share_bmap(natid, natid, struct nstr_sect *, s_char, s_char *);
 /* move.c */
-extern double sector_mcost(struct sctstr *, int);
+/* in path.h */
 /* nstr_subs.c */
-extern s_char *decodep(long, void *);
-extern int decode(natid, long, void *, int);
-extern int nstr_exec(struct nscstr *, register int, void *, int);
+/* in nsc.h */
 /* path.c */
 extern void bp_enable_cachepath(void);
 extern void bp_disable_cachepath(void);
@@ -412,8 +208,6 @@ extern int roll(int);
 extern int roundavg(double);
 extern int chance(double);
 extern void disassoc(void);
-extern s_char *getstarg(s_char *, s_char *, s_char *);
-extern s_char *getstring(s_char *, s_char *);
 #if !defined(_WIN32)
 extern s_char *inet_ntoa(struct in_addr);
 #endif
@@ -422,28 +216,21 @@ extern int diffy(int, int);
 extern int deltax(int, int);
 extern int deltay(int, int);
 extern int mapdist(int, int, int, int);
-extern double dmin(double, double);
 #if !defined(_WIN32)
 extern int max(int, int);
 extern int min(int, int);
 #endif
-extern double dmax(double, double);
-extern s_char *numstr(s_char *, int);
 extern s_char *effadv(int);
 extern int onearg(s_char *, s_char *);
 extern int parse(register s_char *, s_char **, s_char **, s_char *,
                 s_char **);
-extern s_char *esplur(int);
-extern s_char *splur(int);
-extern s_char *iesplur(int);
-extern s_char *plur(int, s_char *, s_char *);
 extern int ldround(double, int);
 extern int roundintby(int, int);
 extern int scthash(register int, register int, int);
 #ifdef NOSTRDUP
 extern char *strdup(char *x);
 #endif
-
+/* more in misc.h */
 
 /*
  * src/lib/global/ *.c 
@@ -471,10 +258,10 @@ extern int dispatch(s_char *, s_char *);
 extern int getcommand(s_char *);
 extern void init_player_commands(void);
 extern void log_last_commands(void);
-extern int explain(void);
 extern int gamedown(void);
 extern void daychange(time_t);
 extern int getminleft(time_t, int *, int *);
+/* more in commands.h */
 /* empmod.c */
 /* init_nats.c */
 extern int init_nats(void);
@@ -487,12 +274,10 @@ extern int natpass(int, s_char *);
 extern struct player *player;  /* current player's context */
 extern s_char *praddr(struct player *);
 extern void player_main(struct player *);
-extern int show_motd(void);
 extern int match_user(char *, struct player *);
 extern int status(void);
 extern int command(void);
-extern int execute(void);
-extern int quit(void);
+/* more in commands.h */
 /* recvclient.c */
 extern int recvclient(s_char *, int);
 
@@ -739,17 +524,10 @@ extern void takeover(register struct sctstr *, natid);
 extern void takeover_ship(register struct shpstr *, natid, int);
 extern void takeover_plane(register struct plnstr *, natid);
 /* trdsub.c */
-extern int trade_check_ok(int, struct trdstr *, union trdgenstr *);
-extern s_char *trade_nameof(struct trdstr *, union trdgenstr *);
-extern int trade_desc(struct trdstr *, union trdgenstr *);
-extern int trade_getitem(struct trdstr *, union trdgenstr *);
-extern long get_couval(int);
-extern long get_outstand(int);
-extern struct ichrstr *whichitem(char);
 /* trechk.c */
 extern int trechk(register natid, register natid, int);
 /* whatitem.c */
-extern struct ichrstr *whatitem(s_char *, s_char *);
+/* in item.h */
 /* wu.c */
 extern void clear_telegram_is_new(natid);
 extern int typed_wu(natid, natid, s_char *, int);
@@ -763,10 +541,7 @@ extern void age_levels(int);
 /* anno.c */
 extern void delete_old_announcements(void);
 /* bp.c */
-extern void fill_update_array(int *, struct sctstr *);
-extern int *get_wp(int *, struct sctstr *, int);
-extern int gt_bg_nmbr(int *, struct sctstr *, int);
-extern void pt_bg_nmbr(int *, struct sctstr *, int, int);
+/* in budg.h */
 /* deliver.c */
 extern int deliver(register struct sctstr *, struct ichrstr *, int, int,
                   int, int);
@@ -782,9 +557,9 @@ extern int feed_people(register int *, int, int *);
 extern int prod_land(int, int, int *, int);
 extern int feed_land(struct lndstr *, register int *, int, int *, int);
 /* main.c */
-extern void update_main(void *);
+/* in server.h */
 /* material.c */
-extern void get_materials(struct sctstr *, int *, int *, int);
+/* in budg.h */
 /* mobility.c */
 extern void mob_sect(register int);
 extern void mob_ship(register int);
@@ -856,7 +631,6 @@ extern int feed_ship(struct shpstr *, register int *, int, int *, int);
  * src/server
  */
 /* shutdown.c */
-extern void shutdown_init(void);
-extern void shutdown_sequence(void *);
+/* in server.h */
 
 #endif /* _PROTOTYPES_H_ */

diff --git a/include/sect.h b/include/sect.h
index dfc0e8c3745b754535b5cbe7ccdf0efa3c51170b..0d81cf6589777816b649b9f051e80f3d5b08c05d 100644 (file)
--- a/include/sect.h
+++ b/include/sect.h
@@ -168,7 +168,6 @@ struct dchrstr {
 
 /* things relating to sectors */
 extern int sctoff(coord x, coord y);
-extern double landgun(int effic, int guns);
 
 extern int sct_maxno;
 extern struct dchrstr dchr[];

diff --git a/include/server.h b/include/server.h
index b233984b6159f5ca238d391f2413a85626b470c3..57b49ed0668a1c67e676ed073a5b88a958f45b5c 100644 (file)
--- a/include/server.h
+++ b/include/server.h
@@ -41,9 +41,9 @@ extern time_t update_time;
 extern int updating_mob;
 
 void mobility_init(void);
+void shutdown_init(void);
 
 /* thread entry points */
-void player_accept(void *);
 void delete_lostitems(void *);
 void market_update(void *);
 void mobility_check(void *);

diff --git a/include/ship.h b/include/ship.h
index 97e20d4e1becea02bf5f11568e2a0817aa979edc..210b1538b79cfe92d8817a10bd4393dbd2339196 100644 (file)
--- a/include/ship.h
+++ b/include/ship.h
@@ -193,8 +193,6 @@ struct mchrstr {
 #define getshipp(n) \
        (struct shpstr *) ef_ptr(EF_SHIP, n)
 
-extern double seagun(int effic, int guns);
-
 extern struct mchrstr mchr[];
 extern int shp_maxno;
 
@@ -216,8 +214,6 @@ struct mlist {
 #define SHP_FIR(b, t) (t ? (b * (logx((double)t, (double)60.0) < 1.0 ? 1.0 : \
                             logx((double)t, (double)60.0))) : b)
 
-double logx(double d, double base);
-
  /* return codes from shp_check_nav */
 #define CN_NAVIGABLE   (0)
 #define CN_LANDLOCKED  (1)

diff --git a/include/tel.h b/include/tel.h
index 866c1b37c957469edc8f5c0563ac5ba9593d7b4e..6f15743155802fb7d46a5d933c6193f62e94c4c9 100644 (file)
--- a/include/tel.h
+++ b/include/tel.h
@@ -53,9 +53,4 @@ struct telstr {
 
 extern s_char *mailbox(s_char *buf, natid cn);
 
-extern s_char *teldir, *telfil;
-
-extern s_char *annfil;
-
-
 #endif /* _TEL_H_ */

diff --git a/include/trade.h b/include/trade.h
index 9beac75cb6014a0a1e76b8b363f68f102c176e97..a63271811e86984c2939ca1d9b8d184a04a7d6bb 100644 (file)
--- a/include/trade.h
+++ b/include/trade.h
@@ -74,7 +74,13 @@ union trdgenstr {
     struct shpstr shp;
 };
 
-s_char *trade_nameof(struct trdstr * tp, union trdgenstr * tgp);
+extern int trade_check_ok(int, struct trdstr *, union trdgenstr *);
+extern s_char *trade_nameof(struct trdstr *, union trdgenstr *);
+extern int trade_desc(struct trdstr *, union trdgenstr *);
+extern int trade_getitem(struct trdstr *, union trdgenstr *);
+extern long get_couval(int);
+extern long get_outstand(int);
+extern struct ichrstr *whichitem(char);
 
 #define gettrade(n, p) \
        ef_read(EF_TRADE, n, (caddr_t)p)

diff --git a/src/lib/commands/rea.c b/src/lib/commands/rea.c
index 4fff09d581ee27121f0a9b29ea8fe9cb19e3e328..2d1311037709b601e00810dbdff95095f054f022 100644 (file)
--- a/src/lib/commands/rea.c
+++ b/src/lib/commands/rea.c
@@ -44,6 +44,7 @@
 #include "file.h"
 #include "tel.h"
 #include "commands.h"
+#include "optlist.h"
 
 #include <stdio.h>
 #include <fcntl.h>

diff --git a/src/lib/common/mailbox.c b/src/lib/common/mailbox.c
index 50c64dfc93419b864ba3f95fd1666608812ba91f..31cf7df1b9079f3d6a986491a2687a485541ef86 100644 (file)
--- a/src/lib/common/mailbox.c
+++ b/src/lib/common/mailbox.c
@@ -33,6 +33,7 @@
 
 #include "misc.h"
 #include "tel.h"
+#include "optlist.h"
 #include <stdio.h>
 
 s_char *

diff --git a/src/lib/lwp/lwpint.h b/src/lib/lwp/lwpint.h
index b5a427f1e9b9ccfb313b2ca9027e67fd5a42b3b5..ba044a349e8d4bede46cc9858a7a550db0fb91f0 100644 (file)
--- a/src/lib/lwp/lwpint.h
+++ b/src/lib/lwp/lwpint.h
@@ -85,7 +85,6 @@ typedef struct {
 /* internal routines */
 void lwpAddTail(struct lwpQueue *, struct lwpProc *);
 struct lwpProc *lwpGetFirst(struct lwpQueue *);
-void lwpReschedule(void);
 void lwpReady(struct lwpProc *);
 void lwpOnalarm(void);
 

diff --git a/src/lib/subs/wu.c b/src/lib/subs/wu.c
index 0803be3038ed4418441bf1cfbae0c5c3d499bdfe..091562d74848c115f2103db7a84831e96e13428b 100644 (file)
--- a/src/lib/subs/wu.c
+++ b/src/lib/subs/wu.c
@@ -43,6 +43,7 @@
 #include "file.h"
 #include "player.h"
 #include "server.h"
+#include "optlist.h"
 #include "prototypes.h"
 
 static struct telstr last_tel[MAXNOC];

diff --git a/src/lib/update/anno.c b/src/lib/update/anno.c
index 06d235b71bed7bf909c5dacb0487dc07445b90c6..e533b36ba3b7e23384b8ac5ef039f496d8fedbf2 100644 (file)
--- a/src/lib/update/anno.c
+++ b/src/lib/update/anno.c
@@ -42,6 +42,7 @@
 #include <unistd.h>
 #endif
 #include "update.h"
+#include "optlist.h"
 #include "common.h"
 
 void

diff --git a/src/lib/update/main.c b/src/lib/update/main.c
index 31d21093a989c08ec272480e15cd26b98d364f2b..4592ef343dc43c1b4109e9c73e5c5925f0f40197 100644 (file)
--- a/src/lib/update/main.c
+++ b/src/lib/update/main.c
@@ -46,6 +46,7 @@
 #include "update.h"
 #include "common.h"
 #include "optlist.h"
+#include "server.h"
 #include <stdlib.h>
 #if !defined(_WIN32)
 #include <sys/time.h>

diff --git a/src/server/shutdown.c b/src/server/shutdown.c
index 5509ba2b944677d613aecda10edc89fbb15734ef..fd483ac72e282cd220ba35bc5100647b71fe68a2 100644 (file)
--- a/src/server/shutdown.c
+++ b/src/server/shutdown.c
@@ -39,6 +39,7 @@
 #include "nat.h"
 #include "file.h"
 #include <time.h>
+#include "server.h"
 #include "prototypes.h"
 
 int shutdown_pending;