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

Comparing deliantra/server/common/object.C (file contents):
Revision 1.214 by root, Tue Apr 22 07:01:46 2008 UTC vs.
Revision 1.219 by root, Wed Apr 23 21:09:10 2008 UTC

         return;
       op->carrying += weight;
       if (object *pl = op->visible_to ())
+        if (pl != op) // player is handled lazily
-        esrv_update_item (UPD_WEIGHT, pl, op);
+          esrv_update_item (UPD_WEIGHT, pl, op);
       op = op->env;
     }
 }
   if (sum != carrying)
     {
       carrying = sum;
       if (object *pl = visible_to ())
+        if (pl != this) // player is handled lazily
-        esrv_update_item (UPD_WEIGHT, pl, this);
+          esrv_update_item (UPD_WEIGHT, pl, this);
     }
 }
 /*
  * Used by: Server DM commands: dumpbelow, dump. Some error messages.
 object::destroy_inv (bool drop_to_ground)
 {
   // need to check first, because the checks below might segfault
   // as we might be on an invalid mapspace and crossfire code
   // is too buggy to ensure that the inventory is empty.
-  // corollary: if you create arrows etc. with stuff in tis inventory,
+  // corollary: if you create arrows etc. with stuff in its inventory,
   // cf will crash below with off-map x and y
   if (!inv)
     return;
   /* Only if the space blocks everything do we not process -
       || map->nodrop
       || ms ().move_block == MOVE_ALL)
     {
       while (inv)
         {
-          inv->destroy_inv (drop_to_ground);
+          inv->destroy_inv (false);
           inv->destroy ();
         }
     }
   else
     {                       /* Put objects in inventory onto this space */
               || op->flag [FLAG_NO_DROP]
               || op->type == RUNE
               || op->type == TRAP
               || op->flag [FLAG_IS_A_TEMPLATE]
               || op->flag [FLAG_DESTROY_ON_DEATH])
-            op->destroy ();
+            op->destroy (true);
           else
             map->insert (op, x, y);
         }
     }
 }
     remove_friendly_object (this);
   remove ();
   attachable::do_destroy ();
-  destroy_inv (true);
   deactivate ();
   unlink ();
   flag [FLAG_FREED] = 1;
 object::destroy (bool destroy_inventory)
 {
   if (destroyed ())
     return;
+  if (!is_head () && !head->destroyed ())
+    {
+      LOG (llevError | logBacktrace, "tried to destroy the tail of an object");
+      head->destroy (destroy_inventory);
+    }
-  if (destroy_inventory)
+  destroy_inv (!destroy_inventory);
-    destroy_inv (false);
   if (is_head ())
     if (sound_destroy)
       play_sound (sound_destroy);
     else if (flag [FLAG_MONSTER])
              * removed (most likely destroyed), update the player view
              * appropriately.
              */
             pl->close_container ();
-          pl->contr->ns->floorbox_update ();
+          //TODO: the floorbox prev/next might need updating
+          esrv_del_item (pl->contr, count);
         }
       for (tmp = ms.bot; tmp; tmp = tmp->above)
         {
           /* No point updating the players look faces if he is the object
 object *
 insert_ob_in_map (object *op, maptile *m, object *originator, int flag)
 {
   assert (!op->flag [FLAG_FREED]);
-  object *top, *floor = NULL;
   op->remove ();
   /* Ideally, the caller figures this out.  However, it complicates a lot
    * of areas of callers (eg, anything that uses find_free_spot would now
    * need extra work
    */
   if (!xy_normalise (m, op->x, op->y))
     {
-      op->destroy ();
+      op->destroy (1);
       return 0;
     }
   if (object *more = op->more)
     if (!insert_ob_in_map (more, m, originator, flag))
    */
   if (op->nrof && !(flag & INS_NO_MERGE))
     for (object *tmp = ms.bot; tmp; tmp = tmp->above)
       if (object::can_merge (op, tmp))
         {
+          // TODO: we atcually want to update tmp, not op,
+          // but some caller surely breaks when we return tmp
+          // from here :/
           op->nrof += tmp->nrof;
           tmp->destroy (1);
         }
   CLEAR_FLAG (op, FLAG_APPLIED);        /* hack for fixing F_APPLIED in items of dead people */
       /* since *below* originator, no need to update top */
       originator->below = op;
     }
   else
     {
+      object *top, *floor = NULL;
       top = ms.bot;
       /* If there are other objects, then */
       if (top)
         {
     }
   op->map->dirty = true;
   if (object *pl = ms.player ())
-    pl->contr->ns->floorbox_update ();
+    //TODO: the floorbox prev/next might need updating
+    esrv_send_item (pl, op);
   /* If this object glows, it may affect lighting conditions that are
    * visible to others on this map.  But update_all_los is really
    * an inefficient way to do this, as it means los for all players
    * on the map will get recalculated.  The players could very well
                  &name,
                  title ? "\",title:\"" : "",
                  title ? (const char *)title : "",
                  flag_desc (flagdesc, 512), type);
-  if (!this->flag[FLAG_REMOVED] && env)
+  if (!flag[FLAG_REMOVED] && env)
     p += snprintf (p, 256, "(in %s)", env->debug_desc (info2));
   if (map)
     p += snprintf (p, 256, "(on %s@%d+%d)", &map->path, x, y);
       if (object *closer = old_container->inv)
         if (closer->type == CLOSE_CON)
           closer->destroy ();
 #endif
-      old_container->flag [FLAG_APPLIED] = 0;
+      old_container->flag [FLAG_APPLIED] = false;
       container = 0;
       esrv_update_item (UPD_FLAGS, this, old_container);
       new_draw_info_format (NDI_UNIQUE, 0, this, "You close %s.", query_name (old_container));
       play_sound (sound_find ("chest_close"));
         }
 #endif
       new_draw_info_format (NDI_UNIQUE, 0, this, "You open %s.", query_name (new_container));
-      new_container->flag [FLAG_APPLIED] = 1;
+      new_container->flag [FLAG_APPLIED] = true;
       container = new_container;
       esrv_update_item (UPD_FLAGS, this, new_container);
       esrv_send_inventory (this, new_container);
       play_sound (sound_find ("chest_open"));

Diff Legend

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

Comparing deliantra/server/common/object.C (file contents): Revision 1.214 by root, Tue Apr 22 07:01:46 2008 UTC vs. Revision 1.219 by root, Wed Apr 23 21:09:10 2008 UTC

Diff Legend

Comparing deliantra/server/common/object.C (file contents):
Revision 1.214 by root, Tue Apr 22 07:01:46 2008 UTC vs.
Revision 1.219 by root, Wed Apr 23 21:09:10 2008 UTC