home · contact · privacy
Don't hardcode map object action id for "move" in turn_over()!
authorChristian Heller <c.heller@plomlompom.de>
Wed, 20 Nov 2013 04:10:42 +0000 (05:10 +0100)
committerChristian Heller <c.heller@plomlompom.de>
Wed, 20 Nov 2013 04:10:42 +0000 (05:10 +0100)
src/control.c
src/map_object_actions.c
src/map_object_actions.h
src/misc.c

index 502f84a2698ea4e97ae51b211f164fa103a01689..ba1b5055ecba2d6540ac3be1cae19a680cc08046 100644 (file)
                           */
 #include "map.h" /* for map_scroll(), map_center() */
 #include "main.h" /* for world global */
-#include "rexit.h" /* for exit_err() */
 #include "wincontrol.h" /* for struct WinConf, scroll_pad(), toggle_window(),
                          * growshrink_active_window(),toggle_winconfig(),
                          * toggle_win_height_type(), toggle_win_width_type()
                          */
 #include "map_object_actions.h" /* for struct MapObjAct, actor_wait(),
                                  * actor_move(), actor_drop(), actor_pick(),
-                                 * actor_pick()
+                                 * actor_pick(), get_moa_id_by_name()
                                  */
 #include "command_db.h" /* for is_command_id_shortdsc() */
 #include "misc.h" /* for reload_interface_conf(), save_interface_conf(),
@@ -36,9 +35,6 @@ static uint8_t try_cmd_1args(int cmd, char * match, void (* f) (char), char c);
 static uint8_t try_cmd_2args(int cmd, char * match,
                              void (* f) (char, char), char c1, char c2);
 
-/* try_player_cmd() helper, returns world.map_obj_acts action id for "name". */
-static uint8_t get_moa_id_by_name(char * name);
-
 /* If "action" is id of command named "match", set player->arg, ->command and
  * call turn_over().
  */
@@ -93,23 +89,6 @@ static uint8_t try_cmd_2args(int cmd, char * match,
 
 
 
-static uint8_t get_moa_id_by_name(char * name)
-{
-    struct MapObjAct * moa = world.map_obj_acts;
-    while (NULL != moa)
-    {
-        if (0 == strcmp(moa->name, name))
-        {
-            break;
-        }
-        moa = moa->next;
-    }
-    exit_err(NULL == moa, "get_moa_id_name() did not find map object action.");
-    return moa->id;
-}
-
-
-
 static uint8_t try_player_cmd(int action, char * match, char * command,
                               uint8_t arg)
 {
index 1d3139fd157b1a2fa2fb806457a3a97c49d4600e..cdd1e97e322eeaa0b07958a71657710d59eeb527 100644 (file)
@@ -233,6 +233,23 @@ extern void free_map_object_actions(struct MapObjAct * moa)
 
 
 
+extern uint8_t get_moa_id_by_name(char * name)
+{
+    struct MapObjAct * moa = world.map_obj_acts;
+    while (NULL != moa)
+    {
+        if (0 == strcmp(moa->name, name))
+        {
+            break;
+        }
+        moa = moa->next;
+    }
+    exit_err(NULL == moa, "get_moa_id_name() did not find map object action.");
+    return moa->id;
+}
+
+
+
 extern void actor_wait(struct MapObj * mo)
 {
     if (mo == get_player())
index e9812289e4bacd8f9657a02cac1f9e25ab958783..bccd6a8a2a4ae5cca22a3f87402019c1414db176 100644 (file)
@@ -30,6 +30,9 @@ extern void init_map_object_actions();
 /* Free MapObjAct * chain starting at "moa". */
 extern void free_map_object_actions(struct MapObjAct * moa);
 
+/* Return world.map_obj_acts MapObjAct.id for "name". */
+extern uint8_t get_moa_id_by_name(char * name);
+
 /* Actor "mo" does nothing. */
 extern void actor_wait(struct MapObj * mo);
 
index 4e6527d1acdeb91c71f6136b0f618e076e807fec..0e8abe9c6eecc755fd2fd37774887f9621ff1727 100644 (file)
@@ -250,7 +250,7 @@ extern void turn_over(char action)
                     break;
                 }
                 char * sel = "NSEW";
-                map_object->command = 1;
+                map_object->command = get_moa_id_by_name("move");
                 map_object->arg = sel[rrand() % 4];
             }
             first_round = 0;