From: Christian Heller <c.heller@plomlompom.de>
Date: Thu, 10 Jul 2014 04:23:19 +0000 (+0200)
Subject: Server: Expect (but don't force) some player commands as sans argument.
X-Git-Tag: tce~712
X-Git-Url: https://plomlompom.com/repos/%22https:/validator.w3.org/balance?a=commitdiff_plain;h=9a0c71899a4d8f61600c6e317ba8d732f19697ea;p=plomrogue

Server: Expect (but don't force) some player commands as sans argument.
---

diff --git a/src/server/run.c b/src/server/run.c
index 8e1914b..bd05402 100644
--- a/src/server/run.c
+++ b/src/server/run.c
@@ -27,7 +27,7 @@
 #include "things.h" /* Thing, get_thing(), own_thing(), add_thing(),
                      * get_thing_action_id_by_name(), get_player()
                      */
-#include "world.h" /* global world */
+#include "world.h" /* world */
 
 
 
@@ -52,8 +52,19 @@ static uint8_t parse_do_fov(char * tok0, char * tok1);
 /* Parse/apply god command in "tok0"/"tok1" manipulating a thing's state. */
 static uint8_t parse_thing_manipulation(char * tok0, char * tok1);
 
-/* Parse player command in "tok0"/"tok1" to action in player thing. */
-static uint8_t parse_player_command(char * tok0, char * tok1);
+/* Parse player command "tok0" with one arguments "tok1" to player action. */
+static uint8_t parse_player_command_0arg(char * tok0, char * tok1);
+
+/* Parse player command "tok0" with no argument to player action, comment on
+ * invalidity of non-zero "tok1" (but do not abort in that case).
+ */
+static uint8_t parse_player_command_1arg(char * tok0, char * tok1);
+
+/* Parse/apply commadn "tok0" with argument "tok1" and test the line for further
+ * tokens, commenting on their invalidity (but don't abort on findingthem).
+ */
+static uint8_t parse_command_1arg(char * tok0, char * tok1);
+
 
 /* Compares first line of server out file to world.server_test, aborts if they
  * don't match, but not before unsetting the flags deleting files in the server
@@ -221,12 +232,26 @@ static uint8_t parse_thing_manipulation(char * tok0, char * tok1)
 
 
 
-static uint8_t parse_player_command(char * tok0, char * tok1)
+static uint8_t parse_player_command_0arg(char * tok0, char * tok1)
+{
+    struct Thing * player = get_player();
+    if (!strcmp(tok0, s[S_CMD_WAIT]) || !strcmp(tok0, s[S_CMD_PICKUP]))
+    {
+        player->command = get_thing_action_id_by_name(tok0);
+        player->arg = 0;
+        turn_over();
+        err_line (NULL != tok1, "No arguments expected, ignoring arguments.");
+        return 1;
+    }
+    return 0;
+}
+
+
+
+static uint8_t parse_player_command_1arg(char * tok0, char * tok1)
 {
     struct Thing * player = get_player();
-    if (   parse_val(tok0, tok1, s[S_CMD_WAIT], '8', (char *) &player->arg)
-        || parse_val(tok0, tok1, s[S_CMD_MOVE], '8', (char *) &player->arg)
-        || parse_val(tok0, tok1, s[S_CMD_PICKUP], '8', (char *) &player->arg)
+    if (   parse_val(tok0, tok1, s[S_CMD_MOVE], '8', (char *) &player->arg)
         || parse_val(tok0, tok1, s[S_CMD_DROP], '8', (char *) &player->arg)
         || parse_val(tok0, tok1, s[S_CMD_USE], '8', (char *) &player->arg))
     {
@@ -242,6 +267,34 @@ static uint8_t parse_player_command(char * tok0, char * tok1)
 
 
 
+static uint8_t parse_command_1arg(char * tok0, char * tok1)
+{
+    char * tok2 = token_from_line(NULL);
+    if (   parse_thing_manipulation(tok0, tok1)
+        || parse_player_command_1arg(tok0, tok1)
+        || parse_val(tok0, tok1, s[S_CMD_SEED_RAND], 'U', (char *) &world.seed)
+        || parse_val(tok0, tok1, s[S_CMD_TURN], 'u', (char *) &world.turn)
+        || parse_do_fov(tok0, tok1));
+    else if (parse_val(tok0,tok1,s[S_CMD_SEED_MAP],'U',(char *)&world.seed_map))
+
+    {
+        remake_map();
+    }
+    else if (parse_val(tok0, tok1, s[S_CMD_MAKE_WORLD],'U',(char *)&world.seed))
+    {
+        remake_world();
+    }
+    else
+    {
+        return 0;
+    }
+    char * err = "But one argument expected, ignoring further arguments.";
+    err_line (NULL != tok2, err);
+    return 1;
+}
+
+
+
 static void server_test()
 {
     char * f_name = "server_test()";
@@ -334,39 +387,24 @@ extern void obey_msg(char * msg, uint8_t do_record)
     set_err_line_options("Trouble with message: ", msg, 0, 0);
     char * msg_copy = strdup(msg);
     char * tok0 = token_from_line(msg_copy);
-    char * tok1 = token_from_line(NULL);
-    char * tok2 = token_from_line(NULL);
-    if (err_line(!(tok0 && tok1) || tok2, "Bad number of tokens."))
-    {
-        return;
-    }
-    if (   parse_thing_manipulation(tok0, tok1)
-        || parse_player_command(tok0, tok1)
-        || parse_val(tok0, tok1, s[S_CMD_SEED_RAND], 'U', (char *) &world.seed)
-        || parse_val(tok0, tok1, s[S_CMD_TURN], 'u', (char *) &world.turn)
-        || parse_do_fov(tok0, tok1));
-    else if (parse_val(tok0,tok1,s[S_CMD_SEED_MAP],'U',(char *)&world.seed_map))
-
-    {
-        remake_map();
-    }
-    else if (parse_val(tok0, tok1, s[S_CMD_MAKE_WORLD],'U',(char *)&world.seed))
-    {
-        remake_world();
-    }
-    else
+    if (NULL != tok0)
     {
-        err_line(1, "Unknown command.");
-        free(msg_copy);
-        return;
+        char * tok1 = token_from_line(NULL);
+        if (    parse_player_command_0arg(tok0, tok1)
+            || (tok1 && parse_command_1arg(tok0, tok1)))
+        {
+            world.do_update = 1;
+            if (do_record)
+            {
+                save_world();
+                record_msg(msg);
+            }
+            free(msg_copy);
+            return;
+        }
     }
-    world.do_update = 1;
+    err_line(1, "Unknown command or bad number of tokens.");
     free(msg_copy);
-    if (do_record)
-    {
-        save_world();
-        record_msg(msg);
-    }
 }