#include "map_objects.h"
#include <stdlib.h> /* for free(), atoi() */
-#include <stdint.h> /* for uint8_t */
+#include <stdint.h> /* for uint8_t, uint16_t */
#include <stdio.h> /* for FILE typedef */
-#include <string.h> /* for strchr(), strlen(), memcpy(), strtok() */
-#include "readwrite.h" /* for get_linemax(), try_fopen(), try_fclose()
- * [read/write]_uint[8/16/23][_bigendian]()
+#include <string.h> /* for strlen(), memcpy(), strtok() */
+#include "readwrite.h" /* for textfile_sizes(), try_fopen(), try_fclose(),
+ * try_fgets()
*/
#include "misc.h" /* for try_malloc(), find_passable_pos() */
#include "main.h" /* for world global */
-#include "rexit.h" /* for err_exit() */
+#include "rexit.h" /* for exit_err() */
#include "yx_uint16.h" /* for yx_uint16 struct, yx_uint16_cmp() */
char line[size];
sprintf(line, "%d %d %d %d %d %d %d %d",
mo->id, mo->type, mo->lifepoints, mo->pos.y, mo->pos.x,
- mo->progress, mo->command, mo->arg);
+ mo->command, mo->arg, mo->progress);
for (mo_ptr = mo->owns; NULL != mo_ptr; mo_ptr = mo_ptr->next)
{
sprintf(line + strlen(line), " %d", mo_ptr->id);
{
char * f_name = "init_map_object_defs()";
FILE * file = try_fopen(filename, "r", f_name);
- uint16_t linemax = get_linemax(file, f_name);
+ uint16_t linemax = textfile_sizes(file, NULL);
struct MapObjDef ** last_mod_ptr_ptr = &world.map_obj_defs;
char * delim = " ";
char line[linemax + 1];
while (try_fgets(line, linemax + 1, file, f_name))
{
- struct MapObjDef * mod;
- mod = try_malloc(sizeof(struct MapObjDef), f_name);
+ struct MapObjDef * mod = try_malloc(sizeof(struct MapObjDef), f_name);
mod->next = NULL;
mod->id = atoi(strtok(line, delim));
mod->corpse_id = atoi(strtok(NULL, delim));
mo->lifepoints = atoi(strtok(NULL, delim));
mo->pos.y = atoi(strtok(NULL, delim));
mo->pos.x = atoi(strtok(NULL, delim));
- mo->progress = atoi(strtok(NULL, delim));;
mo->command = atoi(strtok(NULL, delim));;
mo->arg = atoi(strtok(NULL, delim));;
+ mo->progress = atoi(strtok(NULL, delim));;
mo->owns = NULL;
- if (mo->id > world.map_obj_count)
- {
- world.map_obj_count = mo->id;
- }
* mo_ptr_ptr = mo;
mo_ptr_ptr = &mo->next;
}
{
uint8_t id = atoi(strtok(line, delim));
uint8_t i;
- for (i = 0; i < 7; i++)
- {
- strtok(NULL, delim);
- }
+ for (i = 0; i < 7; i++, strtok(NULL, delim));
char * owned = strtok(NULL, "\n");
if (NULL != owned)
{
owned_id = strtok(owned, delim);
while (NULL != owned_id)
{
- own_map_object(&mo->owns, &world.map_objs,
- (uint8_t) atoi(owned_id));
+ own_map_object(&mo->owns, &world.map_objs, atoi(owned_id));
owned_id = strtok(NULL, delim);
}
}
{
char * f_name = "add_map_object()";
struct MapObjDef * mod = get_map_object_def(type);
- struct MapObj * mo = try_malloc(sizeof(struct MapObj), f_name);
- mo->id = world.map_obj_count;
- world.map_obj_count++;
- mo->type = mod->id;
+ struct MapObj * mo = try_malloc(sizeof(struct MapObj), f_name);
+ mo->id = world.map_obj_count++;
+ mo->type = mod->id;
mo->lifepoints = mod->lifepoints;
while (1)
{
}
}
mo->progress = 0;
- mo->command = 0;
- mo->arg = 0;
- mo->owns = NULL;
- mo->next = NULL;
- struct MapObj ** last_ptr_ptr = &world.map_objs;
- struct MapObj * mo_ptr;
- while (NULL != * last_ptr_ptr)
- {
- mo_ptr = * last_ptr_ptr;
- last_ptr_ptr = & mo_ptr->next;
- }
- * last_ptr_ptr = mo;
+ mo->command = 0;
+ mo->arg = 0;
+ mo->owns = NULL;
+ mo->next = NULL;
+ struct MapObj ** mo_ptr_ptr = &world.map_objs;
+ for (; NULL != * mo_ptr_ptr; mo_ptr_ptr = &(*mo_ptr_ptr)->next);
+ * mo_ptr_ptr = mo;
}
mo = penult->next;
penult->next = mo->next;
}
- struct MapObj ** last_ptr_ptr = target;
- struct MapObj * mo_ptr;
- while (NULL != * last_ptr_ptr)
- {
- mo_ptr = * last_ptr_ptr;
- last_ptr_ptr = & mo_ptr->next;
- }
- * last_ptr_ptr = mo;
+ struct MapObj ** mo_ptr_ptr = target;
+ for (; NULL != * mo_ptr_ptr; mo_ptr_ptr = &(*mo_ptr_ptr)->next);
+ * mo_ptr_ptr = mo;
mo->next = NULL;
}
extern struct MapObjDef * get_map_object_def(uint8_t id)
{
struct MapObjDef * mod = world.map_obj_defs;
- while (id != mod->id)
- {
- mod = mod->next;
- }
+ for (; id != mod->id; mod = mod->next);
return mod;
}
{
mo->pos = pos;
struct MapObj * owned = mo->owns;
- for (; owned != NULL; owned = owned->next)
- {
- set_object_position(owned, pos);
- }
+ for (; owned != NULL; set_object_position(owned, pos), owned = owned->next);
}
+