Magic-number elimination.
authorEric S. Raymond <esr@thyrsus.com>
Sun, 2 Jul 2017 19:32:50 +0000 (15:32 -0400)
committerEric S. Raymond <esr@thyrsus.com>
Sun, 2 Jul 2017 19:32:50 +0000 (15:32 -0400)
actions.c
adventure.yaml

index aa488e018bf39a74bae0f07759761998d216d95f..95a94b821d15c888041e288d483e42933e6559b7 100644 (file)
--- a/actions.c
+++ b/actions.c
@@ -639,7 +639,7 @@ int fill(token_t verb, token_t obj)
         return (discard(verb, obj, true));
     } else if (obj == URN) {
         spk = FULL_URN;
-        if (game.prop[URN] != 0) {
+        if (game.prop[URN] != URN_EMPTY) {
             rspeak(spk);
             return GO_CLEAROBJ;
         }
@@ -652,7 +652,7 @@ int fill(token_t verb, token_t obj)
         game.place[k] = LOC_NOWHERE;
         game.prop[BOTTLE] = EMPTY_BOTTLE;
         if (k == OIL)
-            game.prop[URN] = 1;
+            game.prop[URN] = URN_DARK;
         spk = WATER_URN + game.prop[URN];
         rspeak(spk);
         return GO_CLEAROBJ;
@@ -664,7 +664,7 @@ int fill(token_t verb, token_t obj)
     spk = BOTTLED_WATER;
     if (LIQLOC(game.loc) == 0)
         spk = NO_LIQUID;
-    if (HERE(URN) && game.prop[URN] != 0)
+    if (HERE(URN) && game.prop[URN] != URN_EMPTY)
         spk = URN_NOPOUR;
     if (LIQUID() != 0)
         spk = BOTTLE_FULL;
@@ -974,7 +974,7 @@ static int rub(token_t verb, token_t obj)
     if (obj == URN && game.prop[URN] == URN_LIT) {
         DESTROY(URN);
         drop(AMBER, game.loc);
-        game.prop[AMBER] = 1;
+        game.prop[AMBER] = AMBER_IN_ROCK;
         --game.tally;
         drop(CAVITY, game.loc);
         spk = URN_GENIES;
@@ -992,12 +992,11 @@ static int say(struct command_t *command)
         b = command->wd2x;
         command->wd1 = command->wd2;
     }
-    //int wd = vocab(command->wd1, -1);
     char word1[6];
     packed_to_token(command->wd1, word1);
     int wd = (int) get_vocab_id(word1);
     /* FIXME: magic numbers */
-    if (wd == XYZZY || wd == PLUGH || wd == PLOVER || wd == ACTION_WORD(GIANTWORDS) || wd == ACTION_WORD(PART)) {
+    if (wd == MOTION_WORD(XYZZY) || wd == MOTION_WORD(PLUGH) || wd == MOTION_WORD(PLOVER) || wd == ACTION_WORD(GIANTWORDS) || wd == ACTION_WORD(PART)) {
         /* FIXME: scribbles on the interpreter's command block */
         wordclear(&command->wd2);
         return GO_LOOKUP;
index 8ba4ebeb35c8f64cc81844a496eec92006d8c7e3..243317aa21f4d58c5d1216b8c7055c9c0c34c600 100644 (file)
@@ -3784,8 +3784,8 @@ objects: !!omap
     locations: LOC_NOWHERE
     treasure: true
     descriptions:
-    - 'There is a rare amber gemstone here!'
-    - 'There is an amber gemstone resting in a small cavity in the rock!'
+    - [AMBER_IN_URN, 'There is a rare amber gemstone here!']
+    - [AMBER_IN_ROCK, 'There is an amber gemstone resting in a small cavity in the rock!']
 - SAPPH:
     words: ['sapph']
     inventory: 'Star sapphire'