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

Comparing deliantra/server/common/object.C (file contents):
Revision 1.211 by root, Mon Apr 21 21:32:32 2008 UTC vs.
Revision 1.215 by root, Tue Apr 22 07:28:05 2008 UTC

 #include <global.h>
 #include <stdio.h>
 #include <sys/types.h>
 #include <sys/uio.h>
 #include <object.h>
-#include <funcpoint.h>
 #include <sproto.h>
 #include <loader.h>
 #include <bitset>
   /* Everything passes, must be OK. */
   return 1;
 }
+// find player who can see this object
+object *
+object::visible_to () const
+{
+  if (!flag [FLAG_REMOVED])
+    {
+      // see if we are in a container of sorts
+      if (env)
+        {
+          // the player inventory itself is always visible
+          if (env->type == PLAYER)
+            return env;
+          // else a player could have our env open
+          object *envest = env->outer_env ();
+          // the player itself is always on a map, so we will find him here
+          // even if our inv is in a player.
+          if (envest->is_on_map ())
+            if (object *pl = envest->ms ().player ())
+              if (pl->container == env)
+                return pl;
+        }
+      else
+        {
+          // maybe there is a player standing on the same mapspace
+          // this will catch the case where "this" is a player
+          if (object *pl = ms ().player ())
+            return pl;
+        }
+    }
+  return 0;
+}
 // adjust weight per container type ("of holding")
 static sint32
 weight_adjust (object *op, sint32 weight)
 {
   return op->type == CONTAINER
 {
   while (op)
     {
       weight = weight_adjust (op, weight);
+      if (!weight)
+        return;
       op->carrying += weight;
+      if (object *pl = op->visible_to ())
+        if (pl != op) // player is handled lazily
+          esrv_update_item (UPD_WEIGHT, pl, op);
       op = op->env;
     }
 }
 /*
         op->update_weight ();
       sum += op->total_weight ();
     }
-  carrying = weight_adjust (this, sum);
+  sum = weight_adjust (this, sum);
+  if (sum != carrying)
+    {
+      carrying = sum;
+      if (object *pl = visible_to ())
+        if (pl != this) // player is handled lazily
+          esrv_update_item (UPD_WEIGHT, pl, this);
+    }
 }
 /*
  * Used by: Server DM commands: dumpbelow, dump. Some error messages.
  */
 }
 void
 object::do_destroy ()
 {
-  attachable::do_destroy ();
   if (flag [FLAG_IS_LINKED])
     remove_button_link (this);
   if (flag [FLAG_FRIENDLY])
     remove_friendly_object (this);
-  if (!flag [FLAG_REMOVED])
-    remove ();
+  remove ();
+  attachable::do_destroy ();
   destroy_inv (true);
   deactivate ();
   unlink ();
 object::do_remove ()
 {
   object *tmp, *last = 0;
   object *otmp;
-  if (QUERY_FLAG (this, FLAG_REMOVED))
+  if (flag [FLAG_REMOVED])
     return;
-  SET_FLAG (this, FLAG_REMOVED);
   INVOKE_OBJECT (REMOVE, this);
+  if (object *pl = visible_to ())
+    esrv_del_item (pl->contr, count);
+  flag [FLAG_REMOVED] = true;
   if (more)
     more->remove ();
   /*
   if (!top)
     for (top = op; top && top->above; top = top->above)
       ;
   for (; top; top = top->below)
-    {
-      if (top == op)
-        continue;
-      if (object::can_merge (op, top))
+    if (object::can_merge (op, top))
-        {
+      {
-          top->nrof += op->nrof;
+        top->nrof += op->nrof;
-/*      CLEAR_FLAG(top,FLAG_STARTEQUIP);*/
+        if (object *pl = top->visible_to ())
-          op->weight = 0;       /* Don't want any adjustements now */
+          esrv_update_item (UPD_NROF, pl, top);
+        op->weight   = 0; // cancel the addition above
+        op->carrying = 0; // must be 0 already
-          op->destroy ();
+        op->destroy (1);
-          return top;
+        return top;
-        }
+      }
-    }
   return 0;
 }
 void
     return where->env->insert (this);
   else
     return where->map->insert (this, where->x, where->y, originator, flags);
 }
-// find player who can see this object
-static object *
-visible_to (object *op)
-{
-  if (!op->flag [FLAG_REMOVED])
-    {
-      // see if we are in a container of sorts
-      if (object *env = op->env)
-        {
-          // the player inventory itself is always visible
-          if (env->type == PLAYER)
-            return env;
-          // else a player could have our env open
-          env = env->outer_env ();
-          // the player itself is always on a map, so we will find him here
-          // even if our inv is in a player.
-          if (env->is_on_map ())
-            if (object *pl = env->ms ().player ())
-              if (pl->container == op->env)
-                return pl;
-        }
-      else
-        {
-          // maybe there is a player standing on the same mapspace
-          if (object *pl = op->ms ().player ())
-            return pl;
-        }
-    }
-  return 0;
-}
 /*
  * decrease(object, number) decreases a specified number from
  * the amount of an object. If the amount reaches 0, the object
  * is subsequently removed and freed.
  *
  * Return value: 'op' if something is left, NULL if the amount reached 0
  */
 bool
 object::decrease (sint32 nr)
 {
+  if (!nr)
+    return true;
   nr = min (nr, nrof);
-  if (!nr)
-    return 1;
   nrof -= nr;
-  object *visible = visible_to (this);
   if (nrof)
     {
       adjust_weight (env, -weight * nr); // carrying == 0
-      if (visible)
+      if (object *pl = visible_to ())
-        esrv_send_item (visible, this);
+        esrv_update_item (UPD_NROF, pl, this);
-      return 1;
+      return true;
     }
   else
     {
-      if (visible)
-        esrv_del_item (visible->contr, count);
-      this->destroy (1);
+      destroy (1);
-      return 0;
+      return false;
     }
 }
 /*
  * split(ob,nr) splits up ob into two parts.  The part which
  * On failure, NULL is returned.
  */
 object *
 object::split (sint32 nr)
 {
+  int have = number_of ();
-  if (nrof < nr)
+  if (have < nr)
     return 0;
-  else if (nrof == nr)
+  else if (have == nr)
     {
       remove ();
       return this;
     }
   else
       if (object::can_merge (tmp, op))
         {
           /* return the original object and remove inserted object
              (client needs the original object) */
           tmp->nrof += op->nrof;
+          if (object *pl = tmp->visible_to ())
+            esrv_update_item (UPD_NROF, pl, tmp);
           adjust_weight (this, op->total_weight ());
           op->destroy (1);
           op = tmp;
           goto inserted;
   inv = op;
   op->flag [FLAG_REMOVED] = 0;
+  if (object *pl = op->visible_to ())
+    esrv_send_item (pl, op);
   adjust_weight (this, op->total_weight ());
 inserted:
   /* reset the light list and los of the players on the map */
   if (op->glow_radius && map && map->darkness)
     update_all_los (map, x, y);
-  if (object *otmp = in_player ())
+  // if this is a player's inventory, update stats
-    if (otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
+  if (type == PLAYER && !flag [FLAG_NO_FIX_PLAYER])
-      otmp->update_stats ();
+    update_stats ();
   INVOKE_OBJECT (INSERT, this);
   return op;
 }

Diff Legend

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

Comparing deliantra/server/common/object.C (file contents): Revision 1.211 by root, Mon Apr 21 21:32:32 2008 UTC vs. Revision 1.215 by root, Tue Apr 22 07:28:05 2008 UTC

Diff Legend

Comparing deliantra/server/common/object.C (file contents):
Revision 1.211 by root, Mon Apr 21 21:32:32 2008 UTC vs.
Revision 1.215 by root, Tue Apr 22 07:28:05 2008 UTC