]> granicus.if.org Git - nethack/commitdiff
fix error checking during tile processing
authornhmall <nhmall@nethack.org>
Thu, 28 Oct 2021 20:08:59 +0000 (16:08 -0400)
committernhmall <nhmall@nethack.org>
Thu, 28 Oct 2021 20:08:59 +0000 (16:08 -0400)
noticed following a patch last night

win/share/monsters.txt
win/share/other.txt
win/share/tilemap.c
win/share/tiletext.c

index a23a8c86d96b185321c9e4a8e1d003569a1156e3..deeb7acbd103c1b7a9feea39ed7a12ea99fd77d3 100644 (file)
@@ -10439,7 +10439,7 @@ Z = (195, 195, 195)
   ................
   ................
 }
-# tile 548 (elf-leader,male)
+# tile 548 (elf-noble,male)
 {
   ................
   ................
@@ -10458,7 +10458,7 @@ Z = (195, 195, 195)
   ................
   ................
 }
-# tile 549 (elf-leader,female)
+# tile 549 (elf-noble,female)
 {
   ................
   ................
@@ -10705,7 +10705,7 @@ Z = (195, 195, 195)
   ....LELLLLELAA..
   ................
 }
-# tile 562 (aligned priest,male)
+# tile 562 (aligned cleric,male)
 {
   ................
   INI.............
@@ -10724,7 +10724,7 @@ Z = (195, 195, 195)
   .JACCCJJCCCAA...
   ................
 }
-# tile 563 (aligned priest,female)
+# tile 563 (aligned cleric,female)
 {
   ................
   INI.............
@@ -10743,7 +10743,7 @@ Z = (195, 195, 195)
   .JACCCJJCCCAA...
   ................
 }
-# tile 564 (high priest,male)
+# tile 564 (high cleric,male)
 {
   .INI............
   IIIII.KCCK......
@@ -10762,7 +10762,7 @@ Z = (195, 195, 195)
   ..HACCCJJCCCAA..
   ................
 }
-# tile 565 (high priest,female)
+# tile 565 (high cleric,female)
 {
   .INI............
   IIIII.KCCK......
index e8e2b78ba3430aad77166282c82a1069a8ad1135..d23557e85ca82ca5a7d87f56b7ad7108d92c00ef 100644 (file)
@@ -46,7 +46,7 @@ Z = (195, 195, 195)
   PPAPPPPPPJPPPPPP
   PPPPPPPPPPPPPPPP
 }
-# tile 1 (vertical wall)
+# tile 1 (main walls vertical)
 {
   ANNOA1PPPPP1NNOA
   AOOOA1P1PP11OOOA
@@ -65,7 +65,7 @@ Z = (195, 195, 195)
   A111A11PPPP1111A
   AOPPA11PPPP1OP1A
 }
-# tile 2 (horizontal wall)
+# tile 2 (main walls horizontal)
 {
   AAANOOAAAAANOOAA
   OOONOOA1OOONOOA1
@@ -84,7 +84,7 @@ Z = (195, 195, 195)
   1111111111111111
   AAAAAAAAAAAAAAAA
 }
-# tile 3 (top left corner wall)
+# tile 3 (main walls tlcorn)
 {
   AAANNNNOOOOOOAAA
   NNNNAA11PPPPO111
@@ -103,7 +103,7 @@ Z = (195, 195, 195)
   A111A11PPPP1111A
   AOPPA11PPPP1OP1A
 }
-# tile 4 (top right corner wall)
+# tile 4 (main walls trcorn)
 {
   AAANNNNOOOOOOAAA
   NNNNAA11PPPPO111
@@ -122,7 +122,7 @@ Z = (195, 195, 195)
   A111A11PPPP1111A
   AOPPA11PPPP1OP1A
 }
-# tile 5 (bottom left corner wall)
+# tile 5 (main walls blcorn)
 {
   AAANNNNOOOOOOAAA
   NNNNAA11PPPPO111
@@ -141,7 +141,7 @@ Z = (195, 195, 195)
   1111111111111111
   AAAAAAAAAAAAAAAA
 }
-# tile 6 (bottom right corner wall)
+# tile 6 (main walls brcorn)
 {
   AAANNNNOOOOOOAAA
   NNNNAA11PPPPO111
@@ -160,7 +160,7 @@ Z = (195, 195, 195)
   1111111111111111
   AAAAAAAAAAAAAAAA
 }
-# tile 7 (cross wall)
+# tile 7 (main walls cross wall)
 {
   AAANNNNOOOOOOAAA
   NNNNAA11PPPPO111
@@ -179,7 +179,7 @@ Z = (195, 195, 195)
   A111A11PPPP1111A
   AOPPA11PPPP1OP1A
 }
-# tile 8 (tuwall)
+# tile 8 (main walls tuwall)
 {
   AAANNNNOOOOOOAAA
   NNNNAA11PPPPO111
@@ -198,7 +198,7 @@ Z = (195, 195, 195)
   1111111111111111
   AAAAAAAAAAAAAAAA
 }
-# tile 9 (tdwall)
+# tile 9 (main walls tdwall)
 {
   AAANNNNOOOOOOAAA
   NNNNAA11PPPPO111
@@ -217,7 +217,7 @@ Z = (195, 195, 195)
   A111A11PPPP1111A
   AOPPA11PPPP1OP1A
 }
-# tile 10 (tlwall)
+# tile 10 (main walls tlwall)
 {
   AAANNNNOOOOOOAAA
   NNNNAA11PPPPO111
@@ -236,7 +236,7 @@ Z = (195, 195, 195)
   A111A11PPPP1111A
   AOPPA11PPPP1OP1A
 }
-# tile 11 (trwall)
+# tile 11 (main walls trwall)
 {
   AAANNNNOOOOOOAAA
   NNNNAA11PPPPO111
index 84935b08674e8d35cd79dbc43cce21b1eb425030..77aa3f147250ffcce6bf3a4f5f73de9370474b90 100644 (file)
@@ -215,9 +215,11 @@ tilename(int set, const int file_entry, int gend UNUSED)
                  ++condnum) {
                 if (conditionals[condnum].sequence == MON_GLYPH
                     && conditionals[condnum].predecessor == i) {
-                    tilenum += 2;
-                    if (tilenum == file_entry)
-                        return conditionals[condnum].name;
+                    for (k = 0; k < 2; k++) { /* male and female */
+                        tilenum++;
+                        if (tilenum == file_entry)
+                            return conditionals[condnum].name;
+                    }
                 }
             }
             tilenum++;
index c589d8e8dd105e337603217a0d53fe961ef02c9b..c7d93f8d46eb495aaf748cc494a05ace12f18fa7 100644 (file)
@@ -110,7 +110,7 @@ read_txttile(FILE *txtfile, pixel (*pixels)[TILE_X])
     int ph, i, j, k, reslt;
     char buf[BUFSZ], ttype[BUFSZ], gend[BUFSZ];
     const char *p;
-    char c[2];
+    char c[2], *q;
     static int gidx = 0;
 
     gend[0] = '\0';
@@ -133,11 +133,18 @@ read_txttile(FILE *txtfile, pixel (*pixels)[TILE_X])
      * change when tiles are added
      */
     p = tilename(tile_set, tile_set_indx, gidx);
-    if (p && strcmp(p, buf)
-        && !acceptable_tilename(tile_set, tile_set_indx, buf, p)) {
-        Fprintf(stderr, "warning: for tile %d (numbered %d) of %s,\n",
-                tile_set_indx, i, text_sets[tile_set]);
-        Fprintf(stderr, "\tfound '%s' while expecting '%s'\n", buf, p);
+    if (p && (q = strstr(p, " {")) != 0) {
+        *q = '\0';
+    }
+    if (p && strcmp(p, buf)) {
+        boolean other_mismatch =
+            (tile_set == OTHER_SET
+             && !acceptable_tilename(tile_set, tile_set_indx, buf, p));
+        if (tile_set != OTHER_SET || other_mismatch) {
+            Fprintf(stderr, "warning: for tile %d (numbered %d) of %s,\n",
+                    tile_set_indx, i, text_sets[tile_set]);
+            Fprintf(stderr, "\tfound '%s' while expecting '%s'\n", buf, p);
+        }
     }
     tile_set_indx++;