compilation w/o WIZARD

> Michael Allison wrote:
> There are unresolved functions in the trunk if you build without WIZARD
> defined and attempt to link:
Also to a lesser extent, in the 3.4.4 branch.

diff --git a/src/attrib.c b/src/attrib.c
index 2f6c0cc7f12eacc7d46bf1b523f6276345f39907..a36d044b4141d38a30c5fabf45235b6fcfa026d0 100644 (file)
--- a/src/attrib.c
+++ b/src/attrib.c
@@ -675,9 +675,6 @@ char *
 from_what(propidx)
 int propidx;
 {
-       struct obj *obj = (struct obj *)0;
-       int innate = is_innate(propidx);
-
        static char buf[BUFSZ];
 
        buf[0] = '\0';
@@ -686,6 +683,9 @@ int propidx;
         */
 #ifdef WIZARD
        if (wizard) {
+           struct obj *obj = (struct obj *)0;
+           int innate = is_innate(propidx);
+
            if (innate == 2)
                Strcpy(buf, " because of your experience");
            else if (innate == 1)

diff --git a/src/cmd.c b/src/cmd.c
index 577600849f4fc81609d6f19a9d2f15c196624d09..67fde612a9ad6f1e66fbb81bd4ffc02e7a25c665 100644 (file)
--- a/src/cmd.c
+++ b/src/cmd.c
@@ -150,13 +150,13 @@ STATIC_PTR int NDECL(wiz_show_stats);
 STATIC_DCL int NDECL(wiz_port_debug);
 #  endif
 STATIC_PTR int NDECL(wiz_rumor_check);
-# endif
+STATIC_DCL char FDECL(cmd_from_func, (int NDECL((*))));
+# endif /* WIZARD */
 STATIC_PTR int NDECL(enter_explore_mode);
 STATIC_PTR int NDECL(doattributes);
 STATIC_PTR int NDECL(doconduct); /**/
 STATIC_PTR boolean NDECL(minimal_enlightenment);
 
-STATIC_DCL char FDECL(cmd_from_func, (int NDECL((*))));
 STATIC_DCL void FDECL(enlght_line, (const char *,const char *,const char *,char *));
 STATIC_DCL char *FDECL(enlght_combatinc, (const char *,int,int,char *));
 #if defined(UNIX) || defined(SAFERHANGUP)
@@ -1830,95 +1830,6 @@ add_debug_extended_commands()
        }
 }
 
-/* called at startup and after number_pad is twiddled */
-void
-reset_commands(initial)
-boolean initial;
-{
-    static const char
-       sdir[] = "hykulnjb><", sdir_swap_yz[] = "hzkulnjb><",
-       ndir[] = "47896321><", ndir_phone_layout[] = "41236987><";
-    static const int ylist[] = { 'y', 'Y', C('y'), M('y'), M('Y'), M(C('y')) };
-    const struct func_tab *cmdtmp;
-    boolean flagtemp;
-    int c, i, updated = 0;
-
-    if (initial) {
-       updated = 1;
-       for (i = 0; i < SIZE(cmdlist); i++) {
-           c = cmdlist[i].f_char & 0xff;
-           Cmd.commands[c] = &cmdlist[i];
-       }
-       Cmd.num_pad = FALSE;
-       Cmd.pcHack_compat = Cmd.phone_layout = Cmd.swap_yz = FALSE;
-    } else {
-       /* basic num_pad */
-       flagtemp = iflags.num_pad;
-       if (flagtemp != Cmd.num_pad) {
-           Cmd.num_pad = flagtemp;
-           ++updated;
-       }
-       /* swap_yz mode (only applicable for !num_pad) */
-       flagtemp = (iflags.num_pad_mode & 1) ? !Cmd.num_pad : FALSE;
-       if (flagtemp != Cmd.swap_yz) {
-           Cmd.swap_yz = flagtemp;
-           ++updated;
-           /* Cmd.swap_yz has been toggled;
-              perform the swap (or reverse previous one) */
-           for (i = 0; i < SIZE(ylist); i++) {
-               c = ylist[i] & 0xff;
-               cmdtmp = Cmd.commands[c];               /* tmp = [y] */
-               Cmd.commands[c] = Cmd.commands[c + 1];  /* [y] = [z] */
-               Cmd.commands[c + 1] = cmdtmp;           /* [z] = tmp */
-           }
-       }
-       /* MSDOS compatibility mode (only applicable for num_pad) */
-       flagtemp = (iflags.num_pad_mode & 1) ? Cmd.num_pad : FALSE;
-       if (flagtemp != Cmd.pcHack_compat) {
-           Cmd.pcHack_compat = flagtemp;
-           ++updated;
-           /* pcHack_compat has been toggled */
-           c = M('5') & 0xff;
-           cmdtmp = Cmd.commands['5'];
-           Cmd.commands['5'] = Cmd.commands[c];
-           Cmd.commands[c] = cmdtmp;
-           c = M('0') & 0xff;
-           Cmd.commands[c] = Cmd.pcHack_compat ? Cmd.commands['I'] : 0;
-       }
-       /* phone keypad layout (only applicable for num_pad) */
-       flagtemp = (iflags.num_pad_mode & 2) ? Cmd.num_pad : FALSE;
-       if (flagtemp != Cmd.phone_layout) {
-           Cmd.phone_layout = flagtemp;
-           ++updated;
-           /* phone_layout has been toggled */
-           for (i = 0; i < 3; i++) {
-               c = '1' + i;            /* 1,2,3 <-> 7,8,9 */
-               cmdtmp = Cmd.commands[c];               /* tmp = [1] */
-               Cmd.commands[c] = Cmd.commands[c + 6];  /* [1] = [7] */
-               Cmd.commands[c + 6] = cmdtmp;           /* [7] = tmp */
-               c = (M('1') & 0xff) + i;    /* M-1,M-2,M-3 <-> M-7,M-8,M-9 */
-               cmdtmp = Cmd.commands[c];               /* tmp = [M-1] */
-               Cmd.commands[c] = Cmd.commands[c + 6];  /* [M-1] = [M-7] */
-               Cmd.commands[c + 6] = cmdtmp;           /* [M-7] = tmp */
-           }
-       }
-    } /*?initial*/
-
-    if (updated) Cmd.serialno++;
-    Cmd.dirchars = !Cmd.num_pad ? (!Cmd.swap_yz ? sdir : sdir_swap_yz) :
-                       (!Cmd.phone_layout ? ndir : ndir_phone_layout);
-    Cmd.alphadirchars = !Cmd.num_pad ? Cmd.dirchars : sdir;
-
-    Cmd.move_W  = Cmd.dirchars[0];
-    Cmd.move_NW = Cmd.dirchars[1];
-    Cmd.move_N  = Cmd.dirchars[2];
-    Cmd.move_NE = Cmd.dirchars[3];
-    Cmd.move_E  = Cmd.dirchars[4];
-    Cmd.move_SE = Cmd.dirchars[5];
-    Cmd.move_S  = Cmd.dirchars[6];
-    Cmd.move_SW = Cmd.dirchars[7];
-}
-
 STATIC_OVL char
 cmd_from_func(fn)
 int NDECL((*fn));
@@ -1930,14 +1841,6 @@ int NDECL((*fn));
        return 0;
 }
 
-/* decide whether a character (user input keystroke) requests screen repaint */
-boolean
-redraw_cmd(c)
-char c;
-{
-    return (c == C('r') || (Cmd.num_pad && c == C('l')));
-}
-
 static const char template[] = "%-18s %4ld  %6ld";
 static const char count_str[] = "                   count  bytes";
 static const char separator[] = "------------------ -----  ------";
@@ -2189,6 +2092,94 @@ wiz_migrate_mons()
 #define unctrl(c)      ((c) <= C('z') ? (0x60 | (c)) : (c))
 #define unmeta(c)      (0x7f & (c))
 
+/* called at startup and after number_pad is twiddled */
+void
+reset_commands(initial)
+boolean initial;
+{
+    static const char
+       sdir[] = "hykulnjb><", sdir_swap_yz[] = "hzkulnjb><",
+       ndir[] = "47896321><", ndir_phone_layout[] = "41236987><";
+    static const int ylist[] = { 'y', 'Y', C('y'), M('y'), M('Y'), M(C('y')) };
+    const struct func_tab *cmdtmp;
+    boolean flagtemp;
+    int c, i, updated = 0;
+
+    if (initial) {
+       updated = 1;
+       for (i = 0; i < SIZE(cmdlist); i++) {
+           c = cmdlist[i].f_char & 0xff;
+           Cmd.commands[c] = &cmdlist[i];
+       }
+       Cmd.num_pad = FALSE;
+       Cmd.pcHack_compat = Cmd.phone_layout = Cmd.swap_yz = FALSE;
+    } else {
+       /* basic num_pad */
+       flagtemp = iflags.num_pad;
+       if (flagtemp != Cmd.num_pad) {
+           Cmd.num_pad = flagtemp;
+           ++updated;
+       }
+       /* swap_yz mode (only applicable for !num_pad) */
+       flagtemp = (iflags.num_pad_mode & 1) ? !Cmd.num_pad : FALSE;
+       if (flagtemp != Cmd.swap_yz) {
+           Cmd.swap_yz = flagtemp;
+           ++updated;
+           /* Cmd.swap_yz has been toggled;
+              perform the swap (or reverse previous one) */
+           for (i = 0; i < SIZE(ylist); i++) {
+               c = ylist[i] & 0xff;
+               cmdtmp = Cmd.commands[c];               /* tmp = [y] */
+               Cmd.commands[c] = Cmd.commands[c + 1];  /* [y] = [z] */
+               Cmd.commands[c + 1] = cmdtmp;           /* [z] = tmp */
+           }
+       }
+       /* MSDOS compatibility mode (only applicable for num_pad) */
+       flagtemp = (iflags.num_pad_mode & 1) ? Cmd.num_pad : FALSE;
+       if (flagtemp != Cmd.pcHack_compat) {
+           Cmd.pcHack_compat = flagtemp;
+           ++updated;
+           /* pcHack_compat has been toggled */
+           c = M('5') & 0xff;
+           cmdtmp = Cmd.commands['5'];
+           Cmd.commands['5'] = Cmd.commands[c];
+           Cmd.commands[c] = cmdtmp;
+           c = M('0') & 0xff;
+           Cmd.commands[c] = Cmd.pcHack_compat ? Cmd.commands['I'] : 0;
+       }
+       /* phone keypad layout (only applicable for num_pad) */
+       flagtemp = (iflags.num_pad_mode & 2) ? Cmd.num_pad : FALSE;
+       if (flagtemp != Cmd.phone_layout) {
+           Cmd.phone_layout = flagtemp;
+           ++updated;
+           /* phone_layout has been toggled */
+           for (i = 0; i < 3; i++) {
+               c = '1' + i;            /* 1,2,3 <-> 7,8,9 */
+               cmdtmp = Cmd.commands[c];               /* tmp = [1] */
+               Cmd.commands[c] = Cmd.commands[c + 6];  /* [1] = [7] */
+               Cmd.commands[c + 6] = cmdtmp;           /* [7] = tmp */
+               c = (M('1') & 0xff) + i;    /* M-1,M-2,M-3 <-> M-7,M-8,M-9 */
+               cmdtmp = Cmd.commands[c];               /* tmp = [M-1] */
+               Cmd.commands[c] = Cmd.commands[c + 6];  /* [M-1] = [M-7] */
+               Cmd.commands[c + 6] = cmdtmp;           /* [M-7] = tmp */
+           }
+       }
+    } /*?initial*/
+
+    if (updated) Cmd.serialno++;
+    Cmd.dirchars = !Cmd.num_pad ? (!Cmd.swap_yz ? sdir : sdir_swap_yz) :
+                       (!Cmd.phone_layout ? ndir : ndir_phone_layout);
+    Cmd.alphadirchars = !Cmd.num_pad ? Cmd.dirchars : sdir;
+
+    Cmd.move_W  = Cmd.dirchars[0];
+    Cmd.move_NW = Cmd.dirchars[1];
+    Cmd.move_N  = Cmd.dirchars[2];
+    Cmd.move_NE = Cmd.dirchars[3];
+    Cmd.move_E  = Cmd.dirchars[4];
+    Cmd.move_SE = Cmd.dirchars[5];
+    Cmd.move_S  = Cmd.dirchars[6];
+    Cmd.move_SW = Cmd.dirchars[7];
+}
 
 void
 rhack(cmd)
@@ -2446,6 +2437,14 @@ char sym;
        return !u.dz;
 }
 
+/* decide whether a character (user input keystroke) requests screen repaint */
+boolean
+redraw_cmd(c)
+char c;
+{
+    return (c == C('r') || (Cmd.num_pad && c == C('l')));
+}
+
 /*
  * uses getdir() but unlike getdir() it specifically
  * produces coordinates using the direction from getdir()