Interim magic number replacement.

diff --git a/actions.c b/actions.c
index 0f17bea4d6a38ae45e30e9e9c3454bda71785433..a30d7c0fc8f551af25cfcd0366ff1f2f2d6e4a96 100644 (file)
--- a/actions.c
+++ b/actions.c
@@ -967,7 +967,7 @@ static int say(struct command_t *command)
     packed_to_token(command->wd1, word1);
     int wd = (int) get_vocab_id(word1);
     /* FIXME: magic numbers */
     packed_to_token(command->wd1, word1);
     int wd = (int) get_vocab_id(word1);
     /* FIXME: magic numbers */

-    if (wd == XYZZY || wd == PLUGH || wd == PLOVER || wd == 2000 + GIANTWORDS || wd == 2000 + PART) {
+    if (wd == XYZZY || wd == PLUGH || wd == PLOVER || wd == ACTION_WORD(GIANTWORDS) || wd == ACTION_WORD(PART)) {

         /* FIXME: scribbles on the interpreter's command block */
         wordclear(&command->wd2);
         return GO_LOOKUP;
         /* FIXME: scribbles on the interpreter's command block */
         wordclear(&command->wd2);
         return GO_LOOKUP;

diff --git a/advent.h b/advent.h
index 6f66d1ec4414c3be019c606690fe26911f73bf88..670d7e9cab3f916a35d5f17d4405294d34fe0ade 100644 (file)
--- a/advent.h
+++ b/advent.h
@@ -54,6 +54,11 @@
 #define OUTSID(LOC)  (CNDBIT(LOC, COND_ABOVE) || FOREST(LOC))
 #define INDEEP(LOC)  ((LOC) >= LOC_MISTHALL && !OUTSID(LOC))
 #define BUG(x)       bug(x, #x)
 #define OUTSID(LOC)  (CNDBIT(LOC, COND_ABOVE) || FOREST(LOC))
 #define INDEEP(LOC)  ((LOC) >= LOC_MISTHALL && !OUTSID(LOC))
 #define BUG(x)       bug(x, #x)

+#define MOTION_WORD(n)  ((n) + 0)
+#define OBJECT_WORD(n)  ((n) + 1000)
+#define ACTION_WORD(n)  ((n) + 2000)
+#define SPECIAL_WORD(n) ((n) + 3000)
+#define PROMOTE_WORD(n) ((n) + 1000)

 
 enum bugtype {
     SPECIAL_TRAVEL_500_GT_L_GT_300_EXCEEDS_GOTO_LIST,
 
 enum bugtype {
     SPECIAL_TRAVEL_500_GT_L_GT_300_EXCEEDS_GOTO_LIST,

diff --git a/main.c b/main.c
index 077e3a996d200dde709fda3215035b194f5c9c99..3c4d448ab56009f5f505b6aeb5510fe211e7ee5e 100644 (file)
--- a/main.c
+++ b/main.c
@@ -1113,12 +1113,12 @@ L2607:
             wordclear(&command.wd2);
         } else {
             /* FIXME: Magic numbers related to vocabulary */
             wordclear(&command.wd2);
         } else {
             /* FIXME: Magic numbers related to vocabulary */

-            if (!((V1 != 1000 + WATER && V1 != 1000 + OIL) ||
-                  (V2 != 1000 + PLANT && V2 != 1000 + DOOR))) {
+           if (!((V1 != PROMOTE_WORD(WATER) && V1 != PROMOTE_WORD(OIL)) ||
+                 (V2 != PROMOTE_WORD(PLANT) && V2 != PROMOTE_WORD(DOOR)))) {

                 if (AT(V2 - 1000))
                     command.wd2 = token_to_packed("POUR");
             }
                 if (AT(V2 - 1000))
                     command.wd2 = token_to_packed("POUR");
             }

-            if (V1 == 1000 + CAGE && V2 == 1000 + BIRD && HERE(CAGE) && HERE(BIRD))
+           if (V1 == PROMOTE_WORD(CAGE) && V2 == PROMOTE_WORD(BIRD) && HERE(CAGE) && HERE(BIRD))

                 command.wd1 = token_to_packed("CATCH");
         }
 L2620:
                 command.wd1 = token_to_packed("CATCH");
         }
 L2620: