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

Diff Legend

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