[ViewVC] Diff of: cvs/deliantra/server/common/map.C

Comparing deliantra/server/common/map.C (file contents):
Revision 1.52 by root, Tue Dec 26 20:04:09 2006 UTC vs.
Revision 1.53 by root, Wed Dec 27 09:28:01 2006 UTC

  * return NULL if no match is found.
  */
 maptile *
 has_been_loaded (const char *name)
 {
-  maptile *map;
   if (!name || !*name)
     return 0;
-  for (map = first_map; map; map = map->next)
+  for_all_maps (map)
     if (!strcmp (name, map->path))
-      break;
-  return (map);
+      return map;
+  return 0;
 }
 /*
  * This makes a path absolute outside the world of Crossfire.
  * In other words, it prepends LIBDIR/MAPDIR/ to the given path
  */
 void
 dump_all_maps (void)
 {
-  maptile *m;
+  for_all_maps (m)
-  for (m = first_map; m; m = m->next)
     dump_map (m);
 }
 /* This rolls up wall, blocks_magic, blocks_view, etc, all into
  * one function that just returns a P_.. value (see map.h)
  * for objects whose arch says they are multipart yet according to the
  * info we have, they only have the head (as would be expected when
  * they are saved).  We do have to look for the old maps that did save
  * the more sections and not re-add sections for them.
  */
 static void
 link_multipart_objects (maptile *m)
 {
   int x, y;
   object *tmp, *op, *last, *above;
 {
   in_memory = MAP_SWAPPED;
   /* The maps used to pick up default x and y values from the
    * map archetype.  Mimic that behaviour.
    */
-  this->width = 16;
+  width         = 16;
-  this->height = 16;
+  height        = 16;
-  this->reset_timeout = 0;
+  reset_timeout = 0;
-  this->timeout = 300;
+  timeout       = 300;
-  this->enter_x = 0;
+  enter_x       = 0;
-  this->enter_y = 0;
+  enter_y       = 0;
   /*set part to -1 indicating conversion to weather map not yet done */
-  this->worldpartx = -1;
+  worldpartx    = -1;
-  this->worldparty = -1;
+  worldparty    = -1;
 }
 /*
  * Allocates, initialises, and returns a pointer to a maptile.
  * Modified to no longer take a path option which was not being
  * used anyways.  MSW 2001-07-01
  */
 maptile *
 get_linked_map (void)
 {
-  maptile *mp, *map = new maptile;
+  maptile *map = new maptile;
-  for (mp = first_map; mp && mp->next; mp = mp->next);
+  map->next = first_map;
-  if (mp == NULL)
-    first_map = map;
+  first_map = map;
-  else
-    mp->next = map;
   return map;
 }
 /*
    * that is their poor assumption.
    */
   if (spaces)
     {
       LOG (llevError, "allocate_map called with already allocated map (%s)\n", path);
-      free (spaces);
+      sfree (spaces, size ());
     }
-  spaces = (mapspace *)
+  spaces = salloc0<mapspace> (size ());
-    calloc (1, width * height * sizeof (mapspace));
-  if (!spaces)
-    fatal (OUT_OF_MEMORY);
 }
 /* Create and returns a map of the specific size.  Used
  * in random map code and the editor.
  */
                 op->destroy ();
               }
           }
       }
 }
 /*
  * Loads unique objects from file(s) into the map which is in memory
  * m is the map to load unique items into.
  */
   // TODO: use new/delete
   if (flag && m->spaces)
     free_all_objects (m);
+  sfree (m->spaces, m->size ()), m->spaces = 0;
   free (m->name),     m->name = 0;
-  free (m->spaces),   m->spaces = 0;
   free (m->msg),      m->msg = 0;
   free (m->maplore),  m->maplore = 0;
   free (m->shoprace), m->shoprace = 0;
   delete [] m->shopitems, m->shopitems = 0;
   if (m->buttons)
     free_objectlinkpt (m->buttons), m->buttons = 0;
   for (int i = 0; i < 4; i++)
-    {
-      if (m->tile_path[i])
-        free (m->tile_path[i]), m->tile_path[i] = 0;
+    free (m->tile_path[i]), m->tile_path[i] = 0;
-      m->tile_map[i] = 0;
-    }
   m->in_memory = MAP_SWAPPED;
 }
 maptile::~maptile ()
 {
-  free_map (this, 1);
+  assert (destroyed ());
-  free (tmpname);
 }
 void
 maptile::do_destroy ()
 {
   attachable::do_destroy ();
-  free_all_objects (this);
+  free_map (this, 1);
+  free (tmpname), tmpname = 0;
   /* We need to look through all the maps and see if any maps
    * are pointing at this one for tiling information.  Since
    * tiling can be asymetric, we just can not look to see which
    * maps this map tiles with and clears those.
    */
   //TODO: non-euclidean-tiling MUST GO
-  for (maptile *m = first_map; m; m = m->next)
+  for_all_maps (m)
     for (int i = 0; i < 4; i++)
       if (m->tile_map[i] == this)
         m->tile_map[i] = 0;
   if (first_map == this)
     first_map = next;
   else
-    for (maptile *m = first_map; m; m = m->next)
+    for_all_maps (m)
       if (m->next = this)
         {
           m->next = next;
           break;
         }
       if (first_map->in_memory == MAP_SAVING)
         first_map->in_memory = MAP_IN_MEMORY;
       delete_map (first_map);
       real_maps++;
     }
   LOG (llevDebug, "free_all_maps: Freed %d maps\n", real_maps);
 }
 /* change_map_light() - used to change map light level (darkness)
  * up or down. Returns true if successful.   It should now be

Diff Legend

-–
+Removed lines
-+
+Added lines
-<
+Changed lines
->
+Changed lines

Comparing deliantra/server/common/map.C (file contents): Revision 1.52 by root, Tue Dec 26 20:04:09 2006 UTC vs. Revision 1.53 by root, Wed Dec 27 09:28:01 2006 UTC

Diff Legend

Comparing deliantra/server/common/map.C (file contents):
Revision 1.52 by root, Tue Dec 26 20:04:09 2006 UTC vs.
Revision 1.53 by root, Wed Dec 27 09:28:01 2006 UTC