home · contact · privacy
Make server config files more readable, their parsing more lenient.
authorChristian Heller <c.heller@plomlompom.de>
Sun, 30 Mar 2014 02:40:58 +0000 (04:40 +0200)
committerChristian Heller <c.heller@plomlompom.de>
Sun, 30 Mar 2014 02:40:58 +0000 (04:40 +0200)
Completely re-wrote server's config file reading system for this. Just
a first step; client's config file system is to follow. Also stumbled
upon multiple issues in the meantime, added them to the TODO.

19 files changed:
README
TODO
confserver/map_object_actions [deleted file]
confserver/map_objects [deleted file]
confserver/world [new file with mode: 0644]
src/common/err_try_fgets.c
src/common/err_try_fgets.h
src/common/readwrite.h
src/server/configfile.c [new file with mode: 0644]
src/server/configfile.h [new file with mode: 0644]
src/server/init.c
src/server/io.c
src/server/io.h
src/server/main.c
src/server/map_object_actions.c
src/server/map_object_actions.h
src/server/map_objects.c
src/server/map_objects.h
src/server/world.h

diff --git a/README b/README
index 5f1be55ae5788963aac9a1f5cf7ad6b8df19ca36..17a58469595d17fe5ef92aaccfad482faf1eda4d 100644 (file)
--- a/README
+++ b/README
@@ -79,23 +79,52 @@ option (like "-s100") to start the recording at the respective turn number.
 Hacking / server internals and configuration
 --------------------------------------------
 
-The movements/actions available to the player and the enemies are defined and
-can be changed in ./confserver/map_object_actions. Each entry consists of a
-first line of a numerical ID used internally to uniquely identify and manage the
-action, a second line of the number of turns the action takes, and a third line
-of a string that maps the action to the game logic to do when it is called.
-Finally, a delimiter line of "%%" ends the entry.
-
-The different map object types, i.e. species (including the player's human one)
-and item types, can be edited in ./confserver/map_objects. Here the first value
-is a numerical ID that represents the object type, the second one describes what
-type this object decomposes to when it gets destroyed/killed, the third value is
-the ASCII character used to represent the object visually on the map, the fourth
-value is the number of hitpoints the object starts with (items are dead and
-start with zero hitpoints, anything else moves), the fifth is the string that
-names the object in the game log, the sixth defines if the object is consumable
-(it is if it is non-zero) and how many hitpoints are gained if it is. Finally,
-the same delimiter as for the map object action definitions file follows.
+The ./confserver/world file defines the map object types and the actions
+available to them. Each object type and action is defined by a multi-line block
+wherein each line sets one attribute of the object type or action.
+
+Here's a typical action definition block:
+
+ACTION 1
+NAME move
+EFFORT 5
+
+A line of "ACTION" followed by a number starts an action definition block and
+sets the action's id for internal use to 1. The number after "EFFORT" determines
+how many turns this action takes for the actor performing it. The string after
+"NAME" names the action. Furthermore, if it is one of "move", "pick_up", "drop"
+or "use", it matches internal functions described by these strings to this
+action. All other names (including "wait") currently are matched to a do-nothing
+wait function.
+
+Here's a typical map object type definition block: 
+
+OBJECT 2
+NAME ZOMBIE
+SYMBOL z
+LIFEPOINTS 3
+CORPSE_ID 5
+CONSUMABLE 0
+
+A line of "OBJECT" followed by a number starts it, and the number sets the
+object type's internal id. The number after "CONSUMABLE" defines the object
+as consumable (and to so many hitpoints gain). The character after "SYMBOL" is
+the one shown on the map to represent to object type. "LIFEPOINTS" is the start
+hitpoints value for this object type and defines it as animate if it is
+non-zero. The string after "NAME" sets the object type's name. "CORPSE_ID" sets
+the id of the object type that objects of this type degrade to if their
+hitpoints drop to zero if they start out as inanimate (what is not implemented
+yet: or if they are inanimate, but are otherwise crushed). Note that the
+"CORPSE_ID" must match the id of an object type defined in the file (before or
+after, it may even be the same).
+
+All this definition block members must be present within a block, but only
+"ACTION" / "OBJECT" must be positioned at the respective blocks' first line,
+the others may appear in whatever order and even multiple times. If a block is
+finished, however, it cannot be re-defined by starting a new block wit the same
+object type or action id. Tokens in this config file are separated by
+whitespace. Single quotes can be put around string values that are to include
+whitespace by themslves.
 
 All source files are thoroughly documented to explain more details of
 plomrogue's internals. The ./roguelike-server executable can be run with a -v
diff --git a/TODO b/TODO
index 85282def52fa92671d94e55476487e4bdfc6ba33..b90cb760869500a4469917c237421ae5bf691744 100644 (file)
--- a/TODO
+++ b/TODO
@@ -7,10 +7,14 @@ BOTH SERVER/CLIENT:
 
 - check for return values of *printf()
 
-- make config files format more readable / self-explanatory
-
 - be more strict and humble when allocating memory from the stack
 
+- in textfield_width(), handle too large lines somehow
+
+- use select() polling where possible for server-client communication
+
+- may err_line store the processed line internally as it does the line count
+
 SERVER:
 
 - implement field of view / line of sight and obstacles for those on the map
@@ -22,8 +26,12 @@ SERVER:
 - for game continuation, replace re-playing of whole record files with loading
   game state snapshots / save files
 
+- why does an MapObjAct id of 0 fail?
+
 CLIENT:
 
 - enable toggling of window borders
 
 - make log scrollable
+
+- implement server-config-file-like parsing for client-config-files, too
diff --git a/confserver/map_object_actions b/confserver/map_object_actions
deleted file mode 100644 (file)
index dc0de83..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-1
-1
-wait
-%%
-2
-5
-move
-%%
-3
-15
-pick_up
-%%
-4
-5
-drop
-%%
-5
-45
-use
-%%
diff --git a/confserver/map_objects b/confserver/map_objects
deleted file mode 100644 (file)
index c258e79..0000000
+++ /dev/null
@@ -1,49 +0,0 @@
-0
-5
-@
-5
-HUMAN
-0
-%%
-1
-4
-a
-1
-ANT
-0
-%%
-2
-5
-z
-3
-ZOMBIE
-0
-%%
-3
-6
-S
-9
-SHOGGOTH
-0
-%%
-4
-4
-#
-0
-DIRT
-0
-%%
-5
-4
-%
-0
-SKELETON
-0
-%%
-6
-4
-m
-0
-MAGIC MEAT
-2
-%%
diff --git a/confserver/world b/confserver/world
new file mode 100644 (file)
index 0000000..01d475a
--- /dev/null
@@ -0,0 +1,68 @@
+ACTION 1
+NAME wait
+EFFORT 1
+
+ACTION 2
+NAME move
+EFFORT 5
+
+ACTION 3
+NAME pick_up
+EFFORT 15
+
+ACTION 4
+NAME drop
+EFFORT 5
+
+ACTION 5
+NAME use
+EFFORT 45
+
+OBJECT 0
+NAME HUMAN
+SYMBOL @
+LIFEPOINTS 5
+CORPSE_ID 5
+CONSUMABLE 0
+
+OBJECT 1
+NAME ANT
+SYMBOL a
+LIFEPOINTS 1
+CORPSE_ID 4
+CONSUMABLE 0
+
+OBJECT 2
+NAME ZOMBIE
+SYMBOL z
+LIFEPOINTS 3
+CORPSE_ID 5
+CONSUMABLE 0
+
+OBJECT 3
+NAME SHOGGOTH
+SYMBOL S
+LIFEPOINTS 9
+CORPSE_ID 6
+CONSUMABLE 0
+
+OBJECT 4
+NAME DIRT
+SYMBOL #
+LIFEPOINTS 0
+CORPSE_ID 4
+CONSUMABLE 0
+
+OBJECT 5
+NAME SKELETON
+SYMBOL %
+LIFEPOINTS 0
+CORPSE_ID 4
+CONSUMABLE 0
+
+OBJECT 6
+NAME 'MAGIC MEAT'
+SYMBOL m
+LIFEPOINTS 0
+CORPSE_ID 4
+CONSUMABLE 3
index 9912f115859a43f78bac5b0e7e9bfb3682216afc..797a442ac1dc4945166a39587fe36c1142d61b61 100644 (file)
@@ -13,7 +13,7 @@
 /* Increments by one on each err_try_fgets() call, servers as a line counter. */
 static uint32_t err_try_fgets_counter = 0;
 
-/* Delimiter to use for err_try_fgets()' 'c' test. */
+/* Delimiter to use for err_try_fgets()' 'd' test. */
 char * err_try_fgets_delim = "";
 
 
@@ -49,8 +49,8 @@ extern void err_line(uint8_t test, char * line, char * intro, char * msg)
 
 
 
-extern void err_try_fgets(char * line, uint32_t linemax, FILE * file,
-                          char * context, char * test)
+extern char *  err_try_fgets(char * line, uint32_t linemax, FILE * file,
+                             char * context, char * test)
 {
     char * err_end   = "File ended unexpectedly.";
     char * err_empty = "Hit empty line where non-empty line was expected.";
@@ -61,7 +61,7 @@ extern void err_try_fgets(char * line, uint32_t linemax, FILE * file,
     char * err_uint8 = "Value is too large. Must be 255 or less.";
     char * f_name = "err_try_fgets()";
     line[0] = '\0';
-    try_fgets(line, linemax + 1, file, f_name);
+    char * fgets_return = try_fgets(line, linemax + 1, file, f_name);
     err_try_fgets_counter++;
     err_line(strchr(test, '0') && !(strlen(line)), line, context, err_end);
     err_line(strchr(test, 'n') && line[strlen(line) - 1] != '\n', line, context,
@@ -71,7 +71,7 @@ extern void err_try_fgets(char * line, uint32_t linemax, FILE * file,
     err_line(strchr(test, 's') && strlen(line) > 2, line, context, err_many);
     err_line(strchr(test, 'd') && strcmp(line, err_try_fgets_delim), line,
              context, err_delim);
-    if (strchr(test, 'i'))
+    if (strchr(test, 'i') || strchr(test, '8'))
     {
         err_line(!(strchr(test, 'f')) && strlen(line) < 2, line, context,
                  err_int);
@@ -87,4 +87,5 @@ extern void err_try_fgets(char * line, uint32_t linemax, FILE * file,
     }
     err_line(strchr(test, '8') && atoi(line) > UINT8_MAX, line, context,
              err_uint8);
+    return fgets_return;
 }
index 6bb07ee4036d852d0f333e0f783e77a086a6b622..7258a882f9a01a652cf5e21c2c506ee0f56860be 100644 (file)
@@ -45,8 +45,8 @@ extern void err_line(uint8_t test, char * line, char * intro, char * msg);
  *       be '+' or '-'; a '+' or '-' without digits following is invalid)
  * '8': check for "line" describing an integer smaller than or equal UINT8_MAX
 */
-extern void err_try_fgets(char * line, uint32_t linemax, FILE * file,
-                          char * context, char * test);
+extern char * err_try_fgets(char * line, uint32_t linemax, FILE * file,
+                            char * context, char * test);
 
 
 
index d616a193a31f95f9982ea7242f97e041ee4b9d50..4aebdbc9b1d1fc3e94cad5b9898d51399da6b310 100644 (file)
@@ -22,7 +22,7 @@ extern void try_fputc(uint8_t c, FILE * file, char * f);
 
 /* Wrapper to calling fgetc() and fgets() from function "f". The return code is
  * returned unless ferror() indicates an error (i.e. to signify an end of file,
- * fgetc() may return an EOF and fgets() a NULL.
+ * fgetc() may return an EOF and fgets() a NULL).
  */
 extern int try_fgetc(FILE * file, char * f);
 extern char * try_fgets(char * line, int size, FILE * file, char * f);
diff --git a/src/server/configfile.c b/src/server/configfile.c
new file mode 100644 (file)
index 0000000..213b313
--- /dev/null
@@ -0,0 +1,450 @@
+/* src/server/configfile.c */
+
+#define _POSIX_C_SOURCE 200809L /* strdup() */
+#include <stddef.h> /* size_t, NULL */
+#include <stdio.h> /* FILE, sprintf() */
+#include <stdint.h> /* uint8_t, uint32_t */
+#include <stdlib.h> /* atoi(), free() */
+#include <string.h> /* strchr(), strcmp(), strdup(), strlen() */
+#include <unistd.h> /* access(), F_OK */
+#include "../common/err_try_fgets.h" /* err_line(), err_try_fgets(),
+                                      * reset_err_try_fgets_counter()
+                                      */
+#include "../common/readwrite.h" /* try_fopen(),try_fclose(),textfile_width() */
+#include "../common/rexit.h" /* exit_err() */
+#include "../common/try_malloc.h" /* try_malloc() */
+#include "cleanup.h" /* set_cleanup_flag(), CLEANUP_MAP_OBJ_DEFS,
+                      * CLEANUP_MAP_OBJ_ACTS
+                      */
+#include "map_object_actions.h" /* struct MapObjAct */
+#include "map_objects.h" /* struct MapObj, struct MapObjDef */
+#include "world.h" /* world global */
+
+
+
+/* Flags defining state of object and action entry reading ((un-)finished, ready
+ * for starting the reading of a new definition etc.
+ */
+enum flag
+{
+    EDIT_STARTED   = 0x01,
+    NAME_SET       = 0x02,
+    EFFORT_SET     = 0x04,
+    CORPSE_ID_SET  = 0x04,
+    SYMBOL_SET     = 0x08,
+    LIFEPOINTS_SET = 0x10,
+    CONSUMABLE_SET = 0x20,
+    READY_ACT = NAME_SET | EFFORT_SET,
+    READY_OBJ = NAME_SET | CORPSE_ID_SET | SYMBOL_SET | LIFEPOINTS_SET
+                | CONSUMABLE_SET
+};
+
+
+
+/* What MapObjDef and MapObjAct structs have in common at their top. Used to
+ * have common functions run over structs of both types.
+ */
+struct EntryHead
+{
+    uint8_t id;
+    struct EntryHead * next;
+};
+
+
+
+/* Many functions working on config file lines / tokens work with these elements
+ * that only change on line change. Makes sense to pass them over together.
+ */
+struct Context {
+    char * line;
+    char * token0;
+    char * token1;
+    char * err_pre;
+};
+
+
+
+/* Return next token from "line" or NULL if none is found. Tokens either a)
+ * start at the first non-whitespace character and end before the next
+ * whitespace character after that; or b) if the first non-whitespace character
+ * is a single quote followed by at least one other single quote some time later
+ * on the line, the token starts after that first single quote and ends before
+ * the second, with the next token_from_line() call starting its token search
+ * after that second quote. The only way to return an empty string (instead of
+ * NULL) as a token is to delimit the token by two succeeding single quotes.
+ * */
+static char * token_from_line(char * line);
+
+/* Determines the end of the token_from_line() token. */
+static void set_token_end(char ** start, char ** limit_char);
+
+/* Get tokens from "context" and, by their order (in the individual context and
+ * in subsequent calls of this function), interpret them as data to write into
+ * the MapObjAct / MapObjDef DB.
+ *
+ * Individual MapObjDef / MapObjAct DB entries are put together line by line
+ * before being written. Writing only happens after all necessary members of an
+ * entry have been assembled, and when additionally a) a new entry is started by
+ * a context->token0 of "ACTION" or "OBJECT"; or b) a NULL context->token0 is
+ * passed. This is interpreted as the end of the MapObjDef / MapObjAct DB read,
+ * so the appropriate cleanup flags are set and test_corpse_ids() is called.
+ */
+static void tokens_into_entries(struct Context * context);
+
+/* Start reading a new DB entry of "size" from tokens in "context" if ->token0
+ * matches "comparand". Set EDIT_STARTED in "flags" to mark beginning of new
+ * entry reading. Check that id of new entry in ->token1 has not already been
+ * used in DB starting at "entry_cmp".
+ */
+static uint8_t new_entry(struct Context * context, char * comparand,
+                         enum flag * flags, size_t size,
+                         struct EntryHead ** entry,
+                         struct EntryHead * entry_cmp);
+
+/* Write DB entry pointed to by "entry" to its appropriate location. */
+static void write_if_entry(struct EntryHead ** entry,
+                           struct EntryHead *** entry_p_p_p);
+
+/* Ensure that all .corpse_id members in the MapObjDef DB fit .id members of
+ * MapObjDef DB entries.
+ */
+static void test_corpse_ids();
+
+/* Try to read tokens in "context" as members for the entry currently edited,
+ * which must be either "mod" or "moa". What member of which of the two is set
+ * depends on which of "object_flags" and "action_flags" has EDIT_STARTED set
+ * and on the key name of ->token0. Return 1 if interpretation succeeds, else 0.
+ *
+ * Note that MapObjAct entries' .name also determines their .func.
+ */
+static uint8_t set_members(struct Context * context, enum flag * object_flags,
+                           enum flag * action_flags, struct MapObjDef * mod,
+                           struct MapObjAct * moa);
+
+/* If "context"->token0 fits "comparand", set "element" to value read from
+ * ->token1 as either string (type: "s"), char ("c") or uint8 ("8"), set
+ * that element's flag to "flags" and return 1; else return 0.
+ */
+static uint8_t set_val(struct Context * context, char * comparand,
+                       enum flag * flags, enum flag set_flag, char type,
+                       char * element);
+
+/* Writes "context"->token1 to "target" only if it describes a proper uint8. */
+static void set_uint8(struct Context * context, uint8_t * target);
+
+/* If "name" fits "moa"->name, set "moa"->func to "func". (Derives MapObjAct
+ * .func from .name for set_members().
+ */
+static uint8_t try_func_name(struct MapObjAct * moa,
+                             char * name, void (* func) (struct MapObj *));
+
+
+
+static char * token_from_line(char * line)
+{
+    static char * final_char = NULL;
+    static char * limit_char = NULL;
+    char * start = limit_char + 1;
+    if (line)
+    {
+        start      = line;
+        limit_char = start;
+        final_char = &(line[strlen(line)]);
+        if ('\n' == *(final_char - 1))
+        {
+            *(--final_char) = '\0';
+        }
+    }
+    uint8_t empty = 1;
+    uint32_t i;
+    for (i = 0; '\0' != start[i]; i++)
+    {
+        if (' ' != start[i] && '\t' != start[i])
+        {
+            start = &start[i];
+            empty = 0;
+            break;
+        }
+    }
+    if (empty)
+    {
+        return start = NULL;
+    }
+    set_token_end(&start, &limit_char);
+    return start;
+}
+
+
+
+static void set_token_end(char ** start, char ** limit_char)
+{
+    char * end_quote = ('\'' == (*start)[0]) ? strchr(*start + 1, '\'') : NULL;
+    *start = (end_quote) ? *start + 1 : *start;
+    if (end_quote)
+    {
+        *end_quote = '\0';
+        *limit_char = end_quote;
+        return;
+    }
+    char * space = strchr(*start, ' ');
+    char * tab   = strchr(*start, '\t');
+    space = (!space || (tab && tab < space)) ? tab : space;
+    if (space)
+    {
+        * space = '\0';
+    }
+    *limit_char = strchr(*start, '\0');
+}
+
+
+
+static void tokens_into_entries(struct Context * context)
+{
+    char * str_act = "ACTION";
+    char * str_obj = "OBJECT";
+    static struct MapObjAct ** moa_p_p = &world.map_obj_acts;
+    static struct MapObjDef ** mod_p_p = &world.map_obj_defs;
+    static enum flag action_flags = READY_ACT;
+    static enum flag object_flags = READY_OBJ;
+    static struct EntryHead * moa = NULL;
+    static struct EntryHead * mod = NULL;
+    if (   !context->token0
+        || !strcmp(context->token0,str_act) || !strcmp(context->token0,str_obj))
+    {
+        char * err_fin = "Last definition block not finished yet.";
+        err_line((action_flags & READY_ACT) ^ READY_ACT,
+                 context->line, context->err_pre, err_fin);
+        err_line((object_flags & READY_OBJ) ^ READY_OBJ,
+                 context->line, context->err_pre, err_fin);
+        object_flags = action_flags = READY_OBJ;
+        write_if_entry(&moa, (struct EntryHead ***) &moa_p_p);
+        write_if_entry(&mod, (struct EntryHead ***) &mod_p_p);
+    }
+    if (!context->token0)
+    {
+        set_cleanup_flag(CLEANUP_MAP_OBJECT_ACTS | CLEANUP_MAP_OBJECT_DEFS);
+        test_corpse_ids();
+        return;
+    }
+    if (!(   new_entry(context, str_act, &action_flags,
+                       sizeof(struct MapObjAct), (struct EntryHead**) &moa,
+                       (struct EntryHead *) world.map_obj_acts)
+          || new_entry(context, str_obj, &object_flags,
+                       sizeof(struct MapObjDef), (struct EntryHead**) &mod,
+                       (struct EntryHead *) world.map_obj_defs)
+          || set_members(context, &object_flags, &action_flags,
+                         (struct MapObjDef *) mod, (struct MapObjAct *) moa)))
+    {
+        err_line(1, context->line, context->err_pre, "Unknown argument");
+    }
+}
+
+
+
+static uint8_t new_entry(struct Context * context, char * comparand,
+                         enum flag * flags, size_t size,
+                         struct EntryHead ** entry,struct EntryHead * entry_cmp)
+{
+    char * f_name = "new_entry()";
+    char * err_uni = "Declaration of ID already used.";
+    if (!strcmp(context->token0, comparand))
+    {
+        * flags = EDIT_STARTED;
+        * entry = try_malloc(size, f_name);
+        set_uint8(context, &((*entry)->id));
+        for (; NULL != entry_cmp; entry_cmp = entry_cmp->next)
+        {
+            err_line((*entry)->id == entry_cmp->id,
+                     context->line, context->err_pre, err_uni);
+        }
+        return 1;
+    }
+    return 0;
+}
+
+
+
+static void write_if_entry(struct EntryHead ** entry,
+                           struct EntryHead *** entry_p_p_p)
+{
+    if (*entry)
+    {
+        (* entry)->next = NULL;
+        ** entry_p_p_p = *entry;
+        * entry_p_p_p = &((*entry)->next);
+        * entry = NULL;  /* So later runs of this don't re-append same entry. */
+    }
+}
+
+
+
+static void test_corpse_ids()
+{
+    char * f_name = "test_corpse_ids()";
+    char * err_corpse_prefix = "In the object definition DB, one object corpse "
+                               "ID does not reference any known object in the "
+                               "DB. ID of responsible object: ";
+    char * err_corpse = try_malloc(strlen(err_corpse_prefix) + 3 + 1, f_name);
+    struct MapObjDef * test_entry_0 = world.map_obj_defs;
+    for (; test_entry_0; test_entry_0 = test_entry_0->next)
+    {
+        uint8_t corpse_id_found = 0;
+        struct MapObjDef * test_entry_1 = world.map_obj_defs;
+        for (; test_entry_1; test_entry_1 = test_entry_1->next)
+        {
+            if (test_entry_0->corpse_id == test_entry_1->id)
+            {
+                corpse_id_found = 1;
+            }
+        }
+        sprintf(err_corpse, "%s%d", err_corpse_prefix, test_entry_0->id);
+        exit_err(!corpse_id_found, err_corpse);
+    }
+    free(err_corpse);
+}
+
+
+
+static uint8_t set_members(struct Context * context, enum flag * object_flags,
+                           enum flag * action_flags, struct MapObjDef * mod,
+                           struct MapObjAct * moa)
+{
+    if (   * action_flags & EDIT_STARTED
+        && set_val(context, "NAME", action_flags,
+                   NAME_SET, 's', (char *) &moa->name))
+    {
+        if (!(   try_func_name(moa, "move", actor_move)
+              || try_func_name(moa, "pick_up", actor_pick)
+              || try_func_name(moa, "drop", actor_drop)
+              || try_func_name(moa, "use", actor_use)))
+        {
+            moa->func = actor_wait;
+        }
+        *action_flags = *action_flags | NAME_SET;
+        return 1;
+    }
+    else if (   set_val(context, "NAME", object_flags,
+                        NAME_SET, 's', (char *) &mod->name)
+             || set_val(context, "SYMBOL", object_flags,
+                        SYMBOL_SET, 'c', (char *) &mod->char_on_map)
+             || set_val(context, "EFFORT", action_flags,
+                        EFFORT_SET, '8', (char *) &moa->effort)
+             || set_val(context, "LIFEPOINTS", object_flags,
+                        LIFEPOINTS_SET, '8', (char *) &mod->lifepoints)
+             || set_val(context, "CONSUMABLE", object_flags,
+                        CONSUMABLE_SET, '8', (char *) &mod->consumable)
+             || set_val(context, "CORPSE_ID", object_flags,
+                        CORPSE_ID_SET, '8', (char *) &mod->corpse_id))
+    {
+        return 1;
+    }
+    return 0;
+}
+
+
+
+static uint8_t set_val(struct Context * context, char * comparand,
+                       enum flag * flags, enum flag set_flag, char type,
+                       char * element)
+{
+    char * err_out = "Outside appropriate definition's context.";
+    char * err_singlechar = "Value must be single ASCII character.";
+    if (!strcmp(context->token0, comparand))
+    {
+        err_line(!(* flags & EDIT_STARTED),
+                 context->line, context->err_pre, err_out);
+        * flags = * flags | set_flag;
+        if      ('s' == type)
+        {
+            * (char **) element = strdup(context->token1);
+        }
+        else if ('c' == type)
+        {
+            err_line(1 != strlen(context->token1),
+                     context->line, context->err_pre, err_singlechar);
+            * element = (context->token1)[0];
+        }
+        else if ('8' == type)
+        {
+            set_uint8(context, (uint8_t *) element);
+        }
+        return 1;
+    }
+    return 0;
+}
+
+
+
+static void set_uint8(struct Context * context, uint8_t * target)
+{
+    char * err_uint8 = "Value not unsigned decimal number between 0 and 255.";
+    uint8_t i;
+    uint8_t is_uint8 = 1;
+    for (i = 0; '\0' != context->token1[i]; i++)
+    {
+        if (i > 2 || '0' > context->token1[i] || '9' < context->token1[i])
+        {
+            is_uint8 = 0;
+        }
+    }
+    if (is_uint8 && atoi(context->token1) > UINT8_MAX)
+    {
+        is_uint8 = 0;
+    }
+    err_line(!(is_uint8), context->line, context->err_pre, err_uint8);
+    *target = atoi(context->token1);
+}
+
+
+
+static uint8_t try_func_name(struct MapObjAct * moa,
+                             char * name, void (* func) (struct MapObj *))
+{
+    if (0 == strcmp(moa->name, name))
+    {
+        moa->func = func;
+        return 1;
+    }
+    return 0;
+}
+
+
+
+extern void read_config_file() {
+    char * f_name = "read_new_config_file()";
+    char * path = world.path_config;
+    struct Context context;
+    char * err_pre_prefix = "Failed reading config file: \"";
+    char * err_pre_affix = "\". ";
+    context.err_pre = try_malloc(strlen(err_pre_prefix) + strlen(path)
+                                 + strlen(err_pre_affix) + 1, f_name);
+    sprintf(context.err_pre, "%s%s%s", err_pre_prefix, path, err_pre_affix);
+    exit_err(access(path, F_OK), context.err_pre);
+    FILE * file = try_fopen(path, "r", f_name);
+    uint32_t linemax = textfile_width(file);
+    context.line = try_malloc(linemax + 1, f_name);
+    reset_err_try_fgets_counter();
+    err_line(0 == linemax, context.line, context.err_pre, "File is empty.");
+    context.token0 = NULL; /* For tokens_into_entries() if while() stagnates. */
+    char * err_val = "No value given.";
+    char * err_many = "Too many values.";
+    while (err_try_fgets(context.line, linemax + 1, file, context.err_pre, ""))
+    {
+        char * line_copy = strdup(context.line);
+        context.token0 = token_from_line(line_copy);
+        if (context.token0)
+        {
+            err_line(0 == (context.token1 = token_from_line(NULL)),
+                     context.line, context.err_pre, err_val);
+            err_line(NULL != token_from_line(NULL),
+                     context.line, context.err_pre, err_many);
+            tokens_into_entries(&context);
+            context.token0 = NULL;
+        }
+        free(line_copy);
+    }
+    tokens_into_entries(&context);
+    try_fclose(file, f_name);
+    free(context.line);
+    free(context.err_pre);
+}
diff --git a/src/server/configfile.h b/src/server/configfile.h
new file mode 100644 (file)
index 0000000..5fc2a0d
--- /dev/null
@@ -0,0 +1,18 @@
+/* src/server/configfile.h
+ *
+ * Reading in the config file of map object and map object action definitions.
+ */
+
+#ifndef CONFIGFILE_H
+#define CONFIGFILE_H
+
+
+
+/* Parse file at world.path_config into map object and map object action
+ * definitions at world.map_obj_defs and world.map_obj_acts.
+ */
+extern void read_config_file();
+
+
+
+#endif
index 3a83c34025d53ff56f74b23bb423a368fb9ee306..9b3f86b016c2e87154694503d2f77486dff0c652 100644 (file)
@@ -1,13 +1,14 @@
 /* src/server/init.c */
 
-#define _POSIX_C_SOURCE 2 /* getopt(), optarg */
+//#define _POSIX_C_SOURCE 2 /* getopt(), optarg */
+#define _POSIX_C_SOURCE 200809L /* getopt(), optarg, strdup() */
 #include "init.h"
 #include <errno.h> /* global errno, EEXIST */
 #include <stddef.h> /* NULL */
 #include <stdint.h> /* uint32_t */
-#include <stdio.h> /* sprintf(), fflush() */
-#include <stdlib.h> /* exit(), free() */
-#include <string.h> /* atoi(), strlen() */
+#include <stdio.h> /* FILE, sprintf(), fflush() */
+#include <stdlib.h> /* exit(), free(), atoi() */
+#include <string.h> /* strlen() */
 #include <sys/stat.h> /* mkdir() */
 #include <sys/types.h> /* defines pid_t, time_t */
 #include <time.h> /* time() */
                                   */
 #include "../common/rexit.h" /* exit_err() */
 #include "../common/try_malloc.h" /* try_malloc() */
-#include "cleanup.h" /* set_cleanup_flag(), CLEANUP_MAP_OBJ_DEFS,
-                      * CLEANUP_MAP_OBJ_ACTS
-                      */
-#include "io.h" /* read_config_file(), struct EntrySkeleton */
+#include "cleanup.h" /* set_cleanup_flag() */
+#include "configfile.h" /* read_config_file() */
 #include "map.h" /* init_map() */
-#include "map_object_actions.h" /* struct MapObjAct, read_map_object_action() */
-#include "map_objects.h" /* struct MapObjDef, free_map_objects(),
-                          * add_map_objects(), read_map_object_def()
-                          */
+#include "map_objects.h" /* free_map_objects(), add_map_objects() */
 #include "rrand.h" /* rrand() */
 #include "run.h" /* obey_msg(), io_loop() */
 #include "world.h" /* global world */
@@ -64,16 +60,7 @@ extern void init_map_and_map_objects_configs()
     world.map.size.y = 64;
     world.map.dist_orthogonal = 5;
     world.map.dist_diagonal   = 7;
-    char * err_mod = "No map object definitions file.";
-    char * err_moa = "No map object actions file.";
-    exit_err(access(world.path_map_obj_defs, F_OK), err_mod);
-    exit_err(access(world.path_map_obj_acts, F_OK), err_moa);
-    read_config_file(world.path_map_obj_defs, CLEANUP_MAP_OBJECT_DEFS,
-                     read_map_object_def, sizeof(struct MapObjDef),
-                     (struct EntrySkeleton **) &world.map_obj_defs);
-    read_config_file(world.path_map_obj_acts, CLEANUP_MAP_OBJECT_ACTS,
-                     read_map_object_action, sizeof(struct MapObjAct),
-                     (struct EntrySkeleton **) &world.map_obj_acts);
+    read_config_file();
 }
 
 
index f318ca6c15d8716c5b3eb6394a6216bf0996973c..28c76b8f558754b5c377cd71ccd816a357e242c0 100644 (file)
@@ -5,21 +5,17 @@
 #include <limits.h> /* PIPE_BUF */
 #include <stddef.h> /* size_t, NULL */
 #include <stdint.h> /* uint8_t, uint32_t */
-#include <stdio.h> /* defines EOF, FILE, sprintf(), ungetc() */
-#include <stdlib.h> /* free(), atoi() */
+#include <stdio.h> /* defines EOF, FILE, sprintf() */
+#include <stdlib.h> /* free() */
 #include <string.h> /* strlen(), memcpy() */
 #include <sys/types.h> /* time_t */
 #include <time.h> /* time() */
-#include "../common/err_try_fgets.h" /* err_try_fgets(), err_line(),
-                                      * reset_err_try_fgets_counter()
-                                      */
+#include "../common/err_try_fgets.h" /* err_line() */
 #include "../common/readwrite.h" /* try_fopen(), try_fclose_unlink_rename(),
-                                  * try_fwrite(), try_fputc(), try_fgetc(),
-                                  * try_fclose()
+                                  * try_fwrite(), try_fputc(), try_fgetc()
                                   */
-#include "../common/rexit.h" /* exit_trouble() */
 #include "../common/try_malloc.h" /* try_malloc() */
-#include "cleanup.h" /* set_cleanup_flag(), enum cleanup_flag */
+#include "cleanup.h" /* set_cleanup_flag() */
 #include "map_objects.h" /* structs MapObj, MapObjDef, get_map_obj_def() */
 #include "world.h" /* global world  */
 
@@ -223,49 +219,6 @@ static void write_map(FILE * file)
 
 
 
-extern void read_config_file(char * path, enum cleanup_flag cleanup,
-                             void (* read) (char *, uint32_t, char *,
-                                            struct EntrySkeleton *, FILE *),
-                             size_t size, struct EntrySkeleton ** entry_start)
-{
-    char * f_name = "init_map_object_defs()";
-    char * context_prefix = "Failed reading config file: ";
-    char context[strlen(context_prefix) + strlen(path) + 1];
-    sprintf(context, "%s%s", context_prefix, path);
-    char * err_uniq = "Declaration of ID already used.";
-    FILE * file = try_fopen(path, "r", f_name);
-    uint32_t linemax = textfile_width(file);
-    char line[linemax + 1];
-    reset_err_try_fgets_counter();
-    struct EntrySkeleton ** entry_ptr_ptr = entry_start;
-    while (1)
-    {
-        int test_for_end = try_fgetc(file, f_name);
-        if (EOF == test_for_end || '\n' == test_for_end)
-        {
-            break;
-        }
-        exit_trouble(EOF == ungetc(test_for_end, file), f_name, "ungetc()");
-        err_try_fgets(line, linemax, file, context, "nfi8");
-        struct EntrySkeleton * entry = try_malloc(size, f_name);
-        entry->id = atoi(line);
-        struct EntrySkeleton * entry_test = * entry_start;
-        for (; NULL != entry_test; entry_test = entry_test->next)
-        {
-            err_line(entry->id == entry_test->id, line, context, err_uniq);
-        }
-        read(line, linemax, context, entry, file);
-        entry->next = NULL;
-        * entry_ptr_ptr = entry;
-        entry_ptr_ptr = &entry->next;
-        err_try_fgets(line, linemax, file, context, "d");
-    }
-    try_fclose(file, f_name);
-    set_cleanup_flag(cleanup);
-}
-
-
-
 extern char * io_round()
 {
     char * f_name = "io_round()";
index ad1344fcabf9d9bfee69e2a290198b6f1a124ad8..02f8e9017d2dadc7c2bc6b6581f6b8a3f0d4193c 100644 (file)
@@ -7,28 +7,8 @@
 #ifndef IO_H
 #define IO_H
 
-#include <stddef.h> /* size_t */
-#include <stdint.h> /* uint8_t, uint32_t */
-#include <stdio.h> /* FILE */
-#include "cleanup.h" /* enum cleanup_flag */
 
 
-
-/* Wrapper to reading in config files from "path" for DB entries of "size
- * starting at "entry_start", to be unset by "cleanup" and reading in individual
- * entry data line by line via "read"(). Assumes all entries start with the
- * items collected in the EntrySkeleton struct.
- */
-struct EntrySkeleton
-{
-    uint8_t id;
-    struct EntrySkeleton * next;
-};
-extern void read_config_file(char * path, enum cleanup_flag cleanup,
-                             void (* read) (char *, uint32_t, char *,
-                                            struct EntrySkeleton *, FILE *),
-                             size_t size, struct EntrySkeleton ** entry_start);
-
 /* Return single \0-terminated string read from input queue (world.queue); or,
  * if queue is empty and world.turn is unequal world.last_update_turn, update
  * world state file at world.path_worldstate (and update world.last_update_turn
index 9bfddaecb4bb6d6a21ed5dc6435f55ddacaa4172..3c8829b7067ec010267382f27d80b04d9da47bcf 100644 (file)
@@ -35,8 +35,7 @@ int main(int argc, char ** argv)
             exit_err(-1 == test, printf_err);
         }
     }
-    world.path_map_obj_defs = "confserver/map_objects";
-    world.path_map_obj_acts = "confserver/map_object_actions";
+    world.path_config       = "confserver/world";
     world.path_worldstate   = "server/worldstate";
     world.path_out          = "server/out";
     world.path_in           = "server/in";
index cd233c93940f5f0a06220c914e512844df8385f1..987f13581eb86d9a51d521e712035b06763ba4ca 100644 (file)
@@ -3,14 +3,12 @@
 #include "map_object_actions.h"
 #include <stddef.h> /* NULL */
 #include <stdint.h> /* uint8_t, uint16_t */
-#include <stdio.h> /* sprintf(), ungetc() */
-#include <stdlib.h> /* free(), atoi() */
+#include <stdio.h> /* sprintf() */
+#include <stdlib.h> /* free() */
 #include <string.h> /* strlen(), strcmp(), memcpy(), strncmp() */
-#include "../common/err_try_fgets.h" /* err_try_fgets() */
 #include "../common/rexit.h" /* exit_err() */
 #include "../common/try_malloc.h" /* try_malloc() */
 #include "../common/yx_uint8.h" /* struct yx_uint8 */
-#include "io.h" /* struct EntrySkeleton */
 #include "map_objects.h" /* structs MapObj, MapObjDef, get_player(),
                           * set_object_position(), own_map_object(),
                           * get_map_object_def()
@@ -25,8 +23,8 @@
 static void update_log(char * text);
 
 /* If "name" fits "moa"->name, set "moa"->func to "func". */
-static uint8_t try_func_name(struct MapObjAct * moa,
-                             char * name, void (* func) (struct MapObj *));
+//static uint8_t try_func_name(struct MapObjAct * moa,
+//                             char * name, void (* func) (struct MapObj *));
 
 /* One actor "wounds" another actor, decrementing his lifepoints and, if they
  * reach zero in the process, killing it. Generates appropriate log message.
@@ -88,19 +86,6 @@ static void update_log(char * text)
 
 
 
-static uint8_t try_func_name(struct MapObjAct * moa,
-                             char * name, void (* func) (struct MapObj *))
-{
-    if (0 == strcmp(moa->name, name))
-    {
-        moa->func = func;
-        return 1;
-    }
-    return 0;
-}
-
-
-
 static void actor_hits_actor(struct MapObj * hitter, struct MapObj * hitted)
 {
     struct MapObjDef * mod_hitter = get_map_object_def(hitter->type);
@@ -222,29 +207,6 @@ static void playerbonus_use(uint8_t no_object, uint8_t wrong_object)
 
 
 
-extern void read_map_object_action(char * line, uint32_t linemax,char * context,
-                                   struct EntrySkeleton * entry, FILE * file)
-{
-    char * f_name = "init_map_object_actions()";
-    struct MapObjAct * moa = (struct MapObjAct *) entry;
-    err_try_fgets(line, linemax, file, context, "0nfi8");
-    moa->effort = atoi(line);
-    err_try_fgets(line, linemax, file, context, "0nf");
-    line[strlen(line) - 1] = '\0';
-    uint8_t len_name = strlen(line) + 1;
-    moa->name = try_malloc(len_name, f_name);
-    memcpy(moa->name, line, len_name);
-    if (!(   try_func_name(moa, "move", actor_move)
-          || try_func_name(moa, "pick_up", actor_pick)
-          || try_func_name(moa, "drop", actor_drop)
-          || try_func_name(moa, "use", actor_use)))
-    {
-        moa->func = actor_wait;
-    }
-}
-
-
-
 extern void free_map_object_actions(struct MapObjAct * moa)
 {
     if (NULL == moa)
index 7b5283fc9cff33f4254626ca79c3b686cb9c2dab..81a3fa62e5dd3de8daf0bbc9d81b8860bc0ed84e 100644 (file)
@@ -8,10 +8,8 @@
 #ifndef MAP_OBJECT_ACTIONS_H
 #define MAP_OBJECT_ACTIONS_H
 
-#include <stdint.h> /* uint8_t, uint23_t */
-#include <stdio.h> /* FILE */
+#include <stdint.h> /* uint8_t */
 struct MapObj;
-struct EntrySkeleton;
 
 
 
@@ -26,12 +24,6 @@ struct MapObjAct
 
 
 
-/* Read-in to "entry" multi-line entry from MapObjAct config "file", using
- * pre-allocated "line", "linemax" and "context" as input for err_try_fgets().
- */
-extern void read_map_object_action(char * line, uint32_t linemax,char * context,
-                                   struct EntrySkeleton * entry, FILE * file);
-
 /* Free MapObjAct * chain starting at "moa". */
 extern void free_map_object_actions(struct MapObjAct * moa);
 
index ae3fbd1884146759b09f54ed8f6216573fc35a1e..ff062d7bee8e4995bd347132ce95a9dd2015e94d 100644 (file)
@@ -2,11 +2,9 @@
 
 #include "map_objects.h"
 #include <stddef.h> /* NULL */
-#include <stdio.h> /* FILE typedef */
 #include <stdint.h> /* uint8_t, uint16_t */
-#include <stdlib.h> /* free(), atoi() */
-#include <string.h> /* strlen(), memcpy(), memset() */
-#include "../common/err_try_fgets.h" /* err_try_fgets() */
+#include <stdlib.h> /* free() */
+#include <string.h> /* memset(), strlen() */
 #include "../common/rexit.h" /* exit_err() */
 #include "../common/try_malloc.h" /* try_malloc() */
 #include "../common/yx_uint8.h" /* yx_uint8 struct */
@@ -15,7 +13,7 @@
 #include "world.h" /* global world */
 #include "yx_uint8.h" /* yx_uint8_cmp() */
 
-#include "io.h" /* struct EntrySkeleton */
+
 
 /* Return pointer to map object of "id" in chain starting at "ptr". */
 static struct MapObj * get_map_object(struct MapObj * ptr, uint8_t id);
@@ -95,27 +93,6 @@ static void add_map_object(uint8_t type)
 
 
 
-extern void read_map_object_def(char * line, uint32_t linemax, char * context,
-                                struct EntrySkeleton * entry, FILE * file)
-{
-    char * f_name = "init_map_object_defs()";
-    struct MapObjDef * mod = (struct MapObjDef *) entry;
-    err_try_fgets(line, linemax, file, context, "0nfi8");
-    mod->corpse_id = atoi(line);
-    err_try_fgets(line, linemax, file, context, "0nfs");
-    mod->char_on_map = line[0];
-    err_try_fgets(line, linemax, file, context, "0nfi8");
-    mod->lifepoints = atoi(line);
-    err_try_fgets(line, linemax, file, context, "0nf");
-    line[strlen(line) - 1] = '\0';
-    mod->name = try_malloc(strlen(line) + 1, f_name);
-    memcpy(mod->name, line, strlen(line) + 1);
-    err_try_fgets(line, linemax, file, context, "0nfi8");
-    mod->consumable = atoi(line);
-}
-
-
-
 extern void free_map_object_defs(struct MapObjDef * mod_start)
 {
     if (NULL == mod_start)
index cf575f38b5d4389f206268c7867b5602d496dc12..52d87e1a2276101f265a92892cc834b8ef7e12b7 100644 (file)
@@ -7,10 +7,10 @@
 #ifndef MAP_OBJECTS_H
 #define MAP_OBJECTS_H
 
-#include <stdint.h> /* uint8_t, uint32_t */
-#include <stdio.h> /* FILE */
+#include <stdint.h> /* uint8_t */
+//#include <stdio.h> /* FILE */
 #include "../common/yx_uint8.h" /* yx_uint8 structs */
-struct EntrySkeleton;
+//struct EntrySkeleton;
 
 
 
@@ -43,8 +43,8 @@ struct MapObjDef
 /* Read-in to "entry" multi-line entry from MapObjDef config "file", using
  * pre-allocated "line", "linemax" and "context" as input for err_try_fgets().
  */
-extern void read_map_object_def(char * line, uint32_t linemax, char * context,
-                                struct EntrySkeleton * entry, FILE * file);
+//extern void read_map_object_def(char * line, uint32_t linemax, char * context,
+//                                struct EntrySkeleton * entry, FILE * file);
 
 /* Free map object definitions chain starting at "mod_start". */
 extern void free_map_object_defs(struct MapObjDef * mod_start);
index 47f7c65cb7670134fdc1c9cd939f44c5cd84746d..dcd672aa38c1de2cf1abad5954fb9a6562828b3b 100644 (file)
@@ -29,8 +29,7 @@ struct World
     char * path_out; /* File to write server messages into. */
     char * path_worldstate; /* File to represent world state  to clients.*/
     char * path_record; /* Record file from which to read the game history. */
-    char * path_map_obj_defs; /* path for map object definitions config file */
-    char * path_map_obj_acts; /* path for map object actions config file */
+    char * path_config; /* Path for map object (action) definitions file. */
     char * tmp_suffix; /* Appended to paths of files for their tmp versions. */
     char * queue; /* Stores un-processed messages read from the input file. */
     uint32_t queue_size;/* Length of .queue sequence of \0-terminated strings.*/