ViewVC Help

View File | Revision Log | Show Annotations | Download File

/cvs/deliantra/server/common/object.C

Comparing deliantra/server/common/object.C (file contents):
Revision 1.206 by root, Tue Apr 15 14:21:04 2008 UTC vs.
Revision 1.254 by root, Sun Aug 31 01:17:11 2008 UTC

1	/*	1	/*
2	* This file is part of Deliantra, the Roguelike Realtime MMORPG.	2	* This file is part of Deliantra, the Roguelike Realtime MMORPG.
3	*	3	*
4	* Copyright (©) 2005,2006,2007 Marc Alexander Lehmann / Robin Redeker / the Deliantra team	4	* Copyright (©) 2005,2006,2007,2008 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
5	* Copyright (©) 2001,2007 Mark Wedel & Crossfire Development Team	5	* Copyright (©) 2001,2007 Mark Wedel & Crossfire Development Team
6	* Copyright (©) 1992,2007 Frank Tore Johansen	6	* Copyright (©) 1992,2007 Frank Tore Johansen
7	*	7	*
8	* Deliantra is free software: you can redistribute it and/or modify	8	* Deliantra is free software: you can redistribute it and/or modify
9	* it under the terms of the GNU General Public License as published by	9	* it under the terms of the GNU General Public License as published by
…		…
19	* along with this program. If not, see <http://www.gnu.org/licenses/>.	19	* along with this program. If not, see <http://www.gnu.org/licenses/>.
20	*	20	*
21	* The authors can be reached via e-mail to <support@deliantra.net>	21	* The authors can be reached via e-mail to <support@deliantra.net>
22	*/	22	*/
23		23
24	/* Eneq(@csd.uu.se): Added weight-modifiers in environment of objects.
25	sub/add_weight will transcend the environment updating the carrying
26	variable. */
27	#include <global.h>	24	#include <global.h>
28	#include <stdio.h>	25	#include <stdio.h>
29	#include <sys/types.h>	26	#include <sys/types.h>
30	#include <sys/uio.h>	27	#include <sys/uio.h>
31	#include <object.h>	28	#include <object.h>
32	#include <funcpoint.h>
33	#include <sproto.h>	29	#include <sproto.h>
34	#include <loader.h>	30	#include <loader.h>
35		31
36	#include <bitset>	32	#include <bitset>
37		33
…		…
143		139
144	/* Returns TRUE if every key_values in wants has a partner with the same value in has. */	140	/* Returns TRUE if every key_values in wants has a partner with the same value in has. */
145	static bool	141	static bool
146	compare_ob_value_lists_one (const object *wants, const object *has)	142	compare_ob_value_lists_one (const object *wants, const object *has)
147	{	143	{
148	key_value *wants_field;
149
150	/* n-squared behaviour (see get_ob_key_link()), but I'm hoping both	144	/* n-squared behaviour (see kv_get), but I'm hoping both
151	* objects with lists are rare, and lists stay short. If not, use a	145	* objects with lists are rare, and lists stay short. If not, use a
152	* different structure or at least keep the lists sorted...	146	* different structure or at least keep the lists sorted...
153	*/	147	*/
154		148
155	/* For each field in wants, */	149	/* For each field in wants, */
156	for (wants_field = wants->key_values; wants_field; wants_field = wants_field->next)	150	for (key_value *kv = wants->key_values; kv; kv = kv->next)
157	{	151	if (has->kv_get (kv->key) != kv->value)
158	key_value *has_field;	152	return false;
159
160	/* Look for a field in has with the same key. */
161	has_field = get_ob_key_link (has, wants_field->key);
162
163	if (!has_field)
164	return 0; /* No field with that name. */
165
166	/* Found the matching field. */
167	if (has_field->value != wants_field->value)
168	return 0; /* Values don't match, so this half of the comparison is false. */
169
170	/* If we get here, we found a match. Now for the next field in wants. */
171	}
172		153
173	/* If we get here, every field in wants has a matching field in has. */	154	/* If we get here, every field in wants has a matching field in has. */
174	return 1;	155	return true;
175	}	156	}
176		157
177	/* Returns TRUE if ob1 has the same key_values as ob2. */	158	/* Returns TRUE if ob1 has the same key_values as ob2. */
178	static bool	159	static bool
179	compare_ob_value_lists (const object *ob1, const object *ob2)	160	compare_ob_value_lists (const object *ob1, const object *ob2)
180	{	161	{
181	/* However, there may be fields in has which aren't partnered in wants,	162	/* However, there may be fields in has which aren't partnered in wants,
182	* so we need to run the comparison *twice*. :(	163	* so we need to run the comparison *twice*. :(
183	*/	164	*/
184	return compare_ob_value_lists_one (ob1, ob2) && compare_ob_value_lists_one (ob2, ob1);	165	return compare_ob_value_lists_one (ob1, ob2)
		166	&& compare_ob_value_lists_one (ob2, ob1);
185	}	167	}
186		168
187	/* Function examines the 2 objects given to it, and returns true if	169	/* Function examines the 2 objects given to it, and returns true if
188	* they can be merged together.	170	* they can be merged together.
189	*	171	*
…		…
204	|| ob1->speed != ob2->speed	186	|| ob1->speed != ob2->speed
205	|| ob1->value != ob2->value	187	|| ob1->value != ob2->value
206	|| ob1->name != ob2->name)	188	|| ob1->name != ob2->name)
207	return 0;	189	return 0;
208		190
209	/* Do not merge objects if nrof would overflow. First part checks	191	/* Do not merge objects if nrof would overflow, assume nrof
210	* for unsigned overflow (2c), second part checks wether the result	192	* is always 0 .. 2**31-1 */
211	* would fit into a 32 bit signed int, which is often used to hold	193	if (ob1->nrof > 0x7fffffff - ob2->nrof)
212	* nrof values.
213	*/
214	if (~ob1->nrof < ob2->nrof || ob1->nrof + ob2->nrof > (1UL << 31))
215	return 0;	194	return 0;
216		195
217	/* If the objects have been identified, set the BEEN_APPLIED flag.	196	/* If the objects have been identified, set the BEEN_APPLIED flag.
218	* This is to the comparison of the flags below will be OK. We	197	* This is to the comparison of the flags below will be OK. We
219	* just can't ignore the been applied or identified flags, as they	198	* just can't ignore the been applied or identified flags, as they
…		…
224	SET_FLAG (ob1, FLAG_BEEN_APPLIED);	203	SET_FLAG (ob1, FLAG_BEEN_APPLIED);
225		204
226	if (QUERY_FLAG (ob2, FLAG_IDENTIFIED))	205	if (QUERY_FLAG (ob2, FLAG_IDENTIFIED))
227	SET_FLAG (ob2, FLAG_BEEN_APPLIED);	206	SET_FLAG (ob2, FLAG_BEEN_APPLIED);
228		207
229	if ((ob1->flag ^ ob2->flag).reset (FLAG_INV_LOCKED).reset (FLAG_CLIENT_SENT).any ()	208	if (ob1->arch->archname != ob2->arch->archname
230	|| ob1->arch != ob2->arch
231	|| ob1->name != ob2->name	209	|| ob1->name != ob2->name
232	|| ob1->title != ob2->title	210	|| ob1->title != ob2->title
233	|| ob1->msg != ob2->msg	211	|| ob1->msg != ob2->msg
234	|| ob1->weight != ob2->weight	212	|| ob1->weight != ob2->weight
235	|| memcmp (&ob1->resist, &ob2->resist, sizeof (ob1->resist))
236	|| memcmp (&ob1->stats , &ob2->stats , sizeof (ob1->stats) )
237	|| ob1->attacktype != ob2->attacktype	213	|| ob1->attacktype != ob2->attacktype
238	|| ob1->magic != ob2->magic	214	|| ob1->magic != ob2->magic
239	|| ob1->slaying != ob2->slaying	215	|| ob1->slaying != ob2->slaying
240	|| ob1->skill != ob2->skill	216	|| ob1->skill != ob2->skill
241	|| ob1->value != ob2->value	217	|| ob1->value != ob2->value
242	|| ob1->animation_id != ob2->animation_id	218	|| ob1->animation_id != ob2->animation_id
		219	|| (ob1->face != ob2->face && !ob1->animation_id) // face and animation are dependent on each other
243	|| ob1->client_type != ob2->client_type	220	|| ob1->client_type != ob2->client_type
244	|| ob1->materialname != ob2->materialname	221	|| ob1->materialname != ob2->materialname
245	|| ob1->lore != ob2->lore	222	|| ob1->lore != ob2->lore
246	|| ob1->subtype != ob2->subtype	223	|| ob1->subtype != ob2->subtype
247	|| ob1->move_type != ob2->move_type	224	|| ob1->move_type != ob2->move_type
248	|| ob1->move_block != ob2->move_block	225	|| ob1->move_block != ob2->move_block
249	|| ob1->move_allow != ob2->move_allow	226	|| ob1->move_allow != ob2->move_allow
250	|| ob1->move_on != ob2->move_on	227	|| ob1->move_on != ob2->move_on
251	|| ob1->move_off != ob2->move_off	228	|| ob1->move_off != ob2->move_off
252	|| ob1->move_slow != ob2->move_slow	229	|| ob1->move_slow != ob2->move_slow
253	|| ob1->move_slow_penalty != ob2->move_slow_penalty)	230	|| ob1->move_slow_penalty != ob2->move_slow_penalty
		231	|| memcmp (&ob1->resist, &ob2->resist, sizeof (ob1->resist))
		232	|| memcmp (&ob1->stats , &ob2->stats , sizeof (ob1->stats)))
		233	return 0;
		234
		235	if ((ob1->flag ^ ob2->flag)
		236	.reset (FLAG_INV_LOCKED)
		237	.reset (FLAG_CLIENT_SENT)
		238	.reset (FLAG_REMOVED)
		239	.any ())
254	return 0;	240	return 0;
255		241
256	/* This is really a spellbook check - we should in general	242	/* This is really a spellbook check - we should in general
257	* not merge objects with real inventories, as splitting them	243	* not merge objects with real inventories, as splitting them
258	* is hard.	244	* is hard.
…		…
297		283
298	if (ob1->key_values || ob2->key_values)	284	if (ob1->key_values || ob2->key_values)
299	{	285	{
300	/* At least one of these has key_values. */	286	/* At least one of these has key_values. */
301	if ((!ob1->key_values) != (!ob2->key_values))	287	if ((!ob1->key_values) != (!ob2->key_values))
302	/* One has fields, but the other one doesn't. */	288	return 0; /* One has fields, but the other one doesn't. */
303	return 0;	289
304	else if (!compare_ob_value_lists (ob1, ob2))	290	if (!compare_ob_value_lists (ob1, ob2))
305	return 0;	291	return 0;
306	}	292	}
307		293
308	if (ob1->self || ob2->self)	294	if (ob1->self || ob2->self)
309	{	295	{
…		…
315	int k1 = ob1->self ? HvTOTALKEYS (ob1->self) : 0;	301	int k1 = ob1->self ? HvTOTALKEYS (ob1->self) : 0;
316	int k2 = ob2->self ? HvTOTALKEYS (ob2->self) : 0;	302	int k2 = ob2->self ? HvTOTALKEYS (ob2->self) : 0;
317		303
318	if (k1 != k2)	304	if (k1 != k2)
319	return 0;	305	return 0;
		306
320	else if (k1 == 0)	307	if (k1 == 0)
321	return 1;	308	return 1;
		309
322	else if (!cfperl_can_merge (ob1, ob2))	310	if (!cfperl_can_merge (ob1, ob2))
323	return 0;	311	return 0;
324	}	312	}
325	}	313	}
326		314
327	/* Everything passes, must be OK. */	315	/* Everything passes, must be OK. */
328	return 1;	316	return 1;
329	}	317	}
330		318
		319	// find player who can see this object
		320	object *
		321	object::visible_to () const
		322	{
		323	if (client_visible () && !flag [FLAG_REMOVED])
		324	{
		325	// see if we are in a container of sorts
		326	if (env)
		327	{
		328	// the player inventory itself is always visible
		329	if (env->type == PLAYER)
		330	return env;
		331
		332	// else a player could have our env open
		333	object *envest = env->outer_env ();
		334
		335	// the player itself is always on a map, so we will find him here
		336	// even if our inv is in a player.
		337	if (envest->is_on_map ())
		338	if (object *pl = envest->ms ().player ())
		339	if (pl->container == env)
		340	return pl;
		341	}
		342	else
		343	{
		344	// maybe there is a player standing on the same mapspace
		345	// this will catch the case where "this" is a player
		346	if (object *pl = ms ().player ())
		347	if ((!pl->container && pl->contr->ns && !pl->contr->ns->update_look)
		348	|| this == pl->container)
		349	return pl;
		350	}
		351	}
		352
		353	return 0;
		354	}
		355
		356	// adjust weight per container type ("of holding")
		357	static sint32
		358	weight_adjust_for (object *op, sint32 weight)
		359	{
		360	return op->type == CONTAINER
		361	? lerp (weight, 0, 100, 0, 100 - op->stats.Str)
		362	: weight;
		363	}
		364
331	/*	365	/*
		366	* adjust_weight(object, weight) adds the specified weight to an object,
		367	* and also updates how much the environment(s) is/are carrying.
		368	*/
		369	static void
		370	adjust_weight (object *op, sint32 weight)
		371	{
		372	while (op)
		373	{
		374	// adjust by actual difference to account for rounding errors
		375	// i.e. (w2 - w1) / f != w2 / f - w1 / f and the latter is correct
		376	weight = weight_adjust_for (op, op->carrying)
		377	- weight_adjust_for (op, op->carrying - weight);
		378
		379	if (!weight)
		380	return;
		381
		382	op->carrying += weight;
		383
		384	if (object *pl = op->visible_to ())
		385	if (pl != op) // player is handled lazily
		386	esrv_update_item (UPD_WEIGHT, pl, op);
		387
		388	op = op->env;
		389	}
		390	}
		391
		392	/*
332	* sum_weight() is a recursive function which calculates the weight	393	* this is a recursive function which calculates the weight
333	* an object is carrying. It goes through in figures out how much	394	* an object is carrying. It goes through op and figures out how much
334	* containers are carrying, and sums it up.	395	* containers are carrying, and sums it up.
335	*/	396	*/
336	long	397	void
337	sum_weight (object *op)	398	object::update_weight ()
338	{	399	{
339	long sum;	400	sint32 sum = 0;
340	object *inv;
341		401
342	for (sum = 0, inv = op->inv; inv; inv = inv->below)	402	for (object *op = inv; op; op = op->below)
343	{	403	{
344	if (inv->inv)	404	if (op->inv)
345	sum_weight (inv);	405	op->update_weight ();
346		406
347	sum += inv->carrying + inv->weight * (inv->nrof ? inv->nrof : 1);	407	sum += op->total_weight ();
		408	}
		409
		410	sum = weight_adjust_for (this, sum);
		411
		412	if (sum != carrying)
348	}	413	{
349
350	if (op->type == CONTAINER && op->stats.Str)
351	sum = (sum * (100 - op->stats.Str)) / 100;
352
353	if (op->carrying != sum)
354	op->carrying = sum;	414	carrying = sum;
355		415
356	return sum;	416	if (object *pl = visible_to ())
		417	if (pl != this) // player is handled lazily
		418	esrv_update_item (UPD_WEIGHT, pl, this);
		419	}
357	}	420	}
358		421
359	/**	422	/*
360	* Return the outermost environment object for a given object.	423	* Used by: Server DM commands: dumpbelow, dump. Some error messages.
361	*/
362
363	object *
364	object_get_env_recursive (object *op)
365	{
366	while (op->env != NULL)
367	op = op->env;
368	return op;
369	}
370
371	/*
372	* Used by: Crossedit: dump. Server DM commands: dumpbelow, dump.
373	* Some error messages.
374	* The result of the dump is stored in the static global errmsg array.
375	*/	424	*/
376	char *	425	char *
377	dump_object (object *op)	426	dump_object (object *op)
378	{	427	{
379	if (!op)	428	if (!op)
…		…
393	get_nearest_part (object *op, const object *pl)	442	get_nearest_part (object *op, const object *pl)
394	{	443	{
395	object *tmp, *closest;	444	object *tmp, *closest;
396	int last_dist, i;	445	int last_dist, i;
397		446
398	if (op->more == NULL)	447	if (!op->more)
399	return op;	448	return op;
		449
400	for (last_dist = distance (op, pl), closest = op, tmp = op->more; tmp != NULL; tmp = tmp->more)	450	for (last_dist = distance (op, pl), closest = op, tmp = op->more;
		451	tmp;
		452	tmp = tmp->more)
401	if ((i = distance (tmp, pl)) < last_dist)	453	if ((i = distance (tmp, pl)) < last_dist)
402	closest = tmp, last_dist = i;	454	closest = tmp, last_dist = i;
		455
403	return closest;	456	return closest;
404	}	457	}
405		458
406	/*	459	/*
407	* Returns the object which has the count-variable equal to the argument.	460	* Returns the object which has the count-variable equal to the argument.
		461	* VERRRY slow.
408	*/	462	*/
409	object *	463	object *
410	find_object (tag_t i)	464	find_object (tag_t i)
411	{	465	{
412	for_all_objects (op)	466	for_all_objects (op)
…		…
423	*/	477	*/
424	object *	478	object *
425	find_object_name (const char *str)	479	find_object_name (const char *str)
426	{	480	{
427	shstr_cmp str_ (str);	481	shstr_cmp str_ (str);
428	object *op;
429		482
		483	if (str_)
430	for_all_objects (op)	484	for_all_objects (op)
431	if (op->name == str_)	485	if (op->name == str_)
432	break;	486	return op;
433		487
434	return op;	488	return 0;
435	}	489	}
436		490
437	/*	491	/*
438	* Sets the owner and sets the skill and exp pointers to owner's current	492	* Sets the owner and sets the skill and exp pointers to owner's current
439	* skill and experience objects.	493	* skill and experience objects.
…		…
537	}	591	}
538		592
539	op->key_values = 0;	593	op->key_values = 0;
540	}	594	}
541		595
542	object &	596	/*
543	object::operator =(const object &src)	597	* copy_to first frees everything allocated by the dst object,
		598	* and then copies the contents of itself into the second
		599	* object, allocating what needs to be allocated. Basically, any
		600	* data that is malloc'd needs to be re-malloc/copied. Otherwise,
		601	* if the first object is freed, the pointers in the new object
		602	* will point at garbage.
		603	*/
		604	void
		605	object::copy_to (object *dst)
544	{	606	{
545	bool is_freed = flag [FLAG_FREED];	607	dst->remove ();
546	bool is_removed = flag [FLAG_REMOVED];
547
548	*(object_copy *)this = src;	608	*(object_copy *)dst = *this;
549
550	flag [FLAG_FREED] = is_freed;
551	flag [FLAG_REMOVED] = is_removed;	609	dst->flag [FLAG_REMOVED] = true;
552		610
553	/* Copy over key_values, if any. */	611	/* Copy over key_values, if any. */
554	if (src.key_values)	612	if (key_values)
555	{	613	{
556	key_value *tail = 0;	614	key_value *tail = 0;
557	key_values = 0;	615	dst->key_values = 0;
558		616
559	for (key_value *i = src.key_values; i; i = i->next)	617	for (key_value *i = key_values; i; i = i->next)
560	{	618	{
561	key_value *new_link = new key_value;	619	key_value *new_link = new key_value;
562		620
563	new_link->next = 0;	621	new_link->next = 0;
564	new_link->key = i->key;	622	new_link->key = i->key;
565	new_link->value = i->value;	623	new_link->value = i->value;
566		624
567	/* Try and be clever here, too. */	625	/* Try and be clever here, too. */
568	if (!key_values)	626	if (!dst->key_values)
569	{	627	{
570	key_values = new_link;	628	dst->key_values = new_link;
571	tail = new_link;	629	tail = new_link;
572	}	630	}
573	else	631	else
574	{	632	{
575	tail->next = new_link;	633	tail->next = new_link;
576	tail = new_link;	634	tail = new_link;
577	}	635	}
578	}	636	}
579	}	637	}
580	}
581
582	/*
583	* copy_to first frees everything allocated by the dst object,
584	* and then copies the contents of itself into the second
585	* object, allocating what needs to be allocated. Basically, any
586	* data that is malloc'd needs to be re-malloc/copied. Otherwise,
587	* if the first object is freed, the pointers in the new object
588	* will point at garbage.
589	*/
590	void
591	object::copy_to (object *dst)
592	{
593	*dst = *this;
594		638
595	if (speed < 0)	639	if (speed < 0)
596	dst->speed_left -= rndm ();	640	dst->speed_left -= rndm ();
597		641
598	dst->set_speed (dst->speed);	642	dst->set_speed (dst->speed);
…		…
620	object *	664	object *
621	object::clone ()	665	object::clone ()
622	{	666	{
623	object *neu = create ();	667	object *neu = create ();
624	copy_to (neu);	668	copy_to (neu);
		669	neu->map = map; // not copied by copy_to
625	return neu;	670	return neu;
626	}	671	}
627		672
628	/*	673	/*
629	* If an object with the IS_TURNABLE() flag needs to be turned due	674	* If an object with the IS_TURNABLE() flag needs to be turned due
…		…
680	* UP_OBJ_FACE: only the objects face has changed.	725	* UP_OBJ_FACE: only the objects face has changed.
681	*/	726	*/
682	void	727	void
683	update_object (object *op, int action)	728	update_object (object *op, int action)
684	{	729	{
685	if (op == NULL)	730	if (!op)
686	{	731	{
687	/* this should never happen */	732	/* this should never happen */
688	LOG (llevDebug, "update_object() called for NULL object.\n");	733	LOG (llevError | logBacktrace, "update_object() called for NULL object.\n");
689	return;	734	return;
690	}	735	}
691		736
692	if (op->env)	737	if (!op->is_on_map ())
693	{	738	{
694	/* Animation is currently handled by client, so nothing	739	/* Animation is currently handled by client, so nothing
695	* to do in this case.	740	* to do in this case.
696	*/	741	*/
697	return;	742	return;
698	}	743	}
699
700	/* If the map is saving, don't do anything as everything is
701	* going to get freed anyways.
702	*/
703	if (!op->map || op->map->in_memory == MAP_SAVING)
704	return;
705		744
706	/* make sure the object is within map boundaries */	745	/* make sure the object is within map boundaries */
707	if (op->x < 0 || op->x >= op->map->width || op->y < 0 || op->y >= op->map->height)	746	if (op->x < 0 || op->x >= op->map->width || op->y < 0 || op->y >= op->map->height)
708	{	747	{
709	LOG (llevError, "update_object() called for object out of map!\n");	748	LOG (llevError, "update_object() called for object out of map!\n");
…		…
728	|| (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC))	767	|| (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC))
729	|| (m.move_on | op->move_on ) != m.move_on	768	|| (m.move_on | op->move_on ) != m.move_on
730	|| (m.move_off | op->move_off ) != m.move_off	769	|| (m.move_off | op->move_off ) != m.move_off
731	|| (m.move_slow | op->move_slow) != m.move_slow	770	|| (m.move_slow | op->move_slow) != m.move_slow
732	/* This isn't perfect, but I don't expect a lot of objects to	771	/* This isn't perfect, but I don't expect a lot of objects to
733	* to have move_allow right now.	772	* have move_allow right now.
734	*/	773	*/
735	|| ((m.move_block | op->move_block) & ~op->move_allow) != m.move_block	774	|| ((m.move_block | op->move_block) & ~op->move_allow) != m.move_block
736	|| 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp)	775	|| 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp)
737	m.flags_ = 0;	776	m.flags_ = 0;
738	}	777	}
…		…
852	object::destroy_inv (bool drop_to_ground)	891	object::destroy_inv (bool drop_to_ground)
853	{	892	{
854	// need to check first, because the checks below might segfault	893	// need to check first, because the checks below might segfault
855	// as we might be on an invalid mapspace and crossfire code	894	// as we might be on an invalid mapspace and crossfire code
856	// is too buggy to ensure that the inventory is empty.	895	// is too buggy to ensure that the inventory is empty.
857	// corollary: if you create arrows etc. with stuff in tis inventory,	896	// corollary: if you create arrows etc. with stuff in its inventory,
858	// cf will crash below with off-map x and y	897	// cf will crash below with off-map x and y
859	if (!inv)	898	if (!inv)
860	return;	899	return;
861		900
862	/* Only if the space blocks everything do we not process -	901	/* Only if the space blocks everything do we not process -
…		…
864	* drop on that space.	903	* drop on that space.
865	*/	904	*/
866	if (!drop_to_ground	905	if (!drop_to_ground
867	|| !map	906	|| !map
868	|| map->in_memory != MAP_ACTIVE	907	|| map->in_memory != MAP_ACTIVE
869	|| map->nodrop	908	|| map->no_drop
870	|| ms ().move_block == MOVE_ALL)	909	|| ms ().move_block == MOVE_ALL)
871	{	910	{
872	while (inv)	911	while (inv)
873	{
874	inv->destroy_inv (drop_to_ground);
875	inv->destroy ();	912	inv->destroy (true);
876	}
877	}	913	}
878	else	914	else
879	{ /* Put objects in inventory onto this space */	915	{ /* Put objects in inventory onto this space */
880	while (inv)	916	while (inv)
881	{	917	{
…		…
885	|| op->flag [FLAG_NO_DROP]	921	|| op->flag [FLAG_NO_DROP]
886	|| op->type == RUNE	922	|| op->type == RUNE
887	|| op->type == TRAP	923	|| op->type == TRAP
888	|| op->flag [FLAG_IS_A_TEMPLATE]	924	|| op->flag [FLAG_IS_A_TEMPLATE]
889	|| op->flag [FLAG_DESTROY_ON_DEATH])	925	|| op->flag [FLAG_DESTROY_ON_DEATH])
890	op->destroy ();	926	op->destroy (true);
891	else	927	else
892	map->insert (op, x, y);	928	map->insert (op, x, y);
893	}	929	}
894	}	930	}
895	}	931	}
…		…
899	object *op = new object;	935	object *op = new object;
900	op->link ();	936	op->link ();
901	return op;	937	return op;
902	}	938	}
903		939
		940	static struct freed_map : maptile
		941	{
		942	freed_map ()
		943	{
		944	path = "<freed objects map>";
		945	name = "/internal/freed_objects_map";
		946	width = 3;
		947	height = 3;
		948	no_drop = 1;
		949	no_reset = 1;
		950
		951	alloc ();
		952	in_memory = MAP_ACTIVE;
		953	}
		954
		955	~freed_map ()
		956	{
		957	destroy ();
		958	}
		959	} freed_map; // freed objects are moved here to avoid crashes
		960
904	void	961	void
905	object::do_destroy ()	962	object::do_destroy ()
906	{	963	{
907	attachable::do_destroy ();
908
909	if (flag [FLAG_IS_LINKED])	964	if (flag [FLAG_IS_LINKED])
910	remove_button_link (this);	965	remove_button_link (this);
911		966
912	if (flag [FLAG_FRIENDLY])	967	if (flag [FLAG_FRIENDLY])
913	remove_friendly_object (this);	968	remove_friendly_object (this);
914		969
915	if (!flag [FLAG_REMOVED])
916	remove ();	970	remove ();
917		971
918	destroy_inv (true);	972	attachable::do_destroy ();
919		973
920	deactivate ();	974	deactivate ();
921	unlink ();	975	unlink ();
922		976
923	flag [FLAG_FREED] = 1;	977	flag [FLAG_FREED] = 1;
924		978
925	// hack to ensure that freed objects still have a valid map	979	// hack to ensure that freed objects still have a valid map
926	{
927	static maptile *freed_map; // freed objects are moved here to avoid crashes
928
929	if (!freed_map)
930	{
931	freed_map = new maptile;
932
933	freed_map->path = "<freed objects map>";
934	freed_map->name = "/internal/freed_objects_map";
935	freed_map->width = 3;
936	freed_map->height = 3;
937	freed_map->nodrop = 1;
938
939	freed_map->alloc ();
940	freed_map->in_memory = MAP_ACTIVE;
941	}
942
943	map = freed_map;	980	map = &freed_map;
944	x = 1;	981	x = 1;
945	y = 1;	982	y = 1;
946	}
947		983
948	if (more)	984	if (more)
949	{	985	{
950	more->destroy ();	986	more->destroy ();
951	more = 0;	987	more = 0;
…		…
964	object::destroy (bool destroy_inventory)	1000	object::destroy (bool destroy_inventory)
965	{	1001	{
966	if (destroyed ())	1002	if (destroyed ())
967	return;	1003	return;
968		1004
		1005	if (!is_head () && !head->destroyed ())
		1006	{
		1007	LOG (llevError | logBacktrace, "tried to destroy the tail of an object");
		1008	head->destroy (destroy_inventory);
		1009	return;
		1010	}
		1011
969	if (destroy_inventory)	1012	destroy_inv (!destroy_inventory);
970	destroy_inv (false);
971		1013
972	if (is_head ())	1014	if (is_head ())
973	if (sound_destroy)	1015	if (sound_destroy)
974	play_sound (sound_destroy);	1016	play_sound (sound_destroy);
975	else if (flag [FLAG_MONSTER])	1017	else if (flag [FLAG_MONSTER])
976	play_sound (sound_find ("monster_destroy")); // quick hack, too lazy to create a generic mechanism	1018	play_sound (sound_find ("monster_destroy")); // quick hack, too lazy to create a generic mechanism
977		1019
978	attachable::destroy ();	1020	attachable::destroy ();
979	}
980
981	/*
982	* sub_weight() recursively (outwards) subtracts a number from the
983	* weight of an object (and what is carried by it's environment(s)).
984	*/
985	void
986	sub_weight (object *op, signed long weight)
987	{
988	while (op != NULL)
989	{
990	if (op->type == CONTAINER)
991	weight = (signed long) (weight * (100 - op->stats.Str) / 100);
992
993	op->carrying -= weight;
994	op = op->env;
995	}
996	}	1021	}
997		1022
998	/* op->remove ():	1023	/* op->remove ():
999	* This function removes the object op from the linked list of objects	1024	* This function removes the object op from the linked list of objects
1000	* which it is currently tied to. When this function is done, the	1025	* which it is currently tied to. When this function is done, the
…		…
1006	object::do_remove ()	1031	object::do_remove ()
1007	{	1032	{
1008	object *tmp, *last = 0;	1033	object *tmp, *last = 0;
1009	object *otmp;	1034	object *otmp;
1010		1035
1011	if (QUERY_FLAG (this, FLAG_REMOVED))	1036	if (flag [FLAG_REMOVED])
1012	return;	1037	return;
1013		1038
1014	SET_FLAG (this, FLAG_REMOVED);
1015	INVOKE_OBJECT (REMOVE, this);	1039	INVOKE_OBJECT (REMOVE, this);
		1040
		1041	flag [FLAG_REMOVED] = true;
1016		1042
1017	if (more)	1043	if (more)
1018	more->remove ();	1044	more->remove ();
1019		1045
1020	/*	1046	/*
1021	* In this case, the object to be removed is in someones	1047	* In this case, the object to be removed is in someones
1022	* inventory.	1048	* inventory.
1023	*/	1049	*/
1024	if (env)	1050	if (env)
1025	{	1051	{
1026	if (nrof)	1052	flag [FLAG_REMOVED] = false; // hack around the issue of visible_to checking flag_removed
1027	sub_weight (env, weight * nrof);	1053	if (object *pl = visible_to ())
1028	else	1054	esrv_del_item (pl->contr, count);
1029	sub_weight (env, weight + carrying);	1055	flag [FLAG_REMOVED] = true; // hack around the issue of visible_to checking flag_removed
		1056
		1057	adjust_weight (env, -total_weight ());
		1058
		1059	/* we set up values so that it could be inserted into
		1060	* the map, but we don't actually do that - it is up
		1061	* to the caller to decide what we want to do.
		1062	*/
		1063	map = env->map;
		1064	x = env->x;
		1065	y = env->y;
		1066
		1067	// make sure cmov optimisation is applicable
		1068	*(above ? &above->below : &env->inv) = below;
		1069	*(below ? &below->above : &above ) = above; // &above is just a dummy
		1070
		1071	above = 0;
		1072	below = 0;
		1073	env = 0;
1030		1074
1031	/* NO_FIX_PLAYER is set when a great many changes are being	1075	/* NO_FIX_PLAYER is set when a great many changes are being
1032	* made to players inventory. If set, avoiding the call	1076	* made to players inventory. If set, avoiding the call
1033	* to save cpu time.	1077	* to save cpu time.
1034	*/	1078	*/
1035	if ((otmp = in_player ()) && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))	1079	if ((otmp = in_player ()) && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
1036	otmp->update_stats ();	1080	otmp->update_stats ();
1037
1038	if (above)
1039	above->below = below;
1040	else
1041	env->inv = below;
1042
1043	if (below)
1044	below->above = above;
1045
1046	/* we set up values so that it could be inserted into
1047	* the map, but we don't actually do that - it is up
1048	* to the caller to decide what we want to do.
1049	*/
1050	x = env->x, y = env->y;
1051	map = env->map;
1052	above = 0, below = 0;
1053	env = 0;
1054	}	1081	}
1055	else if (map)	1082	else if (map)
1056	{	1083	{
1057	if (type == PLAYER)
1058	{
1059	// leaving a spot always closes any open container on the ground
1060	if (container && !container->env)
1061	// this causes spurious floorbox updates, but it ensures
1062	// that the CLOSE event is being sent.
1063	close_container ();
1064
1065	--map->players;
1066	map->touch ();
1067	}
1068
1069	map->dirty = true;	1084	map->dirty = true;
1070	mapspace &ms = this->ms ();	1085	mapspace &ms = this->ms ();
1071		1086
		1087	if (object *pl = ms.player ())
		1088	{
		1089	if (type == PLAYER) // this == pl(!)
		1090	{
		1091	// leaving a spot always closes any open container on the ground
		1092	if (container && !container->env)
		1093	// this causes spurious floorbox updates, but it ensures
		1094	// that the CLOSE event is being sent.
		1095	close_container ();
		1096
		1097	--map->players;
		1098	map->touch ();
		1099	}
		1100	else if (pl->container == this)
		1101	{
		1102	// removing a container should close it
		1103	close_container ();
		1104	}
		1105
		1106	esrv_del_item (pl->contr, count);
		1107	}
		1108
1072	/* link the object above us */	1109	/* link the object above us */
1073	if (above)	1110	// re-link, make sure compiler can easily use cmove
1074	above->below = below;	1111	*(above ? &above->below : &ms.top) = below;
1075	else	1112	*(below ? &below->above : &ms.bot) = above;
1076	ms.top = below; /* we were top, set new top */
1077
1078	/* Relink the object below us, if there is one */
1079	if (below)
1080	below->above = above;
1081	else
1082	{
1083	/* Nothing below, which means we need to relink map object for this space
1084	* use translated coordinates in case some oddness with map tiling is
1085	* evident
1086	*/
1087	if (GET_MAP_OB (map, x, y) != this)
1088	LOG (llevError, "remove_ob: GET_MAP_OB does not return object to be removed even though it appears to be on the bottom? %s\n", debug_desc ());
1089
1090	ms.bot = above; /* goes on above it. */
1091	}
1092		1113
1093	above = 0;	1114	above = 0;
1094	below = 0;	1115	below = 0;
		1116
		1117	ms.flags_ = 0;
1095		1118
1096	if (map->in_memory == MAP_SAVING)	1119	if (map->in_memory == MAP_SAVING)
1097	return;	1120	return;
1098		1121
1099	int check_walk_off = !flag [FLAG_NO_APPLY];	1122	int check_walk_off = !flag [FLAG_NO_APPLY];
…		…
1105	* removed (most likely destroyed), update the player view	1128	* removed (most likely destroyed), update the player view
1106	* appropriately.	1129	* appropriately.
1107	*/	1130	*/
1108	pl->close_container ();	1131	pl->close_container ();
1109		1132
		1133	//TODO: the floorbox prev/next might need updating
		1134	//esrv_del_item (pl->contr, count);
		1135	//TODO: update floorbox to preserve ordering
		1136	if (pl->contr->ns)
1110	pl->contr->ns->floorbox_update ();	1137	pl->contr->ns->floorbox_update ();
1111	}	1138	}
1112		1139
1113	for (tmp = ms.bot; tmp; tmp = tmp->above)	1140	for (tmp = ms.bot; tmp; tmp = tmp->above)
1114	{	1141	{
1115	/* No point updating the players look faces if he is the object	1142	/* No point updating the players look faces if he is the object
…		…
1128	}	1155	}
1129		1156
1130	last = tmp;	1157	last = tmp;
1131	}	1158	}
1132		1159
1133	/* last == NULL if there are no objects on this space */
1134	//TODO: this makes little sense, why only update the topmost object?
1135	if (!last)
1136	map->at (x, y).flags_ = 0;
1137	else
1138	update_object (last, UP_OBJ_REMOVE);
1139
1140	if (flag [FLAG_BLOCKSVIEW] || glow_radius)	1160	if (flag [FLAG_BLOCKSVIEW] || glow_radius)
1141	update_all_los (map, x, y);	1161	update_all_los (map, x, y);
1142	}	1162	}
1143	}	1163	}
1144		1164
…		…
1159	if (!top)	1179	if (!top)
1160	for (top = op; top && top->above; top = top->above)	1180	for (top = op; top && top->above; top = top->above)
1161	;	1181	;
1162		1182
1163	for (; top; top = top->below)	1183	for (; top; top = top->below)
1164	{
1165	if (top == op)
1166	continue;
1167
1168	if (object::can_merge (op, top))	1184	if (object::can_merge (op, top))
1169	{	1185	{
1170	top->nrof += op->nrof;	1186	top->nrof += op->nrof;
1171		1187
1172	/* CLEAR_FLAG(top,FLAG_STARTEQUIP);*/	1188	if (object *pl = top->visible_to ())
1173	op->weight = 0; /* Don't want any adjustements now */	1189	esrv_update_item (UPD_NROF, pl, top);
		1190
		1191	op->weight = 0; // cancel the addition above
		1192	op->carrying = 0; // must be 0 already
		1193
1174	op->destroy ();	1194	op->destroy (1);
		1195
1175	return top;	1196	return top;
1176	}	1197	}
1177	}
1178		1198
1179	return 0;	1199	return 0;
1180	}	1200	}
1181		1201
1182	void	1202	void
…		…
1207	* job preparing multi-part monsters.	1227	* job preparing multi-part monsters.
1208	*/	1228	*/
1209	object *	1229	object *
1210	insert_ob_in_map_at (object *op, maptile *m, object *originator, int flag, int x, int y)	1230	insert_ob_in_map_at (object *op, maptile *m, object *originator, int flag, int x, int y)
1211	{	1231	{
		1232	op->remove ();
		1233
1212	for (object *tmp = op->head_ (); tmp; tmp = tmp->more)	1234	for (object *tmp = op->head_ (); tmp; tmp = tmp->more)
1213	{	1235	{
1214	tmp->x = x + tmp->arch->x;	1236	tmp->x = x + tmp->arch->x;
1215	tmp->y = y + tmp->arch->y;	1237	tmp->y = y + tmp->arch->y;
1216	}	1238	}
…		…
1239	* just 'op' otherwise	1261	* just 'op' otherwise
1240	*/	1262	*/
1241	object *	1263	object *
1242	insert_ob_in_map (object *op, maptile *m, object *originator, int flag)	1264	insert_ob_in_map (object *op, maptile *m, object *originator, int flag)
1243	{	1265	{
1244	assert (!op->flag [FLAG_FREED]);	1266	if (op->is_on_map ())
		1267	{
		1268	LOG (llevError, "insert_ob_in_map called for object already on map");
		1269	abort ();
		1270	}
1245		1271
1246	object *top, *floor = NULL;	1272	if (op->env)
1247		1273	{
		1274	LOG (llevError, "insert_ob_in_map called for object in an inventory (proceeding)");
1248	op->remove ();	1275	op->remove ();
		1276	}
		1277
		1278	if (op->face && !face_info (op->face))//D TODO: remove soon
		1279	{//D
		1280	LOG (llevError | logBacktrace, "op->face out of bounds: %s", op->debug_desc ());//D
		1281	op->face = 1;//D
		1282	}//D
1249		1283
1250	/* Ideally, the caller figures this out. However, it complicates a lot	1284	/* Ideally, the caller figures this out. However, it complicates a lot
1251	* of areas of callers (eg, anything that uses find_free_spot would now	1285	* of areas of callers (eg, anything that uses find_free_spot would now
1252	* need extra work	1286	* need extra work
1253	*/	1287	*/
1254	if (!xy_normalise (m, op->x, op->y))	1288	if (!xy_normalise (m, op->x, op->y))
1255	{	1289	{
1256	op->destroy ();	1290	op->head_ ()->destroy (1);// remove head_ once all tail object destroyers found
1257	return 0;	1291	return 0;
1258	}	1292	}
1259		1293
1260	if (object *more = op->more)	1294	if (object *more = op->more)
1261	if (!insert_ob_in_map (more, m, originator, flag))	1295	if (!insert_ob_in_map (more, m, originator, flag))
…		…
1270	*/	1304	*/
1271	if (op->nrof && !(flag & INS_NO_MERGE))	1305	if (op->nrof && !(flag & INS_NO_MERGE))
1272	for (object *tmp = ms.bot; tmp; tmp = tmp->above)	1306	for (object *tmp = ms.bot; tmp; tmp = tmp->above)
1273	if (object::can_merge (op, tmp))	1307	if (object::can_merge (op, tmp))
1274	{	1308	{
		1309	// TODO: we actually want to update tmp, not op,
		1310	// but some caller surely breaks when we return tmp
		1311	// from here :/
1275	op->nrof += tmp->nrof;	1312	op->nrof += tmp->nrof;
1276	tmp->destroy ();	1313	tmp->destroy (1);
1277	}	1314	}
1278		1315
1279	CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */	1316	CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */
1280	CLEAR_FLAG (op, FLAG_INV_LOCKED);	1317	CLEAR_FLAG (op, FLAG_INV_LOCKED);
1281		1318
…		…
1288	{	1325	{
1289	LOG (llevError, "insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n");	1326	LOG (llevError, "insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n");
1290	abort ();	1327	abort ();
1291	}	1328	}
1292		1329
		1330	if (!originator->is_on_map ())
		1331	LOG (llevDebug | logBacktrace, "insert_ob_in_map(%s) called with INS_BELOW_ORIGINATOR when originator '%s' not on map",
		1332	op->debug_desc (), originator->debug_desc ());
		1333
1293	op->above = originator;	1334	op->above = originator;
1294	op->below = originator->below;	1335	op->below = originator->below;
1295
1296	if (op->below)
1297	op->below->above = op;
1298	else
1299	ms.bot = op;
1300
1301	/* since *below* originator, no need to update top */
1302	originator->below = op;	1336	originator->below = op;
		1337
		1338	*(op->below ? &op->below->above : &ms.bot) = op;
1303	}	1339	}
1304	else	1340	else
1305	{	1341	{
1306	top = ms.bot;	1342	object *floor = 0;
		1343	object *top = ms.top;
1307		1344
1308	/* If there are other objects, then */	1345	/* If there are other objects, then */
1309	if (top)	1346	if (top)
1310	{	1347	{
1311	object *last = 0;
1312
1313	/*	1348	/*
1314	* If there are multiple objects on this space, we do some trickier handling.	1349	* If there are multiple objects on this space, we do some trickier handling.
1315	* We've already dealt with merging if appropriate.	1350	* We've already dealt with merging if appropriate.
1316	* Generally, we want to put the new object on top. But if	1351	* Generally, we want to put the new object on top. But if
1317	* flag contains INS_ABOVE_FLOOR_ONLY, once we find the last	1352	* flag contains INS_ABOVE_FLOOR_ONLY, once we find the last
…		…
1320	* once we get to them. This reduces the need to traverse over all of	1355	* once we get to them. This reduces the need to traverse over all of
1321	* them when adding another one - this saves quite a bit of cpu time	1356	* them when adding another one - this saves quite a bit of cpu time
1322	* when lots of spells are cast in one area. Currently, it is presumed	1357	* when lots of spells are cast in one area. Currently, it is presumed
1323	* that flying non pickable objects are spell objects.	1358	* that flying non pickable objects are spell objects.
1324	*/	1359	*/
1325	for (top = ms.bot; top; top = top->above)	1360	for (object *tmp = ms.bot; tmp; tmp = tmp->above)
1326	{	1361	{
1327	if (QUERY_FLAG (top, FLAG_IS_FLOOR) || QUERY_FLAG (top, FLAG_OVERLAY_FLOOR))	1362	if (QUERY_FLAG (tmp, FLAG_IS_FLOOR) || QUERY_FLAG (tmp, FLAG_OVERLAY_FLOOR))
1328	floor = top;	1363	floor = tmp;
1329		1364
1330	if (QUERY_FLAG (top, FLAG_NO_PICK) && (top->move_type & (MOVE_FLY_LOW | MOVE_FLY_HIGH)) && !QUERY_FLAG (top, FLAG_IS_FLOOR))	1365	if (QUERY_FLAG (tmp, FLAG_NO_PICK) && (tmp->move_type & (MOVE_FLY_LOW | MOVE_FLY_HIGH)) && !QUERY_FLAG (tmp, FLAG_IS_FLOOR))
1331	{	1366	{
1332	/* We insert above top, so we want this object below this */	1367	/* We insert above top, so we want this object below this */
1333	top = top->below;	1368	top = tmp->below;
1334	break;	1369	break;
1335	}	1370	}
1336		1371
1337	last = top;	1372	top = tmp;
1338	}	1373	}
1339
1340	/* Don't want top to be NULL, so set it to the last valid object */
1341	top = last;
1342		1374
1343	/* We let update_position deal with figuring out what the space	1375	/* We let update_position deal with figuring out what the space
1344	* looks like instead of lots of conditions here.	1376	* looks like instead of lots of conditions here.
1345	* makes things faster, and effectively the same result.	1377	* makes things faster, and effectively the same result.
1346	*/	1378	*/
…		…
1353	*/	1385	*/
1354	if (!(flag & INS_ON_TOP)	1386	if (!(flag & INS_ON_TOP)
1355	&& ms.flags () & P_BLOCKSVIEW	1387	&& ms.flags () & P_BLOCKSVIEW
1356	&& (op->face && !faces [op->face].visibility))	1388	&& (op->face && !faces [op->face].visibility))
1357	{	1389	{
		1390	object *last;
		1391
1358	for (last = top; last != floor; last = last->below)	1392	for (last = top; last != floor; last = last->below)
1359	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))	1393	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))
1360	break;	1394	break;
1361		1395
1362	/* Check to see if we found the object that blocks view,	1396	/* Check to see if we found the object that blocks view,
…		…
1370	} /* If objects on this space */	1404	} /* If objects on this space */
1371		1405
1372	if (flag & INS_ABOVE_FLOOR_ONLY)	1406	if (flag & INS_ABOVE_FLOOR_ONLY)
1373	top = floor;	1407	top = floor;
1374		1408
1375	/* Top is the object that our object (op) is going to get inserted above.	1409	// insert object above top, or bottom-most if top = 0
1376	*/
1377
1378	/* First object on this space */
1379	if (!top)	1410	if (!top)
1380	{	1411	{
		1412	op->below = 0;
1381	op->above = ms.bot;	1413	op->above = ms.bot;
1382
1383	if (op->above)
1384	op->above->below = op;
1385
1386	op->below = 0;
1387	ms.bot = op;	1414	ms.bot = op;
		1415
		1416	*(op->above ? &op->above->below : &ms.top) = op;
1388	}	1417	}
1389	else	1418	else
1390	{ /* get inserted into the stack above top */	1419	{
1391	op->above = top->above;	1420	op->above = top->above;
1392
1393	if (op->above)
1394	op->above->below = op;	1421	top->above = op;
1395		1422
1396	op->below = top;	1423	op->below = top;
1397	top->above = op;	1424	*(op->above ? &op->above->below : &ms.top) = op;
1398	}	1425	}
1399		1426	}
1400	if (!op->above)
1401	ms.top = op;
1402	} /* else not INS_BELOW_ORIGINATOR */
1403		1427
1404	if (op->type == PLAYER)	1428	if (op->type == PLAYER)
1405	{	1429	{
1406	op->contr->do_los = 1;	1430	op->contr->do_los = 1;
1407	++op->map->players;	1431	++op->map->players;
…		…
1409	}	1433	}
1410		1434
1411	op->map->dirty = true;	1435	op->map->dirty = true;
1412		1436
1413	if (object *pl = ms.player ())	1437	if (object *pl = ms.player ())
		1438	//TODO: the floorbox prev/next might need updating
		1439	//esrv_send_item (pl, op);
		1440	//TODO: update floorbox to preserve ordering
		1441	if (pl->contr->ns)
1414	pl->contr->ns->floorbox_update ();	1442	pl->contr->ns->floorbox_update ();
1415		1443
1416	/* If this object glows, it may affect lighting conditions that are	1444	/* If this object glows, it may affect lighting conditions that are
1417	* visible to others on this map. But update_all_los is really	1445	* visible to others on this map. But update_all_los is really
1418	* an inefficient way to do this, as it means los for all players	1446	* an inefficient way to do this, as it means los for all players
1419	* on the map will get recalculated. The players could very well	1447	* on the map will get recalculated. The players could very well
…		…
1461	* op is the object to insert it under: supplies x and the map.	1489	* op is the object to insert it under: supplies x and the map.
1462	*/	1490	*/
1463	void	1491	void
1464	replace_insert_ob_in_map (const char *arch_string, object *op)	1492	replace_insert_ob_in_map (const char *arch_string, object *op)
1465	{	1493	{
1466	object *tmp, *tmp1;
1467
1468	/* first search for itself and remove any old instances */	1494	/* first search for itself and remove any old instances */
1469		1495
1470	for (tmp = op->ms ().bot; tmp; tmp = tmp->above)	1496	for (object *tmp = op->ms ().bot; tmp; tmp = tmp->above)
1471	if (!strcmp (tmp->arch->archname, arch_string)) /* same archetype */	1497	if (!strcmp (tmp->arch->archname, arch_string)) /* same archetype */
1472	tmp->destroy ();	1498	tmp->destroy (1);
1473		1499
1474	tmp1 = arch_to_object (archetype::find (arch_string));	1500	object *tmp = arch_to_object (archetype::find (arch_string));
1475		1501
1476	tmp1->x = op->x;	1502	tmp->x = op->x;
1477	tmp1->y = op->y;	1503	tmp->y = op->y;
		1504
1478	insert_ob_in_map (tmp1, op->map, op, 0);	1505	insert_ob_in_map (tmp, op->map, op, 0);
1479	}	1506	}
1480		1507
1481	object *	1508	object *
1482	object::insert_at (object *where, object *originator, int flags)	1509	object::insert_at (object *where, object *originator, int flags)
1483	{	1510	{
…		…
1486	else	1513	else
1487	return where->map->insert (this, where->x, where->y, originator, flags);	1514	return where->map->insert (this, where->x, where->y, originator, flags);
1488	}	1515	}
1489		1516
1490	/*	1517	/*
1491	* get_split_ob(ob,nr) splits up ob into two parts. The part which
1492	* is returned contains nr objects, and the remaining parts contains
1493	* the rest (or is removed and freed if that number is 0).
1494	* On failure, NULL is returned, and the reason put into the
1495	* global static errmsg array.
1496	*/
1497	object *
1498	get_split_ob (object *orig_ob, uint32 nr)
1499	{
1500	object *newob;
1501	int is_removed = (QUERY_FLAG (orig_ob, FLAG_REMOVED) != 0);
1502
1503	if (orig_ob->nrof < nr)
1504	{
1505	sprintf (errmsg, "There are only %d %ss.", orig_ob->nrof ? orig_ob->nrof : 1, &orig_ob->name);
1506	return NULL;
1507	}
1508
1509	newob = object_create_clone (orig_ob);
1510
1511	if ((orig_ob->nrof -= nr) < 1)
1512	orig_ob->destroy (1);
1513	else if (!is_removed)
1514	{
1515	if (orig_ob->env != NULL)
1516	sub_weight (orig_ob->env, orig_ob->weight * nr);
1517	if (orig_ob->env == NULL && orig_ob->map->in_memory != MAP_ACTIVE)
1518	{
1519	strcpy (errmsg, "Tried to split object whose map is not in memory.");
1520	LOG (llevDebug, "Error, Tried to split object whose map is not in memory.\n");
1521	return NULL;
1522	}
1523	}
1524
1525	newob->nrof = nr;
1526
1527	return newob;
1528	}
1529
1530	/*
1531	* decrease_ob_nr(object, number) decreases a specified number from	1518	* decrease(object, number) decreases a specified number from
1532	* the amount of an object. If the amount reaches 0, the object	1519	* the amount of an object. If the amount reaches 0, the object
1533	* is subsequently removed and freed.	1520	* is subsequently removed and freed.
1534	*	1521	*
1535	* Return value: 'op' if something is left, NULL if the amount reached 0	1522	* Return value: 'op' if something is left, NULL if the amount reached 0
1536	*/	1523	*/
		1524	bool
		1525	object::decrease (sint32 nr)
		1526	{
		1527	if (!nr)
		1528	return true;
		1529
		1530	nr = min (nr, nrof);
		1531
		1532	if (nrof > nr)
		1533	{
		1534	nrof -= nr;
		1535	adjust_weight (env, -weight * max (1, nr)); // carrying == 0
		1536
		1537	if (object *pl = visible_to ())
		1538	esrv_update_item (UPD_NROF, pl, this);
		1539
		1540	return true;
		1541	}
		1542	else
		1543	{
		1544	destroy ();
		1545	return false;
		1546	}
		1547	}
		1548
		1549	/*
		1550	* split(ob,nr) splits up ob into two parts. The part which
		1551	* is returned contains nr objects, and the remaining parts contains
		1552	* the rest (or is removed and returned if that number is 0).
		1553	* On failure, NULL is returned.
		1554	*/
1537	object *	1555	object *
1538	decrease_ob_nr (object *op, uint32 i)	1556	object::split (sint32 nr)
1539	{	1557	{
1540	object *tmp;	1558	int have = number_of ();
1541		1559
1542	if (i == 0) /* objects with op->nrof require this check */	1560	if (have < nr)
1543	return op;	1561	return 0;
1544		1562	else if (have == nr)
1545	if (i > op->nrof)
1546	i = op->nrof;
1547
1548	if (QUERY_FLAG (op, FLAG_REMOVED))
1549	op->nrof -= i;
1550	else if (op->env)
1551	{	1563	{
1552	/* is this object in the players inventory, or sub container
1553	* therein?
1554	*/
1555	tmp = op->in_player ();
1556	/* nope. Is this a container the player has opened?
1557	* If so, set tmp to that player.
1558	* IMO, searching through all the players will mostly
1559	* likely be quicker than following op->env to the map,
1560	* and then searching the map for a player.
1561	*/
1562	if (!tmp)
1563	for_all_players (pl)
1564	if (pl->ob->container == op->env)
1565	{
1566	tmp = pl->ob;
1567	break;
1568	}
1569
1570	if (i < op->nrof)
1571	{
1572	sub_weight (op->env, op->weight * i);
1573	op->nrof -= i;
1574	if (tmp)
1575	esrv_send_item (tmp, op);
1576	}
1577	else
1578	{
1579	op->remove ();	1564	remove ();
1580	op->nrof = 0;	1565	return this;
1581	if (tmp)
1582	esrv_del_item (tmp->contr, op->count);
1583	}
1584	}	1566	}
1585	else	1567	else
1586	{	1568	{
1587	object *above = op->above;	1569	decrease (nr);
1588		1570
1589	if (i < op->nrof)	1571	object *op = deep_clone ();
1590	op->nrof -= i;	1572	op->nrof = nr;
1591	else
1592	{
1593	op->remove ();
1594	op->nrof = 0;
1595	}
1596
1597	/* Since we just removed op, op->above is null */
1598	for (tmp = above; tmp; tmp = tmp->above)
1599	if (tmp->type == PLAYER)
1600	{
1601	if (op->nrof)
1602	esrv_send_item (tmp, op);
1603	else
1604	esrv_del_item (tmp->contr, op->count);
1605	}
1606	}
1607
1608	if (op->nrof)
1609	return op;	1573	return op;
1610	else
1611	{
1612	op->destroy ();
1613	return 0;
1614	}
1615	}
1616
1617	/*
1618	* add_weight(object, weight) adds the specified weight to an object,
1619	* and also updates how much the environment(s) is/are carrying.
1620	*/
1621	void
1622	add_weight (object *op, signed long weight)
1623	{
1624	while (op != NULL)
1625	{
1626	if (op->type == CONTAINER)
1627	weight = (signed long) (weight * (100 - op->stats.Str) / 100);
1628
1629	op->carrying += weight;
1630	op = op->env;
1631	}	1574	}
1632	}	1575	}
1633		1576
1634	object *	1577	object *
1635	insert_ob_in_ob (object *op, object *where)	1578	insert_ob_in_ob (object *op, object *where)
…		…
1660	* be != op, if items are merged. -Tero	1603	* be != op, if items are merged. -Tero
1661	*/	1604	*/
1662	object *	1605	object *
1663	object::insert (object *op)	1606	object::insert (object *op)
1664	{	1607	{
1665	if (!QUERY_FLAG (op, FLAG_REMOVED))
1666	op->remove ();
1667
1668	if (op->more)	1608	if (op->more)
1669	{	1609	{
1670	LOG (llevError, "Tried to insert multipart object %s (%d)\n", &op->name, op->count);	1610	LOG (llevError, "Tried to insert multipart object %s (%d)\n", &op->name, op->count);
1671	return op;	1611	return op;
1672	}	1612	}
1673		1613
1674	CLEAR_FLAG (op, FLAG_OBJ_ORIGINAL);	1614	op->remove ();
1675	CLEAR_FLAG (op, FLAG_REMOVED);	1615
		1616	op->flag [FLAG_OBJ_ORIGINAL] = 0;
1676		1617
1677	if (op->nrof)	1618	if (op->nrof)
1678	{
1679	for (object *tmp = inv; tmp; tmp = tmp->below)	1619	for (object *tmp = inv; tmp; tmp = tmp->below)
1680	if (object::can_merge (tmp, op))	1620	if (object::can_merge (tmp, op))
1681	{	1621	{
1682	/* return the original object and remove inserted object	1622	/* return the original object and remove inserted object
1683	(client needs the original object) */	1623	(client needs the original object) */
1684	tmp->nrof += op->nrof;	1624	tmp->nrof += op->nrof;
1685	/* Weight handling gets pretty funky. Since we are adding to	1625
1686	* tmp->nrof, we need to increase the weight.	1626	if (object *pl = tmp->visible_to ())
1687	*/	1627	esrv_update_item (UPD_NROF, pl, tmp);
		1628
1688	add_weight (this, op->weight * op->nrof);	1629	adjust_weight (this, op->total_weight ());
1689	SET_FLAG (op, FLAG_REMOVED);	1630
1690	op->destroy (); /* free the inserted object */	1631	op->destroy (1);
1691	op = tmp;	1632	op = tmp;
1692	op->remove (); /* and fix old object's links */	1633	goto inserted;
1693	CLEAR_FLAG (op, FLAG_REMOVED);
1694	break;
1695	}	1634	}
1696		1635
1697	/* I assume combined objects have no inventory
1698	* We add the weight - this object could have just been removed
1699	* (if it was possible to merge). calling remove_ob will subtract
1700	* the weight, so we need to add it in again, since we actually do
1701	* the linking below
1702	*/
1703	add_weight (this, op->weight * op->nrof);
1704	}
1705	else
1706	add_weight (this, (op->weight + op->carrying));
1707
1708	if (object *otmp = this->in_player ())
1709	if (otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
1710	otmp->update_stats ();
1711
1712	op->owner = 0; // its his/hers now. period.	1636	op->owner = 0; // it's his/hers now. period.
1713	op->map = 0;	1637	op->map = 0;
		1638	op->x = 0;
		1639	op->y = 0;
		1640
		1641	op->above = 0;
		1642	op->below = inv;
1714	op->env = this;	1643	op->env = this;
1715	op->above = 0;
1716	op->below = 0;
1717	op->x = op->y = 0;
1718		1644
		1645	if (inv)
		1646	inv->above = op;
		1647
		1648	inv = op;
		1649
		1650	op->flag [FLAG_REMOVED] = 0;
		1651
		1652	if (object *pl = op->visible_to ())
		1653	esrv_send_item (pl, op);
		1654
		1655	adjust_weight (this, op->total_weight ());
		1656
		1657	inserted:
1719	/* reset the light list and los of the players on the map */	1658	/* reset the light list and los of the players on the map */
1720	if (op->glow_radius && map)	1659	if (op->glow_radius && map && map->darkness)
1721	{
1722	#ifdef DEBUG_LIGHTS
1723	LOG (llevDebug, " insert_ob_in_ob(): got %s to insert in map/op\n", op->name);
1724	#endif /* DEBUG_LIGHTS */
1725	if (map->darkness)
1726	update_all_los (map, x, y);	1660	update_all_los (map, x, y);
1727	}
1728		1661
1729	/* Client has no idea of ordering so lets not bother ordering it here.	1662	// if this is a player's inventory, update stats
1730	* It sure simplifies this function...	1663	if (type == PLAYER && !flag [FLAG_NO_FIX_PLAYER])
1731	*/	1664	update_stats ();
1732	if (!inv)
1733	inv = op;
1734	else
1735	{
1736	op->below = inv;
1737	op->below->above = op;
1738	inv = op;
1739	}
1740		1665
1741	INVOKE_OBJECT (INSERT, this);	1666	INVOKE_OBJECT (INSERT, this);
1742		1667
1743	return op;	1668	return op;
1744	}	1669	}
…		…
1872	LOG (llevError, "Present_arch called outside map.\n");	1797	LOG (llevError, "Present_arch called outside map.\n");
1873	return NULL;	1798	return NULL;
1874	}	1799	}
1875		1800
1876	for (object *tmp = m->at (x, y).bot; tmp; tmp = tmp->above)	1801	for (object *tmp = m->at (x, y).bot; tmp; tmp = tmp->above)
1877	if (tmp->arch == at)	1802	if (tmp->arch->archname == at->archname)
1878	return tmp;	1803	return tmp;
1879		1804
1880	return NULL;	1805	return NULL;
1881	}	1806	}
1882		1807
…		…
1946	* The first matching object is returned, or NULL if none.	1871	* The first matching object is returned, or NULL if none.
1947	*/	1872	*/
1948	object *	1873	object *
1949	present_arch_in_ob (const archetype *at, const object *op)	1874	present_arch_in_ob (const archetype *at, const object *op)
1950	{	1875	{
1951	for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below)	1876	for (object *tmp = op->inv; tmp; tmp = tmp->below)
1952	if (tmp->arch == at)	1877	if (tmp->arch->archname == at->archname)
1953	return tmp;	1878	return tmp;
1954		1879
1955	return NULL;	1880	return NULL;
1956	}	1881	}
1957		1882
…		…
2366		2291
2367	/*	2292	/*
2368	* create clone from object to another	2293	* create clone from object to another
2369	*/	2294	*/
2370	object *	2295	object *
2371	object_create_clone (object *asrc)	2296	object::deep_clone ()
2372	{	2297	{
2373	object *dst = 0, *tmp, *src, *prev, *item;	2298	assert (("deep_clone called on non-head object", is_head ()));
2374		2299
2375	if (!asrc)	2300	object *dst = clone ();
2376	return 0;
2377		2301
2378	src = asrc->head_ ();	2302	object *prev = dst;
2379
2380	prev = 0;
2381	for (object *part = src; part; part = part->more)	2303	for (object *part = this->more; part; part = part->more)
2382	{	2304	{
2383	tmp = part->clone ();	2305	object *tmp = part->clone ();
2384	tmp->x -= src->x;
2385	tmp->y -= src->y;
2386
2387	if (!part->head)
2388	{
2389	dst = tmp;
2390	tmp->head = 0;
2391	}
2392	else
2393	tmp->head = dst;	2306	tmp->head = dst;
2394
2395	tmp->more = 0;
2396
2397	if (prev)
2398	prev->more = tmp;	2307	prev->more = tmp;
2399
2400	prev = tmp;	2308	prev = tmp;
2401	}	2309	}
2402		2310
2403	for (item = src->inv; item; item = item->below)	2311	for (object *item = inv; item; item = item->below)
2404	insert_ob_in_ob (object_create_clone (item), dst);	2312	insert_ob_in_ob (item->deep_clone (), dst);
2405		2313
2406	return dst;	2314	return dst;
2407	}	2315	}
2408		2316
2409	/* This returns the first object in who's inventory that	2317	/* This returns the first object in who's inventory that
…		…
2418	return tmp;	2326	return tmp;
2419		2327
2420	return 0;	2328	return 0;
2421	}	2329	}
2422		2330
2423	/* If ob has a field named key, return the link from the list,	2331	const shstr &
2424	* otherwise return NULL.	2332	object::kv_get (const shstr &key) const
2425	*
2426	* key must be a passed in shared string - otherwise, this won't
2427	* do the desired thing.
2428	*/
2429	key_value *
2430	get_ob_key_link (const object *ob, const char *key)
2431	{	2333	{
2432	for (key_value *link = ob->key_values; link; link = link->next)	2334	for (key_value *kv = key_values; kv; kv = kv->next)
2433	if (link->key == key)	2335	if (kv->key == key)
2434	return link;
2435
2436	return 0;
2437	}
2438
2439	/*
2440	* Returns the value of op has an extra_field for key, or NULL.
2441	*
2442	* The argument doesn't need to be a shared string.
2443	*
2444	* The returned string is shared.
2445	*/
2446	const char *
2447	get_ob_key_value (const object *op, const char *const key)
2448	{
2449	key_value *link;
2450	shstr_cmp canonical_key (key);
2451
2452	if (!canonical_key)
2453	{
2454	/* 1. There being a field named key on any object
2455	* implies there'd be a shared string to find.
2456	* 2. Since there isn't, no object has this field.
2457	* 3. Therefore, *this* object doesn't have this field.
2458	*/
2459	return 0;
2460	}
2461
2462	/* This is copied from get_ob_key_link() above -
2463	* only 4 lines, and saves the function call overhead.
2464	*/
2465	for (link = op->key_values; link; link = link->next)
2466	if (link->key == canonical_key)
2467	return link->value;	2336	return kv->value;
2468		2337
2469	return 0;	2338	return shstr_null;
2470	}	2339	}
2471		2340
2472	/*	2341	void
2473	* Updates the canonical_key in op to value.	2342	object::kv_set (const shstr &key, const shstr &value)
2474	*
2475	* canonical_key is a shared string (value doesn't have to be).
2476	*
2477	* Unless add_key is TRUE, it won't add fields, only change the value of existing
2478	* keys.
2479	*
2480	* Returns TRUE on success.
2481	*/
2482	int
2483	set_ob_key_value_s (object *op, const shstr & canonical_key, const char *value, int add_key)
2484	{	2343	{
2485	key_value *field = NULL, *last = NULL;	2344	for (key_value *kv = key_values; kv; kv = kv->next)
2486		2345	if (kv->key == key)
2487	for (field = op->key_values; field != NULL; field = field->next)
2488	{
2489	if (field->key != canonical_key)
2490	{	2346	{
2491	last = field;	2347	kv->value = value;
2492	continue;	2348	return;
2493	}	2349	}
2494		2350
2495	if (value)	2351	key_value *kv = new key_value;
2496	field->value = value;	2352
2497	else	2353	kv->next = key_values;
		2354	kv->key = key;
		2355	kv->value = value;
		2356
		2357	key_values = kv;
		2358	}
		2359
		2360	void
		2361	object::kv_del (const shstr &key)
		2362	{
		2363	for (key_value **kvp = &key_values; *kvp; kvp = &(*kvp)->next)
		2364	if ((*kvp)->key == key)
2498	{	2365	{
2499	/* Basically, if the archetype has this key set,	2366	key_value *kv = *kvp;
2500	* we need to store the null value so when we save	2367	*kvp = (*kvp)->next;
2501	* it, we save the empty value so that when we load,	2368	delete kv;
2502	* we get this value back again.	2369	return;
2503	*/
2504	if (get_ob_key_link (op->arch, canonical_key))
2505	field->value = 0;
2506	else
2507	{
2508	if (last)
2509	last->next = field->next;
2510	else
2511	op->key_values = field->next;
2512
2513	delete field;
2514	}
2515	}	2370	}
2516	return TRUE;
2517	}
2518	/* IF we get here, key doesn't exist */
2519
2520	/* No field, we'll have to add it. */
2521
2522	if (!add_key)
2523	return FALSE;
2524
2525	/* There isn't any good reason to store a null
2526	* value in the key/value list. If the archetype has
2527	* this key, then we should also have it, so shouldn't
2528	* be here. If user wants to store empty strings,
2529	* should pass in ""
2530	*/
2531	if (value == NULL)
2532	return TRUE;
2533
2534	field = new key_value;
2535
2536	field->key = canonical_key;
2537	field->value = value;
2538	/* Usual prepend-addition. */
2539	field->next = op->key_values;
2540	op->key_values = field;
2541
2542	return TRUE;
2543	}
2544
2545	/*
2546	* Updates the key in op to value.
2547	*
2548	* If add_key is FALSE, this will only update existing keys,
2549	* and not add new ones.
2550	* In general, should be little reason FALSE is ever passed in for add_key
2551	*
2552	* Returns TRUE on success.
2553	*/
2554	int
2555	set_ob_key_value (object *op, const char *key, const char *value, int add_key)
2556	{
2557	shstr key_ (key);
2558
2559	return set_ob_key_value_s (op, key_, value, add_key);
2560	}	2371	}
2561		2372
2562	object::depth_iterator::depth_iterator (object *container)	2373	object::depth_iterator::depth_iterator (object *container)
2563	: iterator_base (container)	2374	: iterator_base (container)
2564	{	2375	{
…		…
2614	{	2425	{
2615	char flagdesc[512];	2426	char flagdesc[512];
2616	char info2[256 * 4];	2427	char info2[256 * 4];
2617	char *p = info;	2428	char *p = info;
2618		2429
2619	p += snprintf (p, 512, "{cnt:%d,uuid:%s,name:\"%s\"%s%s,flags:[%s],type:%d}",	2430	p += snprintf (p, 512, "{cnt:%d,uuid:%s,name:\"%s\"%s%s%s,flags:[%s],type:%d}",
2620	count,	2431	count,
2621	uuid.c_str (),	2432	uuid.c_str (),
2622	&name,	2433	&name,
2623	title ? "\",title:\"" : "",	2434	title ? ",title:\"" : "",
2624	title ? (const char *)title : "",	2435	title ? (const char *)title : "",
		2436	title ? "\"" : "",
2625	flag_desc (flagdesc, 512), type);	2437	flag_desc (flagdesc, 512), type);
2626		2438
2627	if (!this->flag[FLAG_REMOVED] && env)	2439	if (!flag[FLAG_REMOVED] && env)
2628	p += snprintf (p, 256, "(in %s)", env->debug_desc (info2));	2440	p += snprintf (p, 256, "(in %s)", env->debug_desc (info2));
2629		2441
2630	if (map)	2442	if (map)
2631	p += snprintf (p, 256, "(on %s@%d+%d)", &map->path, x, y);	2443	p += snprintf (p, 256, "(on %s@%d+%d)", &map->path, x, y);
2632		2444
…		…
2662	object::open_container (object *new_container)	2474	object::open_container (object *new_container)
2663	{	2475	{
2664	if (container == new_container)	2476	if (container == new_container)
2665	return;	2477	return;
2666		2478
2667	if (object *old_container = container)	2479	object *old_container = container;
		2480
		2481	if (old_container)
2668	{	2482	{
2669	if (INVOKE_OBJECT (CLOSE, old_container, ARG_OBJECT (this)))	2483	if (INVOKE_OBJECT (CLOSE, old_container, ARG_OBJECT (this)))
2670	return;	2484	return;
2671		2485
2672	#if 0	2486	#if 0
…		…
2674	if (object *closer = old_container->inv)	2488	if (object *closer = old_container->inv)
2675	if (closer->type == CLOSE_CON)	2489	if (closer->type == CLOSE_CON)
2676	closer->destroy ();	2490	closer->destroy ();
2677	#endif	2491	#endif
2678		2492
		2493	// make sure the container is available
		2494	esrv_send_item (this, old_container);
		2495
2679	old_container->flag [FLAG_APPLIED] = 0;	2496	old_container->flag [FLAG_APPLIED] = false;
2680	container = 0;	2497	container = 0;
2681		2498
		2499	// client needs item update to make it work, client bug requires this to be separate
2682	esrv_update_item (UPD_FLAGS, this, old_container);	2500	esrv_update_item (UPD_FLAGS, this, old_container);
		2501
2683	new_draw_info_format (NDI_UNIQUE, 0, this, "You close %s.", query_name (old_container));	2502	new_draw_info_format (NDI_UNIQUE, 0, this, "You close %s.", query_name (old_container));
2684	play_sound (sound_find ("chest_close"));	2503	play_sound (sound_find ("chest_close"));
2685	}	2504	}
2686		2505
2687	if (new_container)	2506	if (new_container)
…		…
2700	}	2519	}
2701	#endif	2520	#endif
2702		2521
2703	new_draw_info_format (NDI_UNIQUE, 0, this, "You open %s.", query_name (new_container));	2522	new_draw_info_format (NDI_UNIQUE, 0, this, "You open %s.", query_name (new_container));
2704		2523
		2524	// make sure the container is available, client bug requires this to be separate
		2525	esrv_send_item (this, new_container);
		2526
2705	new_container->flag [FLAG_APPLIED] = 1;	2527	new_container->flag [FLAG_APPLIED] = true;
2706	container = new_container;	2528	container = new_container;
2707		2529
		2530	// client needs flag change
2708	esrv_update_item (UPD_FLAGS, this, new_container);	2531	esrv_update_item (UPD_FLAGS, this, new_container);
2709	esrv_send_inventory (this, new_container);	2532	esrv_send_inventory (this, new_container);
2710	play_sound (sound_find ("chest_open"));	2533	play_sound (sound_find ("chest_open"));
2711	}	2534	}
		2535	// else if (!old_container->env && contr && contr->ns)
		2536	// contr->ns->floorbox_reset ();
2712	}	2537	}
2713		2538
2714	object *	2539	object *
2715	object::force_find (const shstr name)	2540	object::force_find (const shstr name)
2716	{	2541	{
…		…
2742		2567
2743	insert (force);	2568	insert (force);
2744	}	2569	}
2745		2570
2746	void	2571	void
2747	object::play_sound (faceidx sound) const	2572	object::play_sound (faceidx sound)
2748	{	2573	{
2749	if (!sound)	2574	if (!sound)
2750	return;	2575	return;
2751		2576
2752	if (flag [FLAG_REMOVED])	2577	if (flag [FLAG_REMOVED])

Diff Legend

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