--- deliantra/server/common/map.C	2007/08/01 01:53:13	1.115
+++ deliantra/server/common/map.C	2007/08/30 05:24:14	1.122
@@ -357,8 +357,6 @@
 void
 maptile::activate ()
 {
-  active = true;
-
   if (spaces)
     for (mapspace *ms = spaces + size (); ms-- > spaces; )
       for (object *op = ms->bot; op; op = op->above)
@@ -368,8 +366,6 @@
 void
 maptile::deactivate ()
 {
-  active = false;
-
   if (spaces)
     for (mapspace *ms = spaces + size (); ms-- > spaces; )
       for (object *op = ms->bot; op; op = op->above)
@@ -389,21 +385,21 @@
 
   for (int i = 0; i < size (); ++i)
     {
-      int unique = 0;
+      bool unique = 0;
+
       for (object *op = spaces [i].bot; op; op = op->above)
         {
-          if (op->flag [FLAG_UNIQUE] && op->flag [FLAG_IS_FLOOR])
-            unique = 1;
+          unique |= op->flag [FLAG_UNIQUE] && op->flag [FLAG_IS_FLOOR];
 
-          if (!op->can_map_save ())
+          if (expect_false (!op->can_map_save ()))
             continue;
 
-          if (unique || op->flag [FLAG_UNIQUE])
+          if (expect_false (unique || op->flag [FLAG_UNIQUE]))
             {
               if (flags & IO_UNIQUES)
                 op->write (f);
             }
-          else if (flags & IO_OBJECTS)
+          else if (expect_true (flags & IO_OBJECTS))
             op->write (f);
         }
     }
@@ -895,7 +891,6 @@
 
         if (QUERY_FLAG (op, FLAG_IS_FLOOR)
             || QUERY_FLAG (op, FLAG_OBJ_ORIGINAL)
-            || QUERY_FLAG (op, FLAG_OBJ_SAVE_ON_OVL)
             || QUERY_FLAG (op, FLAG_UNIQUE)
             || QUERY_FLAG (op, FLAG_OVERLAY_FLOOR)
             || QUERY_FLAG (op, FLAG_UNPAID)
@@ -1230,47 +1225,19 @@
   return vol;
 }
 
-/* this updates the orig_map->tile_map[tile_num] value after finding
- * the map.  It also takes care of linking back the freshly found
- * maps tile_map values if it tiles back to this one.  It returns
- * the value of orig_map->tile_map[tile_num].
- */
-static inline maptile *
-find_and_link (maptile *orig_map, int tile_num)
+bool
+maptile::tile_available (int dir, bool load)
 {
-  maptile *mp = orig_map->tile_map [tile_num];
-
-  if (!mp)
-    {
-      mp = orig_map->find_sync (orig_map->tile_path [tile_num], orig_map);
-
-      if (!mp)
-        {
-          // emergency mode, manufacture a dummy map, this creates a memleak, but thats fine
-          LOG (llevError, "FATAL: cannot load tiled map %s from %s, leaking memory and worse!\n",
-               &orig_map->tile_path[tile_num], &orig_map->path);
-          mp = new maptile (1, 1);
-          mp->alloc ();
-          mp->in_memory = MAP_IN_MEMORY;
-        }
-    }
-
-  int dest_tile = (tile_num + 2) % 4;
-
-  orig_map->tile_map [tile_num] = mp;
+  if (!tile_path[dir])
+    return 0;
 
-  // optimisation: back-link map to origin map if euclidean
-  //TODO: non-euclidean maps MUST GO
-  if (orig_map->tile_map[tile_num]->tile_path[dest_tile] == orig_map->path)
-    orig_map->tile_map[tile_num]->tile_map[dest_tile] = orig_map;
+  if (tile_map[dir] && (!load || tile_map[dir]->in_memory == MAP_IN_MEMORY))
+    return 1;
 
-  return mp;
-}
+  if ((tile_map[dir] = find_async (tile_path[dir], this, load)))
+    return 1;
 
-static inline void
-load_and_link (maptile *orig_map, int tile_num)
-{
-  find_and_link (orig_map, tile_num)->load_sync ();
+  return 0;
 }
 
 /* this returns TRUE if the coordinates (x,y) are out of
@@ -1293,45 +1260,33 @@
 
   if (x < 0)
     {
-      if (!m->tile_path[3])
+      if (!m->tile_available (3))
         return 1;
 
-      if (!m->tile_map[3] || m->tile_map[3]->in_memory != MAP_IN_MEMORY)
-        find_and_link (m, 3);
-
       return out_of_map (m->tile_map[3], x + m->tile_map[3]->width, y);
     }
 
   if (x >= m->width)
     {
-      if (!m->tile_path[1])
+      if (!m->tile_available (1))
         return 1;
 
-      if (!m->tile_map[1] || m->tile_map[1]->in_memory != MAP_IN_MEMORY)
-        find_and_link (m, 1);
-
       return out_of_map (m->tile_map[1], x - m->width, y);
     }
 
   if (y < 0)
     {
-      if (!m->tile_path[0])
+      if (!m->tile_available (0))
         return 1;
 
-      if (!m->tile_map[0] || m->tile_map[0]->in_memory != MAP_IN_MEMORY)
-        find_and_link (m, 0);
-
       return out_of_map (m->tile_map[0], x, y + m->tile_map[0]->height);
     }
 
   if (y >= m->height)
     {
-      if (!m->tile_path[2])
+      if (!m->tile_available (2))
         return 1;
 
-      if (!m->tile_map[2] || m->tile_map[2]->in_memory != MAP_IN_MEMORY)
-        find_and_link (m, 2);
-
       return out_of_map (m->tile_map[2], x, y - m->height);
     }
 
@@ -1354,40 +1309,36 @@
 {
   if (x < 0)
     {
-      if (!tile_path[3])
+      if (!tile_available (3))
         return 0;
 
-      find_and_link (this, 3);
       x += tile_map[3]->width;
       return tile_map[3]->xy_find (x, y);
     }
 
   if (x >= width)
     {
-      if (!tile_path[1])
+      if (!tile_available (1))
         return 0;
 
-      find_and_link (this, 1);
       x -= width;
       return tile_map[1]->xy_find (x, y);
     }
 
   if (y < 0)
     {
-      if (!tile_path[0])
+      if (!tile_available (0))
         return 0;
 
-      find_and_link (this, 0);
       y += tile_map[0]->height;
       return tile_map[0]->xy_find (x, y);
     }
 
   if (y >= height)
     {
-      if (!tile_path[2])
+      if (!tile_available (2))
         return 0;
 
-      find_and_link (this, 2);
       y -= height;
       return tile_map[2]->xy_find (x, y);
     }
@@ -1408,7 +1359,7 @@
   if (!map1 || !map2)
     return 0;
 
-  //TODO: this doesn't actually check corretcly when intermediate maps are not loaded
+  //TODO: this doesn't actually check correctly when intermediate maps are not loaded
   //fix: compare paths instead (this is likely faster, too!)
   if (map1 == map2)
     {