Comment polishing.
[open-adventure.git] / make_dungeon.py
index 9b729cc2505193b54a81e576ed47bb2c2cd3c253..63eee94ed241863772ee71465c6a25b7d07e7c7c 100755 (executable)
@@ -1,4 +1,4 @@
-#!/usr/bin/python3
+#!/usr/bin/env python
 
 # This is the open-adventure dungeon generator. It consumes a YAML description of
 # the dungeon and outputs a dungeon.h and dungeon.c pair of C code files.
 # The nontrivial part of this is the compilation of the YAML for
 # movement rules to the travel array that's actually used by
 # playermove().
+#
+# Copyright (c) 2017 by Eric S. Raymond
+# SPDX-License-Identifier: BSD-2-clause
 
 import sys, yaml
 
-yaml_name = "adventure.yaml"
-h_name = "dungeon.h"
-c_name = "dungeon.c"
-
-statedefines = ""
-
-h_template = """/* Generated from adventure.yaml - do not hand-hack! */
-#ifndef DUNGEON_H
-#define DUNGEON_H
-
-#include <stdio.h>
-#include <stdbool.h>
-
-#define SILENT -1      /* no sound */
-
-/* Symbols for cond bits */
-#define COND_LIT       0       /* Light */
-#define COND_OILY      1       /* If bit 2 is on: on for oil, off for water */
-#define COND_FLUID     2       /* Liquid asset, see bit 1 */
-#define COND_NOARRR    3       /* Pirate doesn't go here unless following */
-#define COND_NOBACK    4       /* Cannot use "back" to move away */
-#define COND_ABOVE     5
-#define COND_DEEP      6       /* Deep - e.g where dwarves are active */
-#define COND_FOREST    7       /* In the forest */
-#define COND_FORCED    8       /* Only one way in or out of here */
-/* Bits past 10 indicate areas of interest to "hint" routines */
-#define COND_HBASE     10      /* Base for location hint bits */
-#define COND_HCAVE     11      /* Trying to get into cave */
-#define COND_HBIRD     12      /* Trying to catch bird */
-#define COND_HSNAKE    13      /* Trying to deal with snake */
-#define COND_HMAZE     14      /* Lost in maze */
-#define COND_HDARK     15      /* Pondering dark room */
-#define COND_HWITT     16      /* At Witt's End */
-#define COND_HCLIFF    17      /* Cliff with urn */
-#define COND_HWOODS    18      /* Lost in forest */
-#define COND_HOGRE     19      /* Trying to deal with ogre */
-#define COND_HJADE     20      /* Found all treasures except jade */
-
-typedef struct {{
-  const char** strs;
-  const int n;
-}} string_group_t;
-
-typedef struct {{
-  const string_group_t words;
-  const char* inventory;
-  int plac, fixd;
-  bool is_treasure;
-  const char** descriptions;
-  const char** sounds;
-  const char** texts;
-  const char** changes;
-}} object_t;
-
-typedef struct {{
-  const char* small;
-  const char* big;
-}} descriptions_t;
-
-typedef struct {{
-  descriptions_t description;
-  const long sound;
-  const bool loud;
-}} location_t;
-
-typedef struct {{
-  const char* query;
-  const char* yes_response;
-}} obituary_t;
-
-typedef struct {{
-  const int threshold;
-  const int point_loss;
-  const char* message;
-}} turn_threshold_t;
-
-typedef struct {{
-  const int threshold;
-  const char* message;
-}} class_t;
-
-typedef struct {{
-  const int number;
-  const int turns;
-  const int penalty;
-  const char* question;
-  const char* hint;
-}} hint_t;
-
-typedef struct {{
-  const string_group_t words;
-}} motion_t;
-
-typedef struct {{
-  const string_group_t words;
-  const char* message;
-}} action_t;
-
-typedef struct {{
-  const string_group_t words;
-  const char* message;
-}} special_t;
-
-enum condtype_t {{cond_goto, cond_pct, cond_carry, cond_with, cond_not}};
-enum desttype_t {{dest_goto, dest_special, dest_speak}};
-
-typedef struct {{
-  const long motion;
-  const long condtype;
-  const long condarg1;
-  const long condarg2;
-  const enum desttype_t desttype;
-  const long destval;
-  const bool nodwarves;
-  const bool stop;
-}} travelop_t;
-
-/* Abstract out the encoding of words in the travel array.  Gives us
- * some hope of getting to a less cryptic representation than we
- * inherited from FORTRAN, someday. To understand these, read the
- * encoding description for travel.
- */
-#define T_TERMINATE(entry)     ((entry).motion == 1)
-
-extern const location_t locations[];
-extern const object_t objects[];
-extern const char* arbitrary_messages[];
-extern const class_t classes[];
-extern const turn_threshold_t turn_thresholds[];
-extern const obituary_t obituaries[];
-extern const hint_t hints[];
-extern long conditions[];
-extern const motion_t motions[];
-extern const action_t actions[];
-extern const special_t specials[];
-extern const travelop_t travel[];
-extern const long tkey[];
-extern const char *ignore;
-
-#define NLOCATIONS     {}
-#define NOBJECTS       {}
-#define NHINTS         {}
-#define NCLASSES       {}
-#define NDEATHS                {}
-#define NTHRESHOLDS    {}
-#define NMOTIONS        {}
-#define NACTIONS       {}
-#define NSPECIALS       {}
-#define NTRAVEL                {}
-#define NKEYS          {}
-
-#define BIRD_ENDSTATE  {}
-
-enum arbitrary_messages_refs {{
-{}
-}};
-
-enum locations_refs {{
-{}
-}};
-
-enum object_refs {{
-{}
-}};
-
-enum motion_refs {{
-{}
-}};
-
-enum action_refs {{
-{}
-}};
-
-enum special_refs {{
-{}
-}};
-
-/* State definitions */
-
-{}
-#endif /* end DUNGEON_H */
-"""
+YAML_NAME = "adventure.yaml"
+H_NAME = "dungeon.h"
+C_NAME = "dungeon.c"
+H_TEMPLATE_PATH = "templates/dungeon.h.tpl"
+C_TEMPLATE_PATH = "templates/dungeon.c.tpl"
 
-c_template = """/* Generated from adventure.yaml - do not hand-hack! */
+DONOTEDIT_COMMENT = "/* Generated from adventure.yaml - do not hand-hack! */\n\n"
 
-#include "{}"
-
-const char* arbitrary_messages[] = {{
-{}
-}};
-
-const class_t classes[] = {{
-{}
-}};
-
-const turn_threshold_t turn_thresholds[] = {{
-{}
-}};
-
-const location_t locations[] = {{
-{}
-}};
-
-const object_t objects[] = {{
-{}
-}};
-
-const obituary_t obituaries[] = {{
-{}
-}};
-
-const hint_t hints[] = {{
-{}
-}};
-
-long conditions[] = {{
-{}
-}};
-
-const motion_t motions[] = {{
-{}
-}};
-
-const action_t actions[] = {{
-{}
-}};
-
-const special_t specials[] = {{
-{}
-}};
-
-const long tkey[] = {{{}}};
-
-const travelop_t travel[] = {{
-{}
-}};
-
-const char *ignore = \"{}\";
-
-/* end */
-"""
+statedefines = ""
 
 def make_c_string(string):
     """Render a Python string into C string literal format."""
@@ -384,8 +155,8 @@ def get_objects(obj):
             if labels:
                 global statedefines
                 statedefines += "/* States for %s */\n" % item[0]
-                for (i, label) in enumerate(labels):
-                    statedefines += "#define %s\t%d\n" % (label, i)
+                for (n, label) in enumerate(labels):
+                    statedefines += "#define %s\t%d\n" % (label, n)
                 statedefines += "\n"
         sounds_str = ""
         if attr.get("sounds") == None:
@@ -435,7 +206,7 @@ def get_obituaries(obit):
     obit_str = obit_str[:-1] # trim trailing newline
     return obit_str
 
-def get_hints(hnt, arb):
+def get_hints(hnt):
     template = """    {{
         .number = {},
         .penalty = {},
@@ -445,7 +216,6 @@ def get_hints(hnt, arb):
     }},
 """
     hnt_str = ""
-    md = dict(arb)
     for member in hnt:
         item = member["hint"]
         number = item["number"]
@@ -501,40 +271,12 @@ def get_actions(actions):
     template = """    {{
         .words = {},
         .message = {},
+        .noaction = {},
     }},
 """
     act_str = ""
     for action in actions:
         contents = action[1]
-        
-        if contents["words"] == None:
-            words_str = get_string_group([])
-        else:
-            words_str = get_string_group(contents["words"])
-
-        if contents["message"] == None:
-            message = "NO_MESSAGE"
-        else:
-            message = contents["message"]
-            
-        act_str += template.format(words_str, message)
-        global ignore
-        if contents.get("oldstyle", True) == False:
-            for word in contents["words"]:
-                if len(word) == 1:
-                    ignore += word.upper()
-    act_str = act_str[:-1] # trim trailing newline
-    return act_str
-
-def get_specials(specials):
-    template = """    {{
-        .words = {},
-        .message = {},
-    }},
-"""
-    spc_str = ""
-    for special in specials:
-        contents = special[1]
 
         if contents["words"] == None:
             words_str = get_string_group([])
@@ -546,14 +288,19 @@ def get_specials(specials):
         else:
             message = make_c_string(contents["message"])
 
-        spc_str += template.format(words_str, message)
+        if contents.get("noaction") == None:
+            noaction = "false"
+        else:
+            noaction = "true"
+
+        act_str += template.format(words_str, message, noaction)
         global ignore
         if contents.get("oldstyle", True) == False:
             for word in contents["words"]:
                 if len(word) == 1:
                     ignore += word.upper()
-    spc_str = spc_str[:-1] # trim trailing newline
-    return spc_str
+    act_str = act_str[:-1] # trim trailing newline
+    return act_str
 
 def bigdump(arr):
     out = ""
@@ -572,7 +319,7 @@ def buildtravel(locs, objs):
     # THIS CODE IS WAAAY MORE COMPLEX THAN IT NEEDS TO BE.  It's the
     # result of a massive refactoring exercise that concentrated all
     # the old nastiness in one spot. It hasn't been finally simplified
-    # because there's no need to do it until one of the asserions
+    # because there's no need to do it until one of the assertions
     # fails. Hint: if you try cleaning this up, the acceptance test is
     # simple - the output dungeon.c must not change.
     #
@@ -626,7 +373,7 @@ def buildtravel(locs, objs):
             try:
                 return locnames.index(action[1])
             except ValueError:
-                sys.stderr.write("dungeon: unknown location %s in goto clause of %s\n" % (cond[1], name))
+                sys.stderr.write("dungeon: unknown location %s in goto clause of %s\n" % (action[1], name))
         elif action[0] == "special":
             return 300 + action[1]
         elif action[0] == "speak":
@@ -693,7 +440,7 @@ def buildtravel(locs, objs):
 
     # At this point the ltravel data is in the Section 3
     # representation from the FORTRAN version.  Next we perform the
-    # same mapping into wgat used to be the runtime format.
+    # same mapping into what used to be the runtime format.
 
     travel = [[0, "LOC_NOWHERE", 0, 0, 0, 0, 0, 0, "false", "false"]]
     tkey = [0]
@@ -775,7 +522,7 @@ def get_travel(travel):
     return out
 
 if __name__ == "__main__":
-    with open(yaml_name, "r") as f:
+    with open(YAML_NAME, "r") as f:
         db = yaml.load(f)
 
     locnames = [x[0] for x in db["locations"]]
@@ -786,22 +533,32 @@ if __name__ == "__main__":
     (travel, tkey) = buildtravel(db["locations"],
                                  db["objects"])
     ignore = ""
+    try:
+        with open(H_TEMPLATE_PATH, "r") as htf:
+            # read in dungeon.h template
+            h_template = DONOTEDIT_COMMENT + htf.read()
+        with open(C_TEMPLATE_PATH, "r") as ctf:
+            # read in dungeon.c template
+            c_template = DONOTEDIT_COMMENT + ctf.read()
+    except IOError as e:
+        print('ERROR: reading template failed ({})'.format(e.strerror))
+        exit(-1)
+
     c = c_template.format(
-        h_name,
-        get_arbitrary_messages(db["arbitrary_messages"]),
-        get_class_messages(db["classes"]),
-        get_turn_thresholds(db["turn_thresholds"]),
-        get_locations(db["locations"]),
-        get_objects(db["objects"]),
-        get_obituaries(db["obituaries"]),
-        get_hints(db["hints"], db["arbitrary_messages"]),
-        get_condbits(db["locations"]),
-        get_motions(db["motions"]),
-        get_specials(db["actions"]),
-        get_specials(db["specials"]),
-        bigdump(tkey),
-        get_travel(travel), 
-        ignore,
+        h_file             = H_NAME,
+        arbitrary_messages = get_arbitrary_messages(db["arbitrary_messages"]),
+        classes            = get_class_messages(db["classes"]),
+        turn_thresholds    = get_turn_thresholds(db["turn_thresholds"]),
+        locations          = get_locations(db["locations"]),
+        objects            = get_objects(db["objects"]),
+        obituaries         = get_obituaries(db["obituaries"]),
+        hints              = get_hints(db["hints"]),
+        conditions         = get_condbits(db["locations"]),
+        motions            = get_motions(db["motions"]),
+        actions            = get_actions(db["actions"]),
+        tkeys              = bigdump(tkey),
+        travel             = get_travel(travel), 
+        ignore             = ignore
     )
 
     # 0-origin index of birds's last song.  Bird should
@@ -809,31 +566,29 @@ if __name__ == "__main__":
     deathbird = len(dict(db["objects"])["BIRD"]["sounds"]) - 1
 
     h = h_template.format(
-        len(db["locations"])-1,
-        len(db["objects"])-1,
-        len(db["hints"]),
-        len(db["classes"])-1,
-        len(db["obituaries"]),
-        len(db["turn_thresholds"]),
-        len(db["motions"]),
-        len(db["actions"]),
-        len(db["specials"]),
-        len(travel),
-        len(tkey),
-        deathbird,
-        get_refs(db["arbitrary_messages"]),
-        get_refs(db["locations"]),
-        get_refs(db["objects"]),
-        get_refs(db["motions"]),
-        get_refs(db["actions"]),
-        get_refs(db["specials"]),
-        statedefines,
+        num_locations      = len(db["locations"])-1,
+        num_objects        = len(db["objects"])-1,
+        num_hints          = len(db["hints"]),
+        num_classes        = len(db["classes"])-1,
+        num_deaths         = len(db["obituaries"]),
+        num_thresholds     = len(db["turn_thresholds"]),
+        num_motions        = len(db["motions"]),
+        num_actions        = len(db["actions"]),
+        num_travel         = len(travel),
+        num_keys           = len(tkey),
+        bird_endstate      = deathbird,
+        arbitrary_messages = get_refs(db["arbitrary_messages"]),
+        locations          = get_refs(db["locations"]),
+        objects            = get_refs(db["objects"]),
+        motions            = get_refs(db["motions"]),
+        actions            = get_refs(db["actions"]),
+        state_definitions  = statedefines
     )
 
-    with open(h_name, "w") as hf:
+    with open(H_NAME, "w") as hf:
         hf.write(h)
 
-    with open(c_name, "w") as cf:
+    with open(C_NAME, "w") as cf:
         cf.write(c)
 
 # end