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

Comparing deliantra/server/common/object.C (file contents):
Revision 1.265 by root, Tue Dec 23 00:39:48 2008 UTC vs.
Revision 1.282 by root, Tue Feb 17 03:53:31 2009 UTC

 /*
  * This file is part of Deliantra, the Roguelike Realtime MMORPG.
  *
- * Copyright (©) 2005,2006,2007,2008 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
+ * Copyright (©) 2005,2006,2007,2008,2009 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
  * Copyright (©) 2001,2007 Mark Wedel & Crossfire Development Team
  * Copyright (©) 1992,2007 Frank Tore Johansen
  *
  * Deliantra is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
 #include <bitset>
 UUID UUID::cur;
 static uint64_t seq_next_save;
 static const uint64 UUID_GAP = 1<<19;
+uint32_t mapspace::smellcount = 10000;
 objectvec objects;
 activevec actives;
 short freearr_x[SIZEOFFREE] = {
     {
       // see if we are in a container of sorts
       if (env)
         {
           // the player inventory itself is always visible
-          if (env->type == PLAYER)
+          if (env->is_player ())
             return env;
           // else a player could have our env open
           object *envest = env->outer_env ();
   else if (action == UP_OBJ_INSERT)
     {
       // this is likely overkill, TODO: revisit (schmorp)
       if ((QUERY_FLAG (op, FLAG_BLOCKSVIEW)  && !(m.flags_ & P_BLOCKSVIEW))
           || (QUERY_FLAG (op, FLAG_NO_MAGIC) && !(m.flags_ & P_NO_MAGIC))
-          || (op->type == PLAYER             && !(m.flags_ & P_PLAYER))
+          || (op->is_player ()               && !(m.flags_ & P_PLAYER))
           || (op->type == SAFE_GROUND        && !(m.flags_ & P_SAFE))
           || (QUERY_FLAG (op, FLAG_ALIVE)    && !(m.flags_ & P_IS_ALIVE))
           || (QUERY_FLAG (op, FLAG_DAMNED)   && !(m.flags_ & P_NO_CLERIC))
           || (m.move_on   | op->move_on  ) != m.move_on
           || (m.move_off  | op->move_off ) != m.move_off
 void
 object::do_destroy ()
 {
   if (flag [FLAG_IS_LINKED])
-    remove_button_link (this);
+    remove_link ();
   if (flag [FLAG_FRIENDLY])
     remove_friendly_object (this);
   remove ();
       /* NO_FIX_PLAYER is set when a great many changes are being
        * made to players inventory.  If set, avoiding the call
        * to save cpu time.
        */
       if (pl)
-        if (pl->contr && !QUERY_FLAG (pl, FLAG_NO_FIX_PLAYER))
+        if (pl->is_player () && (glow_radius || !QUERY_FLAG (pl, FLAG_NO_FIX_PLAYER)))
           {
             pl->update_stats ();
-            if (affects_los () && pl->is_on_map () && pl->map->darkness)
+            if (glow_radius && pl->is_on_map ())
               update_all_los (pl->map, pl->x, pl->y);
           }
     }
   else if (map)
     {
       map->dirty = true;
       mapspace &ms = this->ms ();
       if (object *pl = ms.player ())
         {
-          if (type == PLAYER) // this == pl(!)
+          if (is_player ())
             {
+              if (!flag [FLAG_WIZPASS])
+                ms.smell = ++mapspace::smellcount; // remember the smell of the player
               // leaving a spot always closes any open container on the ground
               if (container && !container->env)
                 // this causes spurious floorbox updates, but it ensures
                 // that the CLOSE event is being sent.
                 close_container ();
             }
           last = tmp;
         }
-      if (map->darkness && affects_los ())
+      if (affects_los ())
         update_all_los (map, x, y);
     }
 }
 /*
   /* 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
    */
+  maptile *newmap = m;
-  if (!xy_normalise (m, op->x, op->y))
+  if (!xy_normalise (newmap, op->x, op->y))
     {
       op->head_ ()->destroy ();// remove head_ once all tail object destroyers found
       return 0;
     }
     if (!insert_ob_in_map (more, m, originator, flag))
       return 0;
   CLEAR_FLAG (op, FLAG_REMOVED);
-  op->map = m;
+  op->map = newmap;
   mapspace &ms = op->ms ();
   /* this has to be done after we translate the coordinates.
    */
   if (op->nrof && !(flag & INS_NO_MERGE))
           LOG (llevError, "insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n");
           abort ();
         }
       if (!originator->is_on_map ())
+        {
-        LOG (llevDebug | logBacktrace, "insert_ob_in_map(%s) called with INS_BELOW_ORIGINATOR when originator '%s' not on map",
+          LOG (llevError, "insert_ob_in_map(%s) called with INS_BELOW_ORIGINATOR when originator '%s' not on map",
-             op->debug_desc (), originator->debug_desc ());
+               op->debug_desc (), originator->debug_desc ());
+          abort ();
+        }
       op->above = originator;
       op->below = originator->below;
       originator->below = op;
           op->below = top;
           *(op->above ? &op->above->below : &ms.top) = op;
         }
     }
-  if (op->type == PLAYER)
+  if (op->is_player ())
     {
       op->contr->do_los = 1;
       ++op->map->players;
       op->map->touch ();
     }
    * be far away from this change and not affected in any way -
    * this should get redone to only look for players within range,
    * or just updating the P_UPTODATE for spaces within this area
    * of effect may be sufficient.
    */
-  if (op->affects_los () && op->map->darkness)
+  if (op->affects_los ())
     {
       op->ms ().invalidate ();
       update_all_los (op->map, op->x, op->y);
     }
 /* this function inserts an object in the map, but if it
  * finds an object of its own type, it'll remove that one first.
  * op is the object to insert it under: supplies x and the map.
  */
 void
-replace_insert_ob_in_map (const char *arch_string, object *op)
+replace_insert_ob_in_map (shstr_tmp archname, object *op)
 {
   /* first search for itself and remove any old instances */
   for (object *tmp = op->ms ().bot; tmp; tmp = tmp->above)
-    if (!strcmp (tmp->arch->archname, arch_string))       /* same archetype */
+    if (tmp->arch->archname == archname)       /* same archetype */
       tmp->destroy ();
-  object *tmp = arch_to_object (archetype::find (arch_string));
+  object *tmp = arch_to_object (archetype::find (archname));
   tmp->x = op->x;
   tmp->y = op->y;
   insert_ob_in_map (tmp, op->map, op, 0);
   adjust_weight (this, op->total_weight ());
 inserted:
   /* reset the light list and los of the players on the map */
-  if (op->glow_radius && is_on_map () && map->darkness)
+  if (op->glow_radius && is_on_map ())
     {
       update_stats ();
       update_all_los (map, x, y);
     }
-  else if (type == PLAYER && !flag [FLAG_NO_FIX_PLAYER])
+  else if (is_player () && !flag [FLAG_NO_FIX_PLAYER])
     // if this is a player's inventory, update stats
     update_stats ();
   INVOKE_OBJECT (INSERT, this);
             {
               float
                 diff = tmp->move_slow_penalty * fabs (op->speed);
-              if (op->type == PLAYER)
+              if (op->is_player ())
                 if ((QUERY_FLAG (tmp, FLAG_IS_HILLY) && find_skill_by_number (op, SK_CLIMBING)) ||
                     (QUERY_FLAG (tmp, FLAG_IS_WOODED) && find_skill_by_number (op, SK_WOODSMAN)))
                   diff /= 4.0;
               op->speed_left -= diff;
  * there is capable of.
  */
 int
 find_dir (maptile *m, int x, int y, object *exclude)
 {
-  int i, max = SIZEOFFREE, mflags;
+  int max = SIZEOFFREE, mflags;
-  sint16 nx, ny;
-  object *tmp;
-  maptile *mp;
-  MoveType blocked, move_type;
+  MoveType move_type;
   if (exclude && exclude->head_ () != exclude)
     {
       exclude = exclude->head;
       move_type = exclude->move_type;
     {
       /* If we don't have anything, presume it can use all movement types. */
       move_type = MOVE_ALL;
     }
-  for (i = 1; i < max; i++)
+  for (int i = 1; i < max; i++)
     {
-      mp = m;
+      mapxy pos (m, x, y);
-      nx = x + freearr_x[i];
+      pos.move (i);
-      ny = y + freearr_y[i];
-      mflags = get_map_flags (m, &mp, nx, ny, &nx, &ny);
+      if (!pos.normalise ())
-      if (mflags & P_OUT_OF_MAP)
         max = maxfree[i];
       else
         {
-          mapspace &ms = mp->at (nx, ny);
+          mapspace &ms = *pos;
-          blocked = ms.move_block;
-          if ((move_type & blocked) == move_type)
+          if ((move_type & ms.move_block) == move_type)
-            max = maxfree[i];
+            max = maxfree [i];
-          else if (mflags & P_IS_ALIVE)
+          else if (ms.flags () & P_IS_ALIVE)
             {
-              for (tmp = ms.bot; tmp; tmp = tmp->above)
+              for (object *tmp = ms.bot; tmp; tmp = tmp->above)
-                if ((tmp->flag [FLAG_MONSTER] || tmp->type == PLAYER)
+                if ((tmp->flag [FLAG_MONSTER] || tmp->is_player ())
                     && (tmp != exclude || (tmp->head_ () != tmp && tmp->head_ () != exclude)))
-                  break;
-              if (tmp)
-                return freedir[i];
+                  return freedir [i];
             }
         }
     }
   return 0;
  * This basically means that if direction is 15, then it could either go
  * direction 4, 14, or 16 to get back to where we are.
  * Moved from spell_util.c to object.c with the other related direction
  * functions.
  */
-int reduction_dir[SIZEOFFREE][3] = {
+const int reduction_dir[SIZEOFFREE][3] = {
   {0, 0, 0},                    /* 0 */
   {0, 0, 0},                    /* 1 */
   {0, 0, 0},                    /* 2 */
   {0, 0, 0},                    /* 3 */
   {0, 0, 0},                    /* 4 */
 int
 can_pick (const object *who, const object *item)
 {
   return                        /*QUERY_FLAG(who,FLAG_WIZ)|| */
     (item->weight > 0 && !QUERY_FLAG (item, FLAG_NO_PICK) &&
-     !QUERY_FLAG (item, FLAG_ALIVE) && !item->invisible && (who->type == PLAYER || item->weight < who->weight / 3));
+     !QUERY_FLAG (item, FLAG_ALIVE) && !item->invisible && (who->is_player () || item->weight < who->weight / 3));
 }
 /*
  * create clone from object to another
  */
       return tmp;
   return 0;
 }
-const shstr &
+shstr_tmp
-object::kv_get (const shstr &key) const
+object::kv_get (shstr_tmp key) const
 {
   for (key_value *kv = key_values; kv; kv = kv->next)
     if (kv->key == key)
       return kv->value;
-  return shstr_null;
+  return shstr ();
 }
 void
-object::kv_set (const shstr &key, const shstr &value)
+object::kv_set (shstr_tmp key, shstr_tmp value)
 {
   for (key_value *kv = key_values; kv; kv = kv->next)
     if (kv->key == key)
       {
         kv->value = value;
   key_values = kv;
 }
 void
-object::kv_del (const shstr &key)
+object::kv_del (shstr_tmp key)
 {
   for (key_value **kvp = &key_values; *kvp; kvp = &(*kvp)->next)
     if ((*kvp)->key == key)
       {
         key_value *kv = *kvp;
       container = 0;
       // client needs item update to make it work, client bug requires this to be separate
       esrv_update_item (UPD_FLAGS, this, old_container);
-      new_draw_info_format (NDI_UNIQUE, 0, this, "You close %s.", query_name (old_container));
+      new_draw_info_format (NDI_UNIQUE, 0, this, "You close %s.", old_container->query_name ());
       play_sound (sound_find ("chest_close"));
     }
   if (new_container)
     {
           closer->flag [FLAG_NO_MAP_SAVE] = 1;
           new_container->insert (closer);
         }
 #endif
-      new_draw_info_format (NDI_UNIQUE, 0, this, "You open %s.", query_name (new_container));
+      new_draw_info_format (NDI_UNIQUE, 0, this, "You open %s.", new_container->query_name ());
       // make sure the container is available, client bug requires this to be separate
       esrv_send_item (this, new_container);
       new_container->flag [FLAG_APPLIED] = true;
 //  else if (!old_container->env && contr && contr->ns)
 //    contr->ns->floorbox_reset ();
 }
 object *
-object::force_find (const shstr name)
+object::force_find (shstr_tmp name)
 {
   /* cycle through his inventory to look for the MARK we want to
    * place
    */
   for (object *tmp = inv; tmp; tmp = tmp->below)
   return 0;
 }
 object *
-object::force_add (const shstr name, int duration)
+object::force_add (shstr_tmp name, int duration)
 {
   if (object *force = force_find (name))
     force->destroy ();
   object *force = get_archetype (FORCE_NAME);
   return insert (force);
 }
 void
-object::play_sound (faceidx sound)
+object::play_sound (faceidx sound) const
 {
   if (!sound)
     return;
-  if (flag [FLAG_REMOVED])
+  if (is_on_map ())
-    return;
-  if (env)
-    {
-      if (object *pl = in_player ())
-        pl->contr->play_sound (sound);
-    }
-  else
     map->play_sound (sound, x, y);
+  else if (object *pl = in_player ())
+    pl->contr->play_sound (sound);
+}
+void
+object::say_msg (const char *msg) const
+{
+  if (is_on_map ())
+    map->say_msg (msg, x, y);
+  else if (object *pl = in_player ())
+    pl->contr->play_sound (sound);
 }
 void
 object::make_noise ()
 {
   // we do not model noise in the map, so instead put
   // a temporary light into the noise source
   // could use the map instead, but that's less reliable for our
   // goal, which is to make invisibility a bit harder to exploit
+  // currently only works sensibly for players
+  if (!is_player ())
+    return;
   // find old force, or create new one
   object *force = force_find (shstr_noise_force);
   if (force)
-    force->speed = 1.f / 4; // patch old speed up
+    force->speed_left = -1.f; // patch old speed up
   else
     {
-      force = get_archetype (shstr_noise_force);
+      force = archetype::get (shstr_noise_force);
-      force->slaying    = name;
+      force->slaying    = shstr_noise_force;
       force->stats.food = 1;
       force->speed_left = -1.f;
-      force->set_speed (1.f / 4);
+      force->set_speed (1.f / 4.f);
       force->flag [FLAG_IS_USED_UP] = true;
       force->flag [FLAG_APPLIED]    = true;
       insert (force);
     }

Diff Legend

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

Comparing deliantra/server/common/object.C (file contents): Revision 1.265 by root, Tue Dec 23 00:39:48 2008 UTC vs. Revision 1.282 by root, Tue Feb 17 03:53:31 2009 UTC

Diff Legend

Comparing deliantra/server/common/object.C (file contents):
Revision 1.265 by root, Tue Dec 23 00:39:48 2008 UTC vs.
Revision 1.282 by root, Tue Feb 17 03:53:31 2009 UTC