--- deliantra/server/common/object.C 2008/05/22 15:37:44 1.246
+++ deliantra/server/common/object.C 2009/11/07 18:30:05 1.297
@@ -1,22 +1,23 @@
/*
* 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
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
+ * Deliantra is free software: you can redistribute it and/or modify it under
+ * the terms of the Affero GNU General Public License as published by the
+ * Free Software Foundation, either version 3 of the License, or (at your
+ * option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
- * You should have received a copy of the GNU General Public License
- * along with this program. If not, see .
+ * You should have received a copy of the Affero GNU General Public License
+ * and the GNU General Public License along with this program. If not, see
+ * .
*
* The authors can be reached via e-mail to
*/
@@ -34,10 +35,13 @@
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;
+//+GPL
+
short freearr_x[SIZEOFFREE] = {
0,
0, 1, 1, 1, 0, -1, -1, -1,
@@ -50,12 +54,6 @@
-2, -2, -2, -1, 0, 1, 2, 2, 2, 2, 2, 1, 0, -1, -2, -2,
-3, -3, -3, -3, -2, -1, 0, 1, 2, 3, 3, 3, 3, 3, 3, 3, 2, 1, 0, -1, -2, -3, -3, -3
};
-int maxfree[SIZEOFFREE] = {
- 0,
- 9, 10, 13, 14, 17, 18, 21, 22,
- 25, 26, 27, 30, 31, 32, 33, 36, 37, 39, 39, 42, 43, 44, 45, 48,
- 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49
-};
int freedir[SIZEOFFREE] = {
0,
1, 2, 3, 4, 5, 6, 7, 8,
@@ -63,6 +61,13 @@
1, 2, 2, 2, 2, 2, 3, 4, 4, 4, 4, 4, 5, 6, 6, 6, 6, 6, 7, 8, 8, 8, 8, 8
};
+static int maxfree[SIZEOFFREE] = {
+ 0,
+ 9, 10, 13, 14, 17, 18, 21, 22,
+ 25, 26, 27, 30, 31, 32, 33, 36, 37, 39, 39, 42, 43, 44, 45, 48,
+ 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49
+};
+
static void
write_uuid (uval64 skip, bool sync)
{
@@ -183,17 +188,14 @@
/* A couple quicksanity checks */
if (ob1 == ob2
|| ob1->type != ob2->type
- || ob1->speed != ob2->speed
+ || fabs (ob1->speed - ob2->speed) < MIN_ACTIVE_SPEED
|| ob1->value != ob2->value
|| ob1->name != ob2->name)
return 0;
- /* Do not merge objects if nrof would overflow. First part checks
- * for unsigned overflow (2c), second part checks whether the result
- * would fit into a 32 bit signed int, which is often used to hold
- * nrof values.
- */
- if (~ob1->nrof < ob2->nrof || ob1->nrof + ob2->nrof > (1UL << 31))
+ /* Do not merge objects if nrof would overflow, assume nrof
+ * is always 0 .. 2**31-1 */
+ if (ob1->nrof > 0x7fffffff - ob2->nrof)
return 0;
/* If the objects have been identified, set the BEEN_APPLIED flag.
@@ -230,14 +232,13 @@
|| ob1->move_on != ob2->move_on
|| ob1->move_off != ob2->move_off
|| ob1->move_slow != ob2->move_slow
- || ob1->move_slow_penalty != ob2->move_slow_penalty
+ || fabs (ob1->move_slow_penalty - ob2->move_slow_penalty) < (1.f / 1024.f)
|| memcmp (&ob1->resist, &ob2->resist, sizeof (ob1->resist))
|| memcmp (&ob1->stats , &ob2->stats , sizeof (ob1->stats)))
return 0;
if ((ob1->flag ^ ob2->flag)
.reset (FLAG_INV_LOCKED)
- .reset (FLAG_CLIENT_SENT)
.reset (FLAG_REMOVED)
.any ())
return 0;
@@ -329,17 +330,17 @@
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 ();
+ object *envest = env->outer_env_or_self ();
// 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)
+ if (pl->container_ () == env)
return pl;
}
else
@@ -347,7 +348,8 @@
// 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 ())
- if (!pl->container || this == pl->container)
+ if ((pl->contr->ns && !pl->container_ () && !pl->contr->ns->update_look)
+ || pl->container_ () == this)
return pl;
}
}
@@ -435,27 +437,10 @@
return freezer.as_string ();
}
-/*
- * get_nearest_part(multi-object, object 2) returns the part of the
- * multi-object 1 which is closest to the second object.
- * If it's not a multi-object, it is returned.
- */
-object *
-get_nearest_part (object *op, const object *pl)
+char *
+object::as_string ()
{
- object *tmp, *closest;
- int last_dist, i;
-
- if (!op->more)
- return op;
-
- for (last_dist = distance (op, pl), closest = op, tmp = op->more;
- tmp;
- tmp = tmp->more)
- if ((i = distance (tmp, pl)) < last_dist)
- closest = tmp, last_dist = i;
-
- return closest;
+ return dump_object (this);
}
/*
@@ -559,7 +544,8 @@
new_draw_info_format (NDI_UNIQUE, 0, this,
"You try to balance all your items at once, "
"but the %s is just too much for your body. "
- "[You need to unapply some items first.]", &ob->name);
+ "[You need to unapply some items first - use the 'body' command to see "
+ "how many items you cna wera on a specific body part.]", &ob->name);
return false;
}
@@ -641,7 +627,7 @@
if (speed < 0)
dst->speed_left -= rndm ();
- dst->set_speed (dst->speed);
+ dst->activate ();
}
void
@@ -695,12 +681,6 @@
void
object::set_speed (float speed)
{
- if (flag [FLAG_FREED] && speed)
- {
- LOG (llevError, "Object %s is freed but has speed.\n", &name);
- speed = 0;
- }
-
this->speed = speed;
if (has_active_speed ())
@@ -760,10 +740,11 @@
/* nop */;
else if (action == UP_OBJ_INSERT)
{
+#if 0
// 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))
@@ -771,18 +752,21 @@
|| (m.move_off | op->move_off ) != m.move_off
|| (m.move_slow | op->move_slow) != m.move_slow
/* This isn't perfect, but I don't expect a lot of objects to
- * to have move_allow right now.
+ * have move_allow right now.
*/
|| ((m.move_block | op->move_block) & ~op->move_allow) != m.move_block
- || 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp)
- m.flags_ = 0;
+ m.invalidate ();
+#else
+ // the above is not strong enough a test to skip updating. los maybe? TODO (schmorp)
+ m.invalidate ();
+#endif
}
/* if the object is being removed, we can't make intelligent
* decisions, because remove_ob can't really pass the object
* that is being removed.
*/
else if (action == UP_OBJ_CHANGE || action == UP_OBJ_REMOVE)
- m.flags_ = 0;
+ m.invalidate ();
else if (action == UP_OBJ_FACE)
/* Nothing to do for that case */ ;
else
@@ -796,7 +780,7 @@
{
SET_FLAG (this, FLAG_REMOVED);
- expmul = 1.0;
+ //expmul = 1.0; declared const for the time being
face = blank_face;
}
@@ -836,7 +820,12 @@
return;
if (has_active_speed ())
- actives.insert (this);
+ {
+ if (flag [FLAG_FREED])
+ LOG (llevError | logBacktrace, "BUG: tried to activate freed object %s\n", debug_desc ());//D
+
+ actives.insert (this);
+ }
}
void
@@ -911,10 +900,7 @@
|| ms ().move_block == MOVE_ALL)
{
while (inv)
- {
- inv->destroy_inv (false);
- inv->destroy ();
- }
+ inv->destroy ();
}
else
{ /* Put objects in inventory onto this space */
@@ -928,7 +914,7 @@
|| op->type == TRAP
|| op->flag [FLAG_IS_A_TEMPLATE]
|| op->flag [FLAG_DESTROY_ON_DEATH])
- op->destroy (true);
+ op->destroy ();
else
map->insert (op, x, y);
}
@@ -967,7 +953,7 @@
object::do_destroy ()
{
if (flag [FLAG_IS_LINKED])
- remove_button_link (this);
+ remove_link ();
if (flag [FLAG_FRIENDLY])
remove_friendly_object (this);
@@ -1002,7 +988,7 @@
}
void
-object::destroy (bool destroy_inventory)
+object::destroy ()
{
if (destroyed ())
return;
@@ -1010,11 +996,11 @@
if (!is_head () && !head->destroyed ())
{
LOG (llevError | logBacktrace, "tried to destroy the tail of an object");
- head->destroy (destroy_inventory);
+ head->destroy ();
return;
}
- destroy_inv (!destroy_inventory);
+ destroy_inv (false);
if (is_head ())
if (sound_destroy)
@@ -1035,9 +1021,6 @@
void
object::do_remove ()
{
- object *tmp, *last = 0;
- object *otmp;
-
if (flag [FLAG_REMOVED])
return;
@@ -1061,6 +1044,8 @@
adjust_weight (env, -total_weight ());
+ object *pl = in_player ();
+
/* we set up values so that it could be inserted into
* the map, but we don't actually do that - it is up
* to the caller to decide what we want to do.
@@ -1081,8 +1066,14 @@
* made to players inventory. If set, avoiding the call
* to save cpu time.
*/
- if ((otmp = in_player ()) && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
- otmp->update_stats ();
+ if (pl)
+ if (pl->is_player () && (glow_radius || !QUERY_FLAG (pl, FLAG_NO_FIX_PLAYER)))
+ {
+ pl->update_stats ();
+
+ if (glow_radius && pl->is_on_map ())
+ update_all_los (pl->map, pl->x, pl->y);
+ }
}
else if (map)
{
@@ -1091,8 +1082,11 @@
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
@@ -1102,7 +1096,7 @@
--map->players;
map->touch ();
}
- else if (pl->container == this)
+ else if (pl->container_ () == this)
{
// removing a container should close it
close_container ();
@@ -1119,6 +1113,8 @@
above = 0;
below = 0;
+ ms.invalidate ();
+
if (map->in_memory == MAP_SAVING)
return;
@@ -1126,7 +1122,7 @@
if (object *pl = ms.player ())
{
- if (pl->container == this)
+ if (pl->container_ () == this)
/* If a container that the player is currently using somehow gets
* removed (most likely destroyed), update the player view
* appropriately.
@@ -1140,34 +1136,22 @@
pl->contr->ns->floorbox_update ();
}
- for (tmp = ms.bot; tmp; tmp = tmp->above)
- {
- /* No point updating the players look faces if he is the object
- * being removed.
- */
-
- /* See if object moving off should effect something */
- if (check_walk_off
- && ((move_type & tmp->move_off)
- && (move_type & ~tmp->move_off & ~tmp->move_block) == 0))
- {
- move_apply (tmp, this, 0);
-
- if (destroyed ())
- LOG (llevError, "BUG: remove_ob(): name %s, destroyed leaving object\n", tmp->debug_desc ());
- }
+ if (check_walk_off)
+ for (object *above, *tmp = ms.bot; tmp; tmp = above)
+ {
+ above = tmp->above;
- last = tmp;
- }
+ /* No point updating the players look faces if he is the object
+ * being removed.
+ */
- /* last == NULL if there are no objects on this space */
- //TODO: this makes little sense, why only update the topmost object?
- if (!last)
- map->at (x, y).flags_ = 0;
- else
- update_object (last, UP_OBJ_REMOVE);
+ /* See if object moving off should effect something */
+ if ((move_type & tmp->move_off)
+ && (move_type & ~tmp->move_off & ~tmp->move_block) == 0)
+ move_apply (tmp, this, 0);
+ }
- if (flag [FLAG_BLOCKSVIEW] || glow_radius)
+ if (affects_los ())
update_all_los (map, x, y);
}
}
@@ -1201,7 +1185,7 @@
op->weight = 0; // cancel the addition above
op->carrying = 0; // must be 0 already
- op->destroy (1);
+ op->destroy ();
return top;
}
@@ -1273,31 +1257,21 @@
object *
insert_ob_in_map (object *op, maptile *m, object *originator, int flag)
{
- if (op->is_on_map ())
- {
- LOG (llevError, "insert_ob_in_map called for object already on map");
- abort ();
- }
-
- if (op->env)
- {
- LOG (llevError, "insert_ob_in_map called for object in an inventory (proceeding)");
- op->remove ();
- }
+ op->remove ();
- if (op->face && !face_info (op->face))//D TODO: remove soon
+ if (m == &freed_map)//D TODO: remove soon
{//D
- LOG (llevError | logBacktrace, "op->face out of bounds: %s", op->debug_desc ());//D
- op->face = 1;//D
+ LOG (llevError | logBacktrace, "tries to insret object on freed objects map: %s", op->debug_desc ());//D
}//D
/* 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))
+ maptile *newmap = m;
+ if (!xy_normalise (newmap, op->x, op->y))
{
- op->head_ ()->destroy (1);// remove head_ once all tail object destroyers found
+ op->head_ ()->destroy ();// remove head_ once all tail object destroyers found
return 0;
}
@@ -1305,9 +1279,10 @@
if (!insert_ob_in_map (more, m, originator, flag))
return 0;
- CLEAR_FLAG (op, FLAG_REMOVED);
+ op->flag [FLAG_REMOVED] = false;
+ op->env = 0;
+ op->map = newmap;
- op->map = m;
mapspace &ms = op->ms ();
/* this has to be done after we translate the coordinates.
@@ -1320,7 +1295,7 @@
// but some caller surely breaks when we return tmp
// from here :/
op->nrof += tmp->nrof;
- tmp->destroy (1);
+ tmp->destroy ();
}
CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */
@@ -1338,8 +1313,11 @@
}
if (!originator->is_on_map ())
- LOG (llevDebug | logBacktrace, "insert_ob_in_map(%s) called with INS_BELOW_ORIGINATOR when originator '%s' not on map",
- op->debug_desc (), originator->debug_desc ());
+ {
+ LOG (llevError, "insert_ob_in_map(%s) called with INS_BELOW_ORIGINATOR when originator '%s' not on map",
+ op->debug_desc (), originator->debug_desc ());
+ abort ();
+ }
op->above = originator;
op->below = originator->below;
@@ -1435,7 +1413,7 @@
}
}
- if (op->type == PLAYER)
+ if (op->is_player ())
{
op->contr->do_los = 1;
++op->map->players;
@@ -1460,8 +1438,11 @@
* or just updating the P_UPTODATE for spaces within this area
* of effect may be sufficient.
*/
- if (op->map->darkness && (op->glow_radius != 0))
- update_all_los (op->map, op->x, op->y);
+ if (op->affects_los ())
+ {
+ op->ms ().invalidate ();
+ update_all_los (op->map, op->x, op->y);
+ }
/* updates flags (blocked, alive, no magic, etc) for this map space */
update_object (op, UP_OBJ_INSERT);
@@ -1478,7 +1459,7 @@
*/
/* if this is not the head or flag has been passed, don't check walk on status */
- if (!(flag & INS_NO_WALK_ON) && op->head_ () == op)
+ if (!(flag & INS_NO_WALK_ON) && op->is_head ())
{
if (check_move_on (op, originator))
return 0;
@@ -1499,15 +1480,15 @@
* 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 */
- tmp->destroy (1);
+ 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;
@@ -1539,11 +1520,10 @@
nr = min (nr, nrof);
- nrof -= nr;
-
- if (nrof)
+ if (nrof > nr)
{
- adjust_weight (env, -weight * nr); // carrying == 0
+ nrof -= nr;
+ adjust_weight (env, -weight * max (1, nr)); // carrying == 0
if (object *pl = visible_to ())
esrv_update_item (UPD_NROF, pl, this);
@@ -1552,7 +1532,7 @@
}
else
{
- destroy (1);
+ destroy ();
return false;
}
}
@@ -1639,7 +1619,7 @@
adjust_weight (this, op->total_weight ());
- op->destroy (1);
+ op->destroy ();
op = tmp;
goto inserted;
}
@@ -1667,11 +1647,13 @@
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 this is a player's inventory, update stats
- if (type == PLAYER && !flag [FLAG_NO_FIX_PLAYER])
+ if (op->glow_radius && is_on_map ())
+ {
+ update_stats ();
+ update_all_los (map, x, y);
+ }
+ else if (is_player () && !flag [FLAG_NO_FIX_PLAYER])
+ // if this is a player's inventory, update stats
update_stats ();
INVOKE_OBJECT (INSERT, this);
@@ -1702,18 +1684,20 @@
int
check_move_on (object *op, object *originator)
{
+ if (QUERY_FLAG (op, FLAG_NO_APPLY))
+ return 0;
+
object *tmp;
maptile *m = op->map;
int x = op->x, y = op->y;
- MoveType move_on, move_slow, move_block;
+ mapspace &ms = m->at (x, y);
- if (QUERY_FLAG (op, FLAG_NO_APPLY))
- return 0;
+ ms.update ();
- move_on = GET_MAP_MOVE_ON (op->map, op->x, op->y);
- move_slow = GET_MAP_MOVE_SLOW (op->map, op->x, op->y);
- move_block = GET_MAP_MOVE_BLOCK (op->map, op->x, op->y);
+ MoveType move_on = ms.move_on;
+ MoveType move_slow = ms.move_slow;
+ MoveType move_block = ms.move_block;
/* if nothing on this space will slow op down or be applied,
* no need to do checking below. have to make sure move_type
@@ -1734,19 +1718,10 @@
/* The objects have to be checked from top to bottom.
* Hence, we first go to the top:
*/
-
- for (tmp = op->ms ().bot; tmp && tmp->above; tmp = tmp->above)
+ for (object *next, *tmp = ms.top; tmp; tmp = next)
{
- /* Trim the search when we find the first other spell effect
- * this helps performance so that if a space has 50 spell objects,
- * we don't need to check all of them.
- */
- if ((tmp->move_type & MOVE_FLY_LOW) && QUERY_FLAG (tmp, FLAG_NO_PICK))
- break;
- }
+ next = tmp->below;
- for (; tmp; tmp = tmp->below)
- {
if (tmp == op)
continue; /* Can't apply yourself */
@@ -1761,13 +1736,11 @@
if ((!op->move_type && tmp->move_slow & MOVE_WALK) ||
((op->move_type & tmp->move_slow) && (op->move_type & ~tmp->move_slow & ~tmp->move_block) == 0))
{
+ float diff = tmp->move_slow_penalty * fabs (op->speed);
- float
- diff = tmp->move_slow_penalty * fabs (op->speed);
-
- if (op->type == 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)))
+ if (op->is_player ())
+ if ((tmp->flag [FLAG_IS_HILLY ] && find_skill_by_number (op, SK_CLIMBING)) ||
+ (tmp->flag [FLAG_IS_WOODED] && find_skill_by_number (op, SK_WOODSMAN)))
diff /= 4.0;
op->speed_left -= diff;
@@ -1983,7 +1956,7 @@
if (OB_TYPE_MOVE_BLOCK (ob, ms.move_block))
continue;
- if (ob->blocked (m, pos.x, pos.y))
+ if (ob->blocked (pos.m, pos.x, pos.y))
continue;
altern [index++] = i;
@@ -2061,13 +2034,8 @@
int
find_dir (maptile *m, int x, int y, object *exclude)
{
- int i, max = SIZEOFFREE, mflags;
-
- sint16 nx, ny;
- object *tmp;
- maptile *mp;
-
- MoveType blocked, move_type;
+ int max = SIZEOFFREE, mflags;
+ MoveType move_type;
if (exclude && exclude->head_ () != exclude)
{
@@ -2080,33 +2048,25 @@
move_type = MOVE_ALL;
}
- for (i = 1; i < max; i++)
+ for (int i = 1; i < max; i++)
{
- mp = m;
- nx = x + freearr_x[i];
- ny = y + freearr_y[i];
-
- mflags = get_map_flags (m, &mp, nx, ny, &nx, &ny);
+ mapxy pos (m, x, y);
+ pos.move (i);
- if (mflags & P_OUT_OF_MAP)
+ if (!pos.normalise ())
max = maxfree[i];
else
{
- mapspace &ms = mp->at (nx, ny);
-
- blocked = ms.move_block;
+ mapspace &ms = *pos;
- if ((move_type & blocked) == move_type)
- max = maxfree[i];
- else if (mflags & P_IS_ALIVE)
+ if ((move_type & ms.move_block) == move_type)
+ max = maxfree [i];
+ else if (ms.flags () & P_IS_ALIVE)
{
- for (tmp = ms.bot; tmp; tmp = tmp->above)
- if ((tmp->flag [FLAG_MONSTER] || tmp->type == PLAYER)
+ for (object *tmp = ms.bot; tmp; tmp = tmp->above)
+ 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];
}
}
}
@@ -2191,7 +2151,7 @@
* Moved from spell_util.c to object.c with the other related direction
* functions.
*/
-int reduction_dir[SIZEOFFREE][3] = {
+static const int reduction_dir[SIZEOFFREE][3] = {
{0, 0, 0}, /* 0 */
{0, 0, 0}, /* 1 */
{0, 0, 0}, /* 2 */
@@ -2297,7 +2257,7 @@
{
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));
}
/*
@@ -2339,18 +2299,18 @@
return 0;
}
-const shstr &
-object::kv_get (const shstr &key) const
+shstr_tmp
+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)
@@ -2369,7 +2329,7 @@
}
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)
@@ -2510,7 +2470,7 @@
// 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"));
}
@@ -2530,7 +2490,7 @@
}
#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);
@@ -2548,7 +2508,7 @@
}
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
@@ -2560,40 +2520,83 @@
return 0;
}
+//-GPL
+
void
-object::force_add (const shstr name, int duration)
+object::force_set_timer (int duration)
+{
+ this->duration = 1;
+ this->speed_left = -1.f;
+
+ this->set_speed (duration ? 1.f / duration : 0.f);
+}
+
+object *
+object::force_add (shstr_tmp name, int duration)
{
if (object *force = force_find (name))
force->destroy ();
object *force = get_archetype (FORCE_NAME);
- force->slaying = name;
- force->stats.food = 1;
- force->speed_left = -1.f;
+ force->slaying = name;
+ force->force_set_timer (duration);
+ force->flag [FLAG_APPLIED] = true;
- force->set_speed (duration ? 1.f / duration : 0.f);
- force->flag [FLAG_IS_USED_UP] = true;
- force->flag [FLAG_APPLIED] = true;
-
- insert (force);
+ 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 ())
+ 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;
- if (env)
+ // find old force, or create new one
+ object *force = force_find (shstr_noise_force);
+
+ if (force)
+ force->speed_left = -1.f; // patch old speed up
+ else
{
- if (object *pl = in_player ())
- pl->contr->play_sound (sound);
+ force = archetype::get (shstr_noise_force);
+
+ force->slaying = shstr_noise_force;
+ force->stats.food = 1;
+ force->speed_left = -1.f;
+
+ force->set_speed (1.f / 4.f);
+ force->flag [FLAG_IS_USED_UP] = true;
+ force->flag [FLAG_APPLIED] = true;
+
+ insert (force);
}
- else
- map->play_sound (sound, x, y);
}