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.219 by root, Wed Apr 23 21:09:10 2008 UTC vs.
Revision 1.291 by root, Mon Oct 12 14:00:57 2009 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,2008 Marc Alexander Lehmann / Robin Redeker / the Deliantra team	4	* Copyright (©) 2005,2006,2007,2008,2009 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 it under
9	* it under the terms of the GNU General Public License as published by	9	* the terms of the Affero GNU General Public License as published by the
10	* the Free Software Foundation, either version 3 of the License, or	10	* Free Software Foundation, either version 3 of the License, or (at your
11	* (at your option) any later version.	11	* option) any later version.
12	*	12	*
13	* This program is distributed in the hope that it will be useful,	13	* This program is distributed in the hope that it will be useful,
14	* but WITHOUT ANY WARRANTY; without even the implied warranty of	14	* but WITHOUT ANY WARRANTY; without even the implied warranty of
15	* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the	15	* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16	* GNU General Public License for more details.	16	* GNU General Public License for more details.
17	*	17	*
18	* You should have received a copy of the GNU General Public License	18	* You should have received a copy of the Affero GNU General Public License
19	* along with this program. If not, see <http://www.gnu.org/licenses/>.	19	* and the GNU General Public License along with this program. If not, see
		20	* <http://www.gnu.org/licenses/>.
20	*	21	*
21	* The authors can be reached via e-mail to <support@deliantra.net>	22	* The authors can be reached via e-mail to <support@deliantra.net>
22	*/	23	*/
23		24
24	#include <global.h>	25	#include <global.h>
…		…
32	#include <bitset>	33	#include <bitset>
33		34
34	UUID UUID::cur;	35	UUID UUID::cur;
35	static uint64_t seq_next_save;	36	static uint64_t seq_next_save;
36	static const uint64 UUID_GAP = 1<<19;	37	static const uint64 UUID_GAP = 1<<19;
		38	uint32_t mapspace::smellcount = 10000;
37		39
38	objectvec objects;	40	objectvec objects;
39	activevec actives;	41	activevec actives;
40		42
41	short freearr_x[SIZEOFFREE] = {	43	short freearr_x[SIZEOFFREE] = {
…		…
139		141
140	/* Returns TRUE if every key_values in wants has a partner with the same value in has. */	142	/* Returns TRUE if every key_values in wants has a partner with the same value in has. */
141	static bool	143	static bool
142	compare_ob_value_lists_one (const object *wants, const object *has)	144	compare_ob_value_lists_one (const object *wants, const object *has)
143	{	145	{
144	key_value *wants_field;
145
146	/* n-squared behaviour (see get_ob_key_link()), but I'm hoping both	146	/* n-squared behaviour (see kv_get), but I'm hoping both
147	* objects with lists are rare, and lists stay short. If not, use a	147	* objects with lists are rare, and lists stay short. If not, use a
148	* different structure or at least keep the lists sorted...	148	* different structure or at least keep the lists sorted...
149	*/	149	*/
150		150
151	/* For each field in wants, */	151	/* For each field in wants, */
152	for (wants_field = wants->key_values; wants_field; wants_field = wants_field->next)	152	for (key_value *kv = wants->key_values; kv; kv = kv->next)
153	{	153	if (has->kv_get (kv->key) != kv->value)
154	key_value *has_field;	154	return false;
155
156	/* Look for a field in has with the same key. */
157	has_field = get_ob_key_link (has, wants_field->key);
158
159	if (!has_field)
160	return 0; /* No field with that name. */
161
162	/* Found the matching field. */
163	if (has_field->value != wants_field->value)
164	return 0; /* Values don't match, so this half of the comparison is false. */
165
166	/* If we get here, we found a match. Now for the next field in wants. */
167	}
168		155
169	/* If we get here, every field in wants has a matching field in has. */	156	/* If we get here, every field in wants has a matching field in has. */
170	return 1;	157	return true;
171	}	158	}
172		159
173	/* Returns TRUE if ob1 has the same key_values as ob2. */	160	/* Returns TRUE if ob1 has the same key_values as ob2. */
174	static bool	161	static bool
175	compare_ob_value_lists (const object *ob1, const object *ob2)	162	compare_ob_value_lists (const object *ob1, const object *ob2)
176	{	163	{
177	/* However, there may be fields in has which aren't partnered in wants,	164	/* However, there may be fields in has which aren't partnered in wants,
178	* so we need to run the comparison *twice*. :(	165	* so we need to run the comparison *twice*. :(
179	*/	166	*/
180	return compare_ob_value_lists_one (ob1, ob2) && compare_ob_value_lists_one (ob2, ob1);	167	return compare_ob_value_lists_one (ob1, ob2)
		168	&& compare_ob_value_lists_one (ob2, ob1);
181	}	169	}
182		170
183	/* Function examines the 2 objects given to it, and returns true if	171	/* Function examines the 2 objects given to it, and returns true if
184	* they can be merged together.	172	* they can be merged together.
185	*	173	*
…		…
200	|| ob1->speed != ob2->speed	188	|| ob1->speed != ob2->speed
201	|| ob1->value != ob2->value	189	|| ob1->value != ob2->value
202	|| ob1->name != ob2->name)	190	|| ob1->name != ob2->name)
203	return 0;	191	return 0;
204		192
205	/* Do not merge objects if nrof would overflow. First part checks	193	/* Do not merge objects if nrof would overflow, assume nrof
206	* for unsigned overflow (2c), second part checks whether the result	194	* is always 0 .. 2**31-1 */
207	* would fit into a 32 bit signed int, which is often used to hold	195	if (ob1->nrof > 0x7fffffff - ob2->nrof)
208	* nrof values.
209	*/
210	if (~ob1->nrof < ob2->nrof || ob1->nrof + ob2->nrof > (1UL << 31))
211	return 0;	196	return 0;
212		197
213	/* If the objects have been identified, set the BEEN_APPLIED flag.	198	/* If the objects have been identified, set the BEEN_APPLIED flag.
214	* This is to the comparison of the flags below will be OK. We	199	* This is to the comparison of the flags below will be OK. We
215	* just can't ignore the been applied or identified flags, as they	200	* just can't ignore the been applied or identified flags, as they
…		…
220	SET_FLAG (ob1, FLAG_BEEN_APPLIED);	205	SET_FLAG (ob1, FLAG_BEEN_APPLIED);
221		206
222	if (QUERY_FLAG (ob2, FLAG_IDENTIFIED))	207	if (QUERY_FLAG (ob2, FLAG_IDENTIFIED))
223	SET_FLAG (ob2, FLAG_BEEN_APPLIED);	208	SET_FLAG (ob2, FLAG_BEEN_APPLIED);
224		209
225	if (ob1->arch->name != ob2->arch->name	210	if (ob1->arch->archname != ob2->arch->archname
226	|| ob1->name != ob2->name	211	|| ob1->name != ob2->name
227	|| ob1->title != ob2->title	212	|| ob1->title != ob2->title
228	|| ob1->msg != ob2->msg	213	|| ob1->msg != ob2->msg
229	|| ob1->weight != ob2->weight	214	|| ob1->weight != ob2->weight
230	|| ob1->attacktype != ob2->attacktype	215	|| ob1->attacktype != ob2->attacktype
231	|| ob1->magic != ob2->magic	216	|| ob1->magic != ob2->magic
232	|| ob1->slaying != ob2->slaying	217	|| ob1->slaying != ob2->slaying
233	|| ob1->skill != ob2->skill	218	|| ob1->skill != ob2->skill
234	|| ob1->value != ob2->value	219	|| ob1->value != ob2->value
235	|| ob1->animation_id != ob2->animation_id	220	|| ob1->animation_id != ob2->animation_id
		221	|| (ob1->face != ob2->face && !ob1->animation_id) // face and animation are dependent on each other
236	|| ob1->client_type != ob2->client_type	222	|| ob1->client_type != ob2->client_type
237	|| ob1->materialname != ob2->materialname	223	|| ob1->materialname != ob2->materialname
238	|| ob1->lore != ob2->lore	224	|| ob1->lore != ob2->lore
239	|| ob1->subtype != ob2->subtype	225	|| ob1->subtype != ob2->subtype
240	|| ob1->move_type != ob2->move_type	226	|| ob1->move_type != ob2->move_type
…		…
248	|| memcmp (&ob1->stats , &ob2->stats , sizeof (ob1->stats)))	234	|| memcmp (&ob1->stats , &ob2->stats , sizeof (ob1->stats)))
249	return 0;	235	return 0;
250		236
251	if ((ob1->flag ^ ob2->flag)	237	if ((ob1->flag ^ ob2->flag)
252	.reset (FLAG_INV_LOCKED)	238	.reset (FLAG_INV_LOCKED)
253	.reset (FLAG_CLIENT_SENT)
254	.reset (FLAG_REMOVED)	239	.reset (FLAG_REMOVED)
255	.any ())	240	.any ())
256	return 0;	241	return 0;
257		242
258	/* This is really a spellbook check - we should in general	243	/* This is really a spellbook check - we should in general
…		…
334		319
335	// find player who can see this object	320	// find player who can see this object
336	object *	321	object *
337	object::visible_to () const	322	object::visible_to () const
338	{	323	{
339	if (!flag [FLAG_REMOVED])	324	if (client_visible () && !flag [FLAG_REMOVED])
340	{	325	{
341	// see if we are in a container of sorts	326	// see if we are in a container of sorts
342	if (env)	327	if (env)
343	{	328	{
344	// the player inventory itself is always visible	329	// the player inventory itself is always visible
345	if (env->type == PLAYER)	330	if (env->is_player ())
346	return env;	331	return env;
347		332
348	// else a player could have our env open	333	// else a player could have our env open
349	object *envest = env->outer_env ();	334	object *envest = env->outer_env_or_self ();
350		335
351	// the player itself is always on a map, so we will find him here	336	// the player itself is always on a map, so we will find him here
352	// even if our inv is in a player.	337	// even if our inv is in a player.
353	if (envest->is_on_map ())	338	if (envest->is_on_map ())
354	if (object *pl = envest->ms ().player ())	339	if (object *pl = envest->ms ().player ())
…		…
358	else	343	else
359	{	344	{
360	// maybe there is a player standing on the same mapspace	345	// maybe there is a player standing on the same mapspace
361	// this will catch the case where "this" is a player	346	// this will catch the case where "this" is a player
362	if (object *pl = ms ().player ())	347	if (object *pl = ms ().player ())
		348	if ((!pl->container && pl->contr->ns && !pl->contr->ns->update_look)
		349	|| this == pl->container)
363	return pl;	350	return pl;
364	}	351	}
365	}	352	}
366		353
367	return 0;	354	return 0;
368	}	355	}
369		356
370	// adjust weight per container type ("of holding")	357	// adjust weight per container type ("of holding")
371	static sint32	358	static sint32
372	weight_adjust (object *op, sint32 weight)	359	weight_adjust_for (object *op, sint32 weight)
373	{	360	{
374	return op->type == CONTAINER	361	return op->type == CONTAINER
375	? lerp (weight, 0, 100, 0, 100 - op->stats.Str)	362	? lerp (weight, 0, 100, 0, 100 - op->stats.Str)
376	: weight;	363	: weight;
377	}	364	}
…		…
383	static void	370	static void
384	adjust_weight (object *op, sint32 weight)	371	adjust_weight (object *op, sint32 weight)
385	{	372	{
386	while (op)	373	while (op)
387	{	374	{
		375	// adjust by actual difference to account for rounding errors
		376	// i.e. (w2 - w1) / f != w2 / f - w1 / f and the latter is correct
388	weight = weight_adjust (op, weight);	377	weight = weight_adjust_for (op, op->carrying)
		378	- weight_adjust_for (op, op->carrying - weight);
389		379
390	if (!weight)	380	if (!weight)
391	return;	381	return;
392		382
393	op->carrying += weight;	383	op->carrying += weight;
…		…
416	op->update_weight ();	406	op->update_weight ();
417		407
418	sum += op->total_weight ();	408	sum += op->total_weight ();
419	}	409	}
420		410
421	sum = weight_adjust (this, sum);	411	sum = weight_adjust_for (this, sum);
422		412
423	if (sum != carrying)	413	if (sum != carrying)
424	{	414	{
425	carrying = sum;	415	carrying = sum;
426		416
…		…
440	return strdup ("[NULLOBJ]");	430	return strdup ("[NULLOBJ]");
441		431
442	object_freezer freezer;	432	object_freezer freezer;
443	op->write (freezer);	433	op->write (freezer);
444	return freezer.as_string ();	434	return freezer.as_string ();
		435	}
		436
		437	char *
		438	object::as_string ()
		439	{
		440	return dump_object (this);
445	}	441	}
446		442
447	/*	443	/*
448	* get_nearest_part(multi-object, object 2) returns the part of the	444	* get_nearest_part(multi-object, object 2) returns the part of the
449	* multi-object 1 which is closest to the second object.	445	* multi-object 1 which is closest to the second object.
…		…
488	*/	484	*/
489	object *	485	object *
490	find_object_name (const char *str)	486	find_object_name (const char *str)
491	{	487	{
492	shstr_cmp str_ (str);	488	shstr_cmp str_ (str);
493	object *op;
494		489
		490	if (str_)
495	for_all_objects (op)	491	for_all_objects (op)
496	if (op->name == str_)	492	if (op->name == str_)
497	break;	493	return op;
498		494
499	return op;	495	return 0;
500	}	496	}
501		497
502	/*	498	/*
503	* Sets the owner and sets the skill and exp pointers to owner's current	499	* Sets the owner and sets the skill and exp pointers to owner's current
504	* skill and experience objects.	500	* skill and experience objects.
…		…
566	update_stats ();	562	update_stats ();
567		563
568	new_draw_info_format (NDI_UNIQUE, 0, this,	564	new_draw_info_format (NDI_UNIQUE, 0, this,
569	"You try to balance all your items at once, "	565	"You try to balance all your items at once, "
570	"but the %s is just too much for your body. "	566	"but the %s is just too much for your body. "
571	"[You need to unapply some items first.]", &ob->name);	567	"[You need to unapply some items first - use the 'body' command to see "
		568	"how many items you cna wera on a specific body part.]", &ob->name);
572	return false;	569	return false;
573	}	570	}
574		571
575	//new_draw_info_format (NDI_UNIQUE, 0, this, "You switch to your %s.", &ob->name);	572	//new_draw_info_format (NDI_UNIQUE, 0, this, "You switch to your %s.", &ob->name);
576	}	573	}
…		…
602	}	599	}
603		600
604	op->key_values = 0;	601	op->key_values = 0;
605	}	602	}
606		603
607	object &	604	/*
608	object::operator =(const object &src)	605	* copy_to first frees everything allocated by the dst object,
		606	* and then copies the contents of itself into the second
		607	* object, allocating what needs to be allocated. Basically, any
		608	* data that is malloc'd needs to be re-malloc/copied. Otherwise,
		609	* if the first object is freed, the pointers in the new object
		610	* will point at garbage.
		611	*/
		612	void
		613	object::copy_to (object *dst)
609	{	614	{
610	bool is_freed = flag [FLAG_FREED];	615	dst->remove ();
611	bool is_removed = flag [FLAG_REMOVED];
612
613	*(object_copy *)this = src;	616	*(object_copy *)dst = *this;
614
615	flag [FLAG_FREED] = is_freed;
616	flag [FLAG_REMOVED] = is_removed;	617	dst->flag [FLAG_REMOVED] = true;
617		618
618	/* Copy over key_values, if any. */	619	/* Copy over key_values, if any. */
619	if (src.key_values)	620	if (key_values)
620	{	621	{
621	key_value *tail = 0;	622	key_value *tail = 0;
622	key_values = 0;	623	dst->key_values = 0;
623		624
624	for (key_value *i = src.key_values; i; i = i->next)	625	for (key_value *i = key_values; i; i = i->next)
625	{	626	{
626	key_value *new_link = new key_value;	627	key_value *new_link = new key_value;
627		628
628	new_link->next = 0;	629	new_link->next = 0;
629	new_link->key = i->key;	630	new_link->key = i->key;
630	new_link->value = i->value;	631	new_link->value = i->value;
631		632
632	/* Try and be clever here, too. */	633	/* Try and be clever here, too. */
633	if (!key_values)	634	if (!dst->key_values)
634	{	635	{
635	key_values = new_link;	636	dst->key_values = new_link;
636	tail = new_link;	637	tail = new_link;
637	}	638	}
638	else	639	else
639	{	640	{
640	tail->next = new_link;	641	tail->next = new_link;
641	tail = new_link;	642	tail = new_link;
642	}	643	}
643	}	644	}
644	}	645	}
645	}
646
647	/*
648	* copy_to first frees everything allocated by the dst object,
649	* and then copies the contents of itself into the second
650	* object, allocating what needs to be allocated. Basically, any
651	* data that is malloc'd needs to be re-malloc/copied. Otherwise,
652	* if the first object is freed, the pointers in the new object
653	* will point at garbage.
654	*/
655	void
656	object::copy_to (object *dst)
657	{
658	*dst = *this;
659		646
660	if (speed < 0)	647	if (speed < 0)
661	dst->speed_left -= rndm ();	648	dst->speed_left -= rndm ();
662		649
663	dst->set_speed (dst->speed);	650	dst->activate ();
664	}	651	}
665		652
666	void	653	void
667	object::instantiate ()	654	object::instantiate ()
668	{	655	{
…		…
685	object *	672	object *
686	object::clone ()	673	object::clone ()
687	{	674	{
688	object *neu = create ();	675	object *neu = create ();
689	copy_to (neu);	676	copy_to (neu);
		677	neu->map = map; // not copied by copy_to
690	return neu;	678	return neu;
691	}	679	}
692		680
693	/*	681	/*
694	* If an object with the IS_TURNABLE() flag needs to be turned due	682	* If an object with the IS_TURNABLE() flag needs to be turned due
…		…
745	* UP_OBJ_FACE: only the objects face has changed.	733	* UP_OBJ_FACE: only the objects face has changed.
746	*/	734	*/
747	void	735	void
748	update_object (object *op, int action)	736	update_object (object *op, int action)
749	{	737	{
750	if (op == NULL)	738	if (!op)
751	{	739	{
752	/* this should never happen */	740	/* this should never happen */
753	LOG (llevDebug, "update_object() called for NULL object.\n");	741	LOG (llevError | logBacktrace, "update_object() called for NULL object.\n");
754	return;	742	return;
755	}	743	}
756		744
757	if (op->env)	745	if (!op->is_on_map ())
758	{	746	{
759	/* Animation is currently handled by client, so nothing	747	/* Animation is currently handled by client, so nothing
760	* to do in this case.	748	* to do in this case.
761	*/	749	*/
762	return;	750	return;
763	}	751	}
764
765	/* If the map is saving, don't do anything as everything is
766	* going to get freed anyways.
767	*/
768	if (!op->map || op->map->in_memory == MAP_SAVING)
769	return;
770		752
771	/* make sure the object is within map boundaries */	753	/* make sure the object is within map boundaries */
772	if (op->x < 0 || op->x >= op->map->width || op->y < 0 || op->y >= op->map->height)	754	if (op->x < 0 || op->x >= op->map->width || op->y < 0 || op->y >= op->map->height)
773	{	755	{
774	LOG (llevError, "update_object() called for object out of map!\n");	756	LOG (llevError, "update_object() called for object out of map!\n");
…		…
785	else if (action == UP_OBJ_INSERT)	767	else if (action == UP_OBJ_INSERT)
786	{	768	{
787	// this is likely overkill, TODO: revisit (schmorp)	769	// this is likely overkill, TODO: revisit (schmorp)
788	if ((QUERY_FLAG (op, FLAG_BLOCKSVIEW) && !(m.flags_ & P_BLOCKSVIEW))	770	if ((QUERY_FLAG (op, FLAG_BLOCKSVIEW) && !(m.flags_ & P_BLOCKSVIEW))
789	|| (QUERY_FLAG (op, FLAG_NO_MAGIC) && !(m.flags_ & P_NO_MAGIC))	771	|| (QUERY_FLAG (op, FLAG_NO_MAGIC) && !(m.flags_ & P_NO_MAGIC))
790	|| (op->type == PLAYER && !(m.flags_ & P_PLAYER))	772	|| (op->is_player () && !(m.flags_ & P_PLAYER))
791	|| (op->type == SAFE_GROUND && !(m.flags_ & P_SAFE))	773	|| (op->type == SAFE_GROUND && !(m.flags_ & P_SAFE))
792	|| (QUERY_FLAG (op, FLAG_ALIVE) && !(m.flags_ & P_IS_ALIVE))	774	|| (QUERY_FLAG (op, FLAG_ALIVE) && !(m.flags_ & P_IS_ALIVE))
793	|| (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC))	775	|| (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC))
794	|| (m.move_on | op->move_on ) != m.move_on	776	|| (m.move_on | op->move_on ) != m.move_on
795	|| (m.move_off | op->move_off ) != m.move_off	777	|| (m.move_off | op->move_off ) != m.move_off
796	|| (m.move_slow | op->move_slow) != m.move_slow	778	|| (m.move_slow | op->move_slow) != m.move_slow
797	/* This isn't perfect, but I don't expect a lot of objects to	779	/* This isn't perfect, but I don't expect a lot of objects to
798	* to have move_allow right now.	780	* have move_allow right now.
799	*/	781	*/
800	|| ((m.move_block | op->move_block) & ~op->move_allow) != m.move_block	782	|| ((m.move_block | op->move_block) & ~op->move_allow) != m.move_block
801	|| 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp)	783	|| 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp)
802	m.flags_ = 0;	784	m.invalidate ();
803	}	785	}
804	/* if the object is being removed, we can't make intelligent	786	/* if the object is being removed, we can't make intelligent
805	* decisions, because remove_ob can't really pass the object	787	* decisions, because remove_ob can't really pass the object
806	* that is being removed.	788	* that is being removed.
807	*/	789	*/
808	else if (action == UP_OBJ_CHANGE || action == UP_OBJ_REMOVE)	790	else if (action == UP_OBJ_CHANGE || action == UP_OBJ_REMOVE)
809	m.flags_ = 0;	791	m.invalidate ();
810	else if (action == UP_OBJ_FACE)	792	else if (action == UP_OBJ_FACE)
811	/* Nothing to do for that case */ ;	793	/* Nothing to do for that case */ ;
812	else	794	else
813	LOG (llevError, "update_object called with invalid action: %d\n", action);	795	LOG (llevError, "update_object called with invalid action: %d\n", action);
814		796
…		…
818		800
819	object::object ()	801	object::object ()
820	{	802	{
821	SET_FLAG (this, FLAG_REMOVED);	803	SET_FLAG (this, FLAG_REMOVED);
822		804
823	expmul = 1.0;	805	//expmul = 1.0; declared const for the time being
824	face = blank_face;	806	face = blank_face;
825	}	807	}
826		808
827	object::~object ()	809	object::~object ()
828	{	810	{
…		…
858	/* If already on active list, don't do anything */	840	/* If already on active list, don't do anything */
859	if (active)	841	if (active)
860	return;	842	return;
861		843
862	if (has_active_speed ())	844	if (has_active_speed ())
		845	{
		846	if (flag [FLAG_FREED])
		847	LOG (llevError | logBacktrace, "BUG: tried to activate freed object %s\n", debug_desc ());//D
		848
863	actives.insert (this);	849	actives.insert (this);
		850	}
864	}	851	}
865		852
866	void	853	void
867	object::activate_recursive ()	854	object::activate_recursive ()
868	{	855	{
…		…
929	* drop on that space.	916	* drop on that space.
930	*/	917	*/
931	if (!drop_to_ground	918	if (!drop_to_ground
932	|| !map	919	|| !map
933	|| map->in_memory != MAP_ACTIVE	920	|| map->in_memory != MAP_ACTIVE
934	|| map->nodrop	921	|| map->no_drop
935	|| ms ().move_block == MOVE_ALL)	922	|| ms ().move_block == MOVE_ALL)
936	{	923	{
937	while (inv)	924	while (inv)
938	{
939	inv->destroy_inv (false);
940	inv->destroy ();	925	inv->destroy ();
941	}
942	}	926	}
943	else	927	else
944	{ /* Put objects in inventory onto this space */	928	{ /* Put objects in inventory onto this space */
945	while (inv)	929	while (inv)
946	{	930	{
…		…
950	|| op->flag [FLAG_NO_DROP]	934	|| op->flag [FLAG_NO_DROP]
951	|| op->type == RUNE	935	|| op->type == RUNE
952	|| op->type == TRAP	936	|| op->type == TRAP
953	|| op->flag [FLAG_IS_A_TEMPLATE]	937	|| op->flag [FLAG_IS_A_TEMPLATE]
954	|| op->flag [FLAG_DESTROY_ON_DEATH])	938	|| op->flag [FLAG_DESTROY_ON_DEATH])
955	op->destroy (true);	939	op->destroy ();
956	else	940	else
957	map->insert (op, x, y);	941	map->insert (op, x, y);
958	}	942	}
959	}	943	}
960	}	944	}
…		…
964	object *op = new object;	948	object *op = new object;
965	op->link ();	949	op->link ();
966	return op;	950	return op;
967	}	951	}
968		952
		953	static struct freed_map : maptile
		954	{
		955	freed_map ()
		956	{
		957	path = "<freed objects map>";
		958	name = "/internal/freed_objects_map";
		959	width = 3;
		960	height = 3;
		961	no_drop = 1;
		962	no_reset = 1;
		963
		964	alloc ();
		965	in_memory = MAP_ACTIVE;
		966	}
		967
		968	~freed_map ()
		969	{
		970	destroy ();
		971	}
		972	} freed_map; // freed objects are moved here to avoid crashes
		973
969	void	974	void
970	object::do_destroy ()	975	object::do_destroy ()
971	{	976	{
972	if (flag [FLAG_IS_LINKED])	977	if (flag [FLAG_IS_LINKED])
973	remove_button_link (this);	978	remove_link ();
974		979
975	if (flag [FLAG_FRIENDLY])	980	if (flag [FLAG_FRIENDLY])
976	remove_friendly_object (this);	981	remove_friendly_object (this);
977		982
978	remove ();	983	remove ();
…		…
983	unlink ();	988	unlink ();
984		989
985	flag [FLAG_FREED] = 1;	990	flag [FLAG_FREED] = 1;
986		991
987	// hack to ensure that freed objects still have a valid map	992	// hack to ensure that freed objects still have a valid map
988	{
989	static maptile *freed_map; // freed objects are moved here to avoid crashes
990
991	if (!freed_map)
992	{
993	freed_map = new maptile;
994
995	freed_map->path = "<freed objects map>";
996	freed_map->name = "/internal/freed_objects_map";
997	freed_map->width = 3;
998	freed_map->height = 3;
999	freed_map->nodrop = 1;
1000
1001	freed_map->alloc ();
1002	freed_map->in_memory = MAP_ACTIVE;
1003	}
1004
1005	map = freed_map;	993	map = &freed_map;
1006	x = 1;	994	x = 1;
1007	y = 1;	995	y = 1;
1008	}
1009		996
1010	if (more)	997	if (more)
1011	{	998	{
1012	more->destroy ();	999	more->destroy ();
1013	more = 0;	1000	more = 0;
…		…
1021	attacked_by = 0;	1008	attacked_by = 0;
1022	current_weapon = 0;	1009	current_weapon = 0;
1023	}	1010	}
1024		1011
1025	void	1012	void
1026	object::destroy (bool destroy_inventory)	1013	object::destroy ()
1027	{	1014	{
1028	if (destroyed ())	1015	if (destroyed ())
1029	return;	1016	return;
1030		1017
1031	if (!is_head () && !head->destroyed ())	1018	if (!is_head () && !head->destroyed ())
1032	{	1019	{
1033	LOG (llevError | logBacktrace, "tried to destroy the tail of an object");	1020	LOG (llevError | logBacktrace, "tried to destroy the tail of an object");
1034	head->destroy (destroy_inventory);	1021	head->destroy ();
		1022	return;
1035	}	1023	}
1036		1024
1037	destroy_inv (!destroy_inventory);	1025	destroy_inv (false);
1038		1026
1039	if (is_head ())	1027	if (is_head ())
1040	if (sound_destroy)	1028	if (sound_destroy)
1041	play_sound (sound_destroy);	1029	play_sound (sound_destroy);
1042	else if (flag [FLAG_MONSTER])	1030	else if (flag [FLAG_MONSTER])
…		…
1061	if (flag [FLAG_REMOVED])	1049	if (flag [FLAG_REMOVED])
1062	return;	1050	return;
1063		1051
1064	INVOKE_OBJECT (REMOVE, this);	1052	INVOKE_OBJECT (REMOVE, this);
1065		1053
1066	if (object *pl = visible_to ())
1067	esrv_del_item (pl->contr, count);
1068
1069	flag [FLAG_REMOVED] = true;	1054	flag [FLAG_REMOVED] = true;
1070		1055
1071	if (more)	1056	if (more)
1072	more->remove ();	1057	more->remove ();
1073		1058
…		…
1075	* In this case, the object to be removed is in someones	1060	* In this case, the object to be removed is in someones
1076	* inventory.	1061	* inventory.
1077	*/	1062	*/
1078	if (env)	1063	if (env)
1079	{	1064	{
		1065	flag [FLAG_REMOVED] = false; // hack around the issue of visible_to checking flag_removed
		1066	if (object *pl = visible_to ())
		1067	esrv_del_item (pl->contr, count);
		1068	flag [FLAG_REMOVED] = true; // hack around the issue of visible_to checking flag_removed
		1069
1080	adjust_weight (env, -total_weight ());	1070	adjust_weight (env, -total_weight ());
1081		1071
1082	*(above ? &above->below : &env->inv) = below;	1072	object *pl = in_player ();
1083
1084	if (below)
1085	below->above = above;
1086		1073
1087	/* we set up values so that it could be inserted into	1074	/* we set up values so that it could be inserted into
1088	* the map, but we don't actually do that - it is up	1075	* the map, but we don't actually do that - it is up
1089	* to the caller to decide what we want to do.	1076	* to the caller to decide what we want to do.
1090	*/	1077	*/
1091	map = env->map;	1078	map = env->map;
1092	x = env->x;	1079	x = env->x;
1093	y = env->y;	1080	y = env->y;
		1081
		1082	// make sure cmov optimisation is applicable
		1083	*(above ? &above->below : &env->inv) = below;
		1084	*(below ? &below->above : &above ) = above; // &above is just a dummy
		1085
1094	above = 0;	1086	above = 0;
1095	below = 0;	1087	below = 0;
1096	env = 0;	1088	env = 0;
1097		1089
1098	/* NO_FIX_PLAYER is set when a great many changes are being	1090	/* NO_FIX_PLAYER is set when a great many changes are being
1099	* made to players inventory. If set, avoiding the call	1091	* made to players inventory. If set, avoiding the call
1100	* to save cpu time.	1092	* to save cpu time.
1101	*/	1093	*/
1102	if ((otmp = in_player ()) && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))	1094	if (pl)
		1095	if (pl->is_player () && (glow_radius || !QUERY_FLAG (pl, FLAG_NO_FIX_PLAYER)))
		1096	{
1103	otmp->update_stats ();	1097	pl->update_stats ();
		1098
		1099	if (glow_radius && pl->is_on_map ())
		1100	update_all_los (pl->map, pl->x, pl->y);
		1101	}
1104	}	1102	}
1105	else if (map)	1103	else if (map)
1106	{	1104	{
1107	if (type == PLAYER)
1108	{
1109	// leaving a spot always closes any open container on the ground
1110	if (container && !container->env)
1111	// this causes spurious floorbox updates, but it ensures
1112	// that the CLOSE event is being sent.
1113	close_container ();
1114
1115	--map->players;
1116	map->touch ();
1117	}
1118
1119	map->dirty = true;	1105	map->dirty = true;
1120	mapspace &ms = this->ms ();	1106	mapspace &ms = this->ms ();
1121		1107
		1108	if (object *pl = ms.player ())
		1109	{
		1110	if (is_player ())
		1111	{
		1112	if (!flag [FLAG_WIZPASS])
		1113	ms.smell = ++mapspace::smellcount; // remember the smell of the player
		1114
		1115	// leaving a spot always closes any open container on the ground
		1116	if (container && !container->env)
		1117	// this causes spurious floorbox updates, but it ensures
		1118	// that the CLOSE event is being sent.
		1119	close_container ();
		1120
		1121	--map->players;
		1122	map->touch ();
		1123	}
		1124	else if (pl->container == this)
		1125	{
		1126	// removing a container should close it
		1127	close_container ();
		1128	}
		1129
		1130	esrv_del_item (pl->contr, count);
		1131	}
		1132
1122	/* link the object above us */	1133	/* link the object above us */
1123	if (above)	1134	// re-link, make sure compiler can easily use cmove
1124	above->below = below;	1135	*(above ? &above->below : &ms.top) = below;
1125	else	1136	*(below ? &below->above : &ms.bot) = above;
1126	ms.top = below; /* we were top, set new top */
1127
1128	/* Relink the object below us, if there is one */
1129	if (below)
1130	below->above = above;
1131	else
1132	{
1133	/* Nothing below, which means we need to relink map object for this space
1134	* use translated coordinates in case some oddness with map tiling is
1135	* evident
1136	*/
1137	if (GET_MAP_OB (map, x, y) != this)
1138	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 ());
1139
1140	ms.bot = above; /* goes on above it. */
1141	}
1142		1137
1143	above = 0;	1138	above = 0;
1144	below = 0;	1139	below = 0;
		1140
		1141	ms.invalidate ();
1145		1142
1146	if (map->in_memory == MAP_SAVING)	1143	if (map->in_memory == MAP_SAVING)
1147	return;	1144	return;
1148		1145
1149	int check_walk_off = !flag [FLAG_NO_APPLY];	1146	int check_walk_off = !flag [FLAG_NO_APPLY];
…		…
1156	* appropriately.	1153	* appropriately.
1157	*/	1154	*/
1158	pl->close_container ();	1155	pl->close_container ();
1159		1156
1160	//TODO: the floorbox prev/next might need updating	1157	//TODO: the floorbox prev/next might need updating
1161	esrv_del_item (pl->contr, count);	1158	//esrv_del_item (pl->contr, count);
		1159	//TODO: update floorbox to preserve ordering
		1160	if (pl->contr->ns)
		1161	pl->contr->ns->floorbox_update ();
1162	}	1162	}
1163		1163
1164	for (tmp = ms.bot; tmp; tmp = tmp->above)	1164	for (tmp = ms.bot; tmp; tmp = tmp->above)
1165	{	1165	{
1166	/* No point updating the players look faces if he is the object	1166	/* No point updating the players look faces if he is the object
…		…
1179	}	1179	}
1180		1180
1181	last = tmp;	1181	last = tmp;
1182	}	1182	}
1183		1183
1184	/* last == NULL if there are no objects on this space */	1184	if (affects_los ())
1185	//TODO: this makes little sense, why only update the topmost object?
1186	if (!last)
1187	map->at (x, y).flags_ = 0;
1188	else
1189	update_object (last, UP_OBJ_REMOVE);
1190
1191	if (flag [FLAG_BLOCKSVIEW] || glow_radius)
1192	update_all_los (map, x, y);	1185	update_all_los (map, x, y);
1193	}	1186	}
1194	}	1187	}
1195		1188
1196	/*	1189	/*
…		…
1220	esrv_update_item (UPD_NROF, pl, top);	1213	esrv_update_item (UPD_NROF, pl, top);
1221		1214
1222	op->weight = 0; // cancel the addition above	1215	op->weight = 0; // cancel the addition above
1223	op->carrying = 0; // must be 0 already	1216	op->carrying = 0; // must be 0 already
1224		1217
1225	op->destroy (1);	1218	op->destroy ();
1226		1219
1227	return top;	1220	return top;
1228	}	1221	}
1229		1222
1230	return 0;	1223	return 0;
…		…
1258	* job preparing multi-part monsters.	1251	* job preparing multi-part monsters.
1259	*/	1252	*/
1260	object *	1253	object *
1261	insert_ob_in_map_at (object *op, maptile *m, object *originator, int flag, int x, int y)	1254	insert_ob_in_map_at (object *op, maptile *m, object *originator, int flag, int x, int y)
1262	{	1255	{
		1256	op->remove ();
		1257
1263	for (object *tmp = op->head_ (); tmp; tmp = tmp->more)	1258	for (object *tmp = op->head_ (); tmp; tmp = tmp->more)
1264	{	1259	{
1265	tmp->x = x + tmp->arch->x;	1260	tmp->x = x + tmp->arch->x;
1266	tmp->y = y + tmp->arch->y;	1261	tmp->y = y + tmp->arch->y;
1267	}	1262	}
…		…
1290	* just 'op' otherwise	1285	* just 'op' otherwise
1291	*/	1286	*/
1292	object *	1287	object *
1293	insert_ob_in_map (object *op, maptile *m, object *originator, int flag)	1288	insert_ob_in_map (object *op, maptile *m, object *originator, int flag)
1294	{	1289	{
1295	assert (!op->flag [FLAG_FREED]);
1296
1297	op->remove ();	1290	op->remove ();
		1291
		1292	if (m == &freed_map)//D TODO: remove soon
		1293	{//D
		1294	LOG (llevError | logBacktrace, "tries to insret object on freed objects map: %s", op->debug_desc ());//D
		1295	}//D
1298		1296
1299	/* Ideally, the caller figures this out. However, it complicates a lot	1297	/* Ideally, the caller figures this out. However, it complicates a lot
1300	* of areas of callers (eg, anything that uses find_free_spot would now	1298	* of areas of callers (eg, anything that uses find_free_spot would now
1301	* need extra work	1299	* need extra work
1302	*/	1300	*/
		1301	maptile *newmap = m;
1303	if (!xy_normalise (m, op->x, op->y))	1302	if (!xy_normalise (newmap, op->x, op->y))
1304	{	1303	{
1305	op->destroy (1);	1304	op->head_ ()->destroy ();// remove head_ once all tail object destroyers found
1306	return 0;	1305	return 0;
1307	}	1306	}
1308		1307
1309	if (object *more = op->more)	1308	if (object *more = op->more)
1310	if (!insert_ob_in_map (more, m, originator, flag))	1309	if (!insert_ob_in_map (more, m, originator, flag))
1311	return 0;	1310	return 0;
1312		1311
1313	CLEAR_FLAG (op, FLAG_REMOVED);	1312	op->flag [FLAG_REMOVED] = false;
1314		1313	op->env = 0;
1315	op->map = m;	1314	op->map = newmap;
		1315
1316	mapspace &ms = op->ms ();	1316	mapspace &ms = op->ms ();
1317		1317
1318	/* this has to be done after we translate the coordinates.	1318	/* this has to be done after we translate the coordinates.
1319	*/	1319	*/
1320	if (op->nrof && !(flag & INS_NO_MERGE))	1320	if (op->nrof && !(flag & INS_NO_MERGE))
1321	for (object *tmp = ms.bot; tmp; tmp = tmp->above)	1321	for (object *tmp = ms.bot; tmp; tmp = tmp->above)
1322	if (object::can_merge (op, tmp))	1322	if (object::can_merge (op, tmp))
1323	{	1323	{
1324	// TODO: we atcually want to update tmp, not op,	1324	// TODO: we actually want to update tmp, not op,
1325	// but some caller surely breaks when we return tmp	1325	// but some caller surely breaks when we return tmp
1326	// from here :/	1326	// from here :/
1327	op->nrof += tmp->nrof;	1327	op->nrof += tmp->nrof;
1328	tmp->destroy (1);	1328	tmp->destroy ();
1329	}	1329	}
1330		1330
1331	CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */	1331	CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */
1332	CLEAR_FLAG (op, FLAG_INV_LOCKED);	1332	CLEAR_FLAG (op, FLAG_INV_LOCKED);
1333		1333
…		…
1340	{	1340	{
1341	LOG (llevError, "insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n");	1341	LOG (llevError, "insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n");
1342	abort ();	1342	abort ();
1343	}	1343	}
1344		1344
		1345	if (!originator->is_on_map ())
		1346	{
		1347	LOG (llevError, "insert_ob_in_map(%s) called with INS_BELOW_ORIGINATOR when originator '%s' not on map",
		1348	op->debug_desc (), originator->debug_desc ());
		1349	abort ();
		1350	}
		1351
1345	op->above = originator;	1352	op->above = originator;
1346	op->below = originator->below;	1353	op->below = originator->below;
1347
1348	if (op->below)
1349	op->below->above = op;
1350	else
1351	ms.bot = op;
1352
1353	/* since *below* originator, no need to update top */
1354	originator->below = op;	1354	originator->below = op;
		1355
		1356	*(op->below ? &op->below->above : &ms.bot) = op;
1355	}	1357	}
1356	else	1358	else
1357	{	1359	{
1358	object *top, *floor = NULL;	1360	object *floor = 0;
1359		1361	object *top = ms.top;
1360	top = ms.bot;
1361		1362
1362	/* If there are other objects, then */	1363	/* If there are other objects, then */
1363	if (top)	1364	if (top)
1364	{	1365	{
1365	object *last = 0;
1366
1367	/*	1366	/*
1368	* If there are multiple objects on this space, we do some trickier handling.	1367	* If there are multiple objects on this space, we do some trickier handling.
1369	* We've already dealt with merging if appropriate.	1368	* We've already dealt with merging if appropriate.
1370	* Generally, we want to put the new object on top. But if	1369	* Generally, we want to put the new object on top. But if
1371	* flag contains INS_ABOVE_FLOOR_ONLY, once we find the last	1370	* flag contains INS_ABOVE_FLOOR_ONLY, once we find the last
…		…
1374	* once we get to them. This reduces the need to traverse over all of	1373	* once we get to them. This reduces the need to traverse over all of
1375	* them when adding another one - this saves quite a bit of cpu time	1374	* them when adding another one - this saves quite a bit of cpu time
1376	* when lots of spells are cast in one area. Currently, it is presumed	1375	* when lots of spells are cast in one area. Currently, it is presumed
1377	* that flying non pickable objects are spell objects.	1376	* that flying non pickable objects are spell objects.
1378	*/	1377	*/
1379	for (top = ms.bot; top; top = top->above)	1378	for (object *tmp = ms.bot; tmp; tmp = tmp->above)
1380	{	1379	{
1381	if (QUERY_FLAG (top, FLAG_IS_FLOOR) || QUERY_FLAG (top, FLAG_OVERLAY_FLOOR))	1380	if (QUERY_FLAG (tmp, FLAG_IS_FLOOR) || QUERY_FLAG (tmp, FLAG_OVERLAY_FLOOR))
1382	floor = top;	1381	floor = tmp;
1383		1382
1384	if (QUERY_FLAG (top, FLAG_NO_PICK) && (top->move_type & (MOVE_FLY_LOW | MOVE_FLY_HIGH)) && !QUERY_FLAG (top, FLAG_IS_FLOOR))	1383	if (QUERY_FLAG (tmp, FLAG_NO_PICK) && (tmp->move_type & (MOVE_FLY_LOW | MOVE_FLY_HIGH)) && !QUERY_FLAG (tmp, FLAG_IS_FLOOR))
1385	{	1384	{
1386	/* We insert above top, so we want this object below this */	1385	/* We insert above top, so we want this object below this */
1387	top = top->below;	1386	top = tmp->below;
1388	break;	1387	break;
1389	}	1388	}
1390		1389
1391	last = top;	1390	top = tmp;
1392	}	1391	}
1393
1394	/* Don't want top to be NULL, so set it to the last valid object */
1395	top = last;
1396		1392
1397	/* We let update_position deal with figuring out what the space	1393	/* We let update_position deal with figuring out what the space
1398	* looks like instead of lots of conditions here.	1394	* looks like instead of lots of conditions here.
1399	* makes things faster, and effectively the same result.	1395	* makes things faster, and effectively the same result.
1400	*/	1396	*/
…		…
1407	*/	1403	*/
1408	if (!(flag & INS_ON_TOP)	1404	if (!(flag & INS_ON_TOP)
1409	&& ms.flags () & P_BLOCKSVIEW	1405	&& ms.flags () & P_BLOCKSVIEW
1410	&& (op->face && !faces [op->face].visibility))	1406	&& (op->face && !faces [op->face].visibility))
1411	{	1407	{
		1408	object *last;
		1409
1412	for (last = top; last != floor; last = last->below)	1410	for (last = top; last != floor; last = last->below)
1413	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))	1411	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))
1414	break;	1412	break;
1415		1413
1416	/* Check to see if we found the object that blocks view,	1414	/* Check to see if we found the object that blocks view,
…		…
1424	} /* If objects on this space */	1422	} /* If objects on this space */
1425		1423
1426	if (flag & INS_ABOVE_FLOOR_ONLY)	1424	if (flag & INS_ABOVE_FLOOR_ONLY)
1427	top = floor;	1425	top = floor;
1428		1426
1429	/* Top is the object that our object (op) is going to get inserted above.	1427	// insert object above top, or bottom-most if top = 0
1430	*/
1431
1432	/* First object on this space */
1433	if (!top)	1428	if (!top)
1434	{	1429	{
		1430	op->below = 0;
1435	op->above = ms.bot;	1431	op->above = ms.bot;
1436
1437	if (op->above)
1438	op->above->below = op;
1439
1440	op->below = 0;
1441	ms.bot = op;	1432	ms.bot = op;
		1433
		1434	*(op->above ? &op->above->below : &ms.top) = op;
1442	}	1435	}
1443	else	1436	else
1444	{ /* get inserted into the stack above top */	1437	{
1445	op->above = top->above;	1438	op->above = top->above;
1446
1447	if (op->above)
1448	op->above->below = op;	1439	top->above = op;
1449		1440
1450	op->below = top;	1441	op->below = top;
1451	top->above = op;	1442	*(op->above ? &op->above->below : &ms.top) = op;
1452	}	1443	}
		1444	}
1453		1445
1454	if (!op->above)	1446	if (op->is_player ())
1455	ms.top = op;
1456	} /* else not INS_BELOW_ORIGINATOR */
1457
1458	if (op->type == PLAYER)
1459	{	1447	{
1460	op->contr->do_los = 1;	1448	op->contr->do_los = 1;
1461	++op->map->players;	1449	++op->map->players;
1462	op->map->touch ();	1450	op->map->touch ();
1463	}	1451	}
1464		1452
1465	op->map->dirty = true;	1453	op->map->dirty = true;
1466		1454
1467	if (object *pl = ms.player ())	1455	if (object *pl = ms.player ())
1468	//TODO: the floorbox prev/next might need updating	1456	//TODO: the floorbox prev/next might need updating
1469	esrv_send_item (pl, op);	1457	//esrv_send_item (pl, op);
		1458	//TODO: update floorbox to preserve ordering
		1459	if (pl->contr->ns)
		1460	pl->contr->ns->floorbox_update ();
1470		1461
1471	/* If this object glows, it may affect lighting conditions that are	1462	/* If this object glows, it may affect lighting conditions that are
1472	* visible to others on this map. But update_all_los is really	1463	* visible to others on this map. But update_all_los is really
1473	* an inefficient way to do this, as it means los for all players	1464	* an inefficient way to do this, as it means los for all players
1474	* on the map will get recalculated. The players could very well	1465	* on the map will get recalculated. The players could very well
1475	* be far away from this change and not affected in any way -	1466	* be far away from this change and not affected in any way -
1476	* this should get redone to only look for players within range,	1467	* this should get redone to only look for players within range,
1477	* or just updating the P_UPTODATE for spaces within this area	1468	* or just updating the P_UPTODATE for spaces within this area
1478	* of effect may be sufficient.	1469	* of effect may be sufficient.
1479	*/	1470	*/
1480	if (op->map->darkness && (op->glow_radius != 0))	1471	if (op->affects_los ())
		1472	{
		1473	op->ms ().invalidate ();
1481	update_all_los (op->map, op->x, op->y);	1474	update_all_los (op->map, op->x, op->y);
		1475	}
1482		1476
1483	/* updates flags (blocked, alive, no magic, etc) for this map space */	1477	/* updates flags (blocked, alive, no magic, etc) for this map space */
1484	update_object (op, UP_OBJ_INSERT);	1478	update_object (op, UP_OBJ_INSERT);
1485		1479
1486	INVOKE_OBJECT (INSERT, op);	1480	INVOKE_OBJECT (INSERT, op);
…		…
1493	* blocked() and wall() work properly), and these flags are updated by	1487	* blocked() and wall() work properly), and these flags are updated by
1494	* update_object().	1488	* update_object().
1495	*/	1489	*/
1496		1490
1497	/* if this is not the head or flag has been passed, don't check walk on status */	1491	/* if this is not the head or flag has been passed, don't check walk on status */
1498	if (!(flag & INS_NO_WALK_ON) && op->head_ () == op)	1492	if (!(flag & INS_NO_WALK_ON) && op->is_head ())
1499	{	1493	{
1500	if (check_move_on (op, originator))	1494	if (check_move_on (op, originator))
1501	return 0;	1495	return 0;
1502		1496
1503	/* If we are a multi part object, lets work our way through the check	1497	/* If we are a multi part object, lets work our way through the check
…		…
1514	/* this function inserts an object in the map, but if it	1508	/* this function inserts an object in the map, but if it
1515	* finds an object of its own type, it'll remove that one first.	1509	* finds an object of its own type, it'll remove that one first.
1516	* op is the object to insert it under: supplies x and the map.	1510	* op is the object to insert it under: supplies x and the map.
1517	*/	1511	*/
1518	void	1512	void
1519	replace_insert_ob_in_map (const char *arch_string, object *op)	1513	replace_insert_ob_in_map (shstr_tmp archname, object *op)
1520	{	1514	{
1521	/* first search for itself and remove any old instances */	1515	/* first search for itself and remove any old instances */
1522		1516
1523	for (object *tmp = op->ms ().bot; tmp; tmp = tmp->above)	1517	for (object *tmp = op->ms ().bot; tmp; tmp = tmp->above)
1524	if (!strcmp (tmp->arch->archname, arch_string)) /* same archetype */	1518	if (tmp->arch->archname == archname) /* same archetype */
1525	tmp->destroy (1);	1519	tmp->destroy ();
1526		1520
1527	object *tmp = arch_to_object (archetype::find (arch_string));	1521	object *tmp = arch_to_object (archetype::find (archname));
1528		1522
1529	tmp->x = op->x;	1523	tmp->x = op->x;
1530	tmp->y = op->y;	1524	tmp->y = op->y;
1531		1525
1532	insert_ob_in_map (tmp, op->map, op, 0);	1526	insert_ob_in_map (tmp, op->map, op, 0);
…		…
1554	if (!nr)	1548	if (!nr)
1555	return true;	1549	return true;
1556		1550
1557	nr = min (nr, nrof);	1551	nr = min (nr, nrof);
1558		1552
		1553	if (nrof > nr)
		1554	{
1559	nrof -= nr;	1555	nrof -= nr;
1560
1561	if (nrof)
1562	{
1563	adjust_weight (env, -weight * nr); // carrying == 0	1556	adjust_weight (env, -weight * max (1, nr)); // carrying == 0
1564		1557
1565	if (object *pl = visible_to ())	1558	if (object *pl = visible_to ())
1566	esrv_update_item (UPD_NROF, pl, this);	1559	esrv_update_item (UPD_NROF, pl, this);
1567		1560
1568	return true;	1561	return true;
1569	}	1562	}
1570	else	1563	else
1571	{	1564	{
1572	destroy (1);	1565	destroy ();
1573	return false;	1566	return false;
1574	}	1567	}
1575	}	1568	}
1576		1569
1577	/*	1570	/*
…		…
1594	}	1587	}
1595	else	1588	else
1596	{	1589	{
1597	decrease (nr);	1590	decrease (nr);
1598		1591
1599	object *op = object_create_clone (this);	1592	object *op = deep_clone ();
1600	op->nrof = nr;	1593	op->nrof = nr;
1601	return op;	1594	return op;
1602	}	1595	}
1603	}	1596	}
1604		1597
…		…
1654	if (object *pl = tmp->visible_to ())	1647	if (object *pl = tmp->visible_to ())
1655	esrv_update_item (UPD_NROF, pl, tmp);	1648	esrv_update_item (UPD_NROF, pl, tmp);
1656		1649
1657	adjust_weight (this, op->total_weight ());	1650	adjust_weight (this, op->total_weight ());
1658		1651
1659	op->destroy (1);	1652	op->destroy ();
1660	op = tmp;	1653	op = tmp;
1661	goto inserted;	1654	goto inserted;
1662	}	1655	}
1663		1656
1664	op->owner = 0; // it's his/hers now. period.	1657	op->owner = 0; // it's his/hers now. period.
…		…
1682		1675
1683	adjust_weight (this, op->total_weight ());	1676	adjust_weight (this, op->total_weight ());
1684		1677
1685	inserted:	1678	inserted:
1686	/* reset the light list and los of the players on the map */	1679	/* reset the light list and los of the players on the map */
1687	if (op->glow_radius && map && map->darkness)	1680	if (op->glow_radius && is_on_map ())
		1681	{
		1682	update_stats ();
1688	update_all_los (map, x, y);	1683	update_all_los (map, x, y);
1689		1684	}
		1685	else if (is_player () && !flag [FLAG_NO_FIX_PLAYER])
1690	// if this is a player's inventory, update stats	1686	// if this is a player's inventory, update stats
1691	if (type == PLAYER && !flag [FLAG_NO_FIX_PLAYER])
1692	update_stats ();	1687	update_stats ();
1693		1688
1694	INVOKE_OBJECT (INSERT, this);	1689	INVOKE_OBJECT (INSERT, this);
1695		1690
1696	return op;	1691	return op;
…		…
1717	* on top.	1712	* on top.
1718	*/	1713	*/
1719	int	1714	int
1720	check_move_on (object *op, object *originator)	1715	check_move_on (object *op, object *originator)
1721	{	1716	{
		1717	if (QUERY_FLAG (op, FLAG_NO_APPLY))
		1718	return 0;
		1719
1722	object *tmp;	1720	object *tmp;
1723	maptile *m = op->map;	1721	maptile *m = op->map;
1724	int x = op->x, y = op->y;	1722	int x = op->x, y = op->y;
1725		1723
1726	MoveType move_on, move_slow, move_block;	1724	mapspace &ms = m->at (x, y);
1727		1725
1728	if (QUERY_FLAG (op, FLAG_NO_APPLY))	1726	ms.update ();
1729	return 0;
1730		1727
1731	move_on = GET_MAP_MOVE_ON (op->map, op->x, op->y);	1728	MoveType move_on = ms.move_on;
1732	move_slow = GET_MAP_MOVE_SLOW (op->map, op->x, op->y);	1729	MoveType move_slow = ms.move_slow;
1733	move_block = GET_MAP_MOVE_BLOCK (op->map, op->x, op->y);	1730	MoveType move_block = ms.move_block;
1734		1731
1735	/* if nothing on this space will slow op down or be applied,	1732	/* if nothing on this space will slow op down or be applied,
1736	* no need to do checking below. have to make sure move_type	1733	* no need to do checking below. have to make sure move_type
1737	* is set, as lots of objects don't have it set - we treat that	1734	* is set, as lots of objects don't have it set - we treat that
1738	* as walking.	1735	* as walking.
…		…
1749	return 0;	1746	return 0;
1750		1747
1751	/* The objects have to be checked from top to bottom.	1748	/* The objects have to be checked from top to bottom.
1752	* Hence, we first go to the top:	1749	* Hence, we first go to the top:
1753	*/	1750	*/
1754		1751	for (object *next, *tmp = ms.top; tmp; tmp = next)
1755	for (tmp = op->ms ().bot; tmp && tmp->above; tmp = tmp->above)
1756	{
1757	/* Trim the search when we find the first other spell effect
1758	* this helps performance so that if a space has 50 spell objects,
1759	* we don't need to check all of them.
1760	*/
1761	if ((tmp->move_type & MOVE_FLY_LOW) && QUERY_FLAG (tmp, FLAG_NO_PICK))
1762	break;
1763	}	1752	{
		1753	next = tmp->below;
1764		1754
1765	for (; tmp; tmp = tmp->below)
1766	{
1767	if (tmp == op)	1755	if (tmp == op)
1768	continue; /* Can't apply yourself */	1756	continue; /* Can't apply yourself */
1769		1757
1770	/* Check to see if one of the movement types should be slowed down.	1758	/* Check to see if one of the movement types should be slowed down.
1771	* Second check makes sure that the movement types not being slowed	1759	* Second check makes sure that the movement types not being slowed
…		…
1776	if (!QUERY_FLAG (op, FLAG_WIZPASS))	1764	if (!QUERY_FLAG (op, FLAG_WIZPASS))
1777	{	1765	{
1778	if ((!op->move_type && tmp->move_slow & MOVE_WALK) ||	1766	if ((!op->move_type && tmp->move_slow & MOVE_WALK) ||
1779	((op->move_type & tmp->move_slow) && (op->move_type & ~tmp->move_slow & ~tmp->move_block) == 0))	1767	((op->move_type & tmp->move_slow) && (op->move_type & ~tmp->move_slow & ~tmp->move_block) == 0))
1780	{	1768	{
1781
1782	float
1783	diff = tmp->move_slow_penalty * fabs (op->speed);	1769	float diff = tmp->move_slow_penalty * fabs (op->speed);
1784		1770
1785	if (op->type == PLAYER)	1771	if (op->is_player ())
1786	if ((QUERY_FLAG (tmp, FLAG_IS_HILLY) && find_skill_by_number (op, SK_CLIMBING)) ||	1772	if ((tmp->flag [FLAG_IS_HILLY ] && find_skill_by_number (op, SK_CLIMBING)) ||
1787	(QUERY_FLAG (tmp, FLAG_IS_WOODED) && find_skill_by_number (op, SK_WOODSMAN)))	1773	(tmp->flag [FLAG_IS_WOODED] && find_skill_by_number (op, SK_WOODSMAN)))
1788	diff /= 4.0;	1774	diff /= 4.0;
1789		1775
1790	op->speed_left -= diff;	1776	op->speed_left -= diff;
1791	}	1777	}
1792	}	1778	}
…		…
1825	LOG (llevError, "Present_arch called outside map.\n");	1811	LOG (llevError, "Present_arch called outside map.\n");
1826	return NULL;	1812	return NULL;
1827	}	1813	}
1828		1814
1829	for (object *tmp = m->at (x, y).bot; tmp; tmp = tmp->above)	1815	for (object *tmp = m->at (x, y).bot; tmp; tmp = tmp->above)
1830	if (tmp->arch == at)	1816	if (tmp->arch->archname == at->archname)
1831	return tmp;	1817	return tmp;
1832		1818
1833	return NULL;	1819	return NULL;
1834	}	1820	}
1835		1821
…		…
1899	* The first matching object is returned, or NULL if none.	1885	* The first matching object is returned, or NULL if none.
1900	*/	1886	*/
1901	object *	1887	object *
1902	present_arch_in_ob (const archetype *at, const object *op)	1888	present_arch_in_ob (const archetype *at, const object *op)
1903	{	1889	{
1904	for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below)	1890	for (object *tmp = op->inv; tmp; tmp = tmp->below)
1905	if (tmp->arch == at)	1891	if (tmp->arch->archname == at->archname)
1906	return tmp;	1892	return tmp;
1907		1893
1908	return NULL;	1894	return NULL;
1909	}	1895	}
1910		1896
…		…
1998	* head of the object should correspond for the entire object.	1984	* head of the object should correspond for the entire object.
1999	*/	1985	*/
2000	if (OB_TYPE_MOVE_BLOCK (ob, ms.move_block))	1986	if (OB_TYPE_MOVE_BLOCK (ob, ms.move_block))
2001	continue;	1987	continue;
2002		1988
2003	if (ob->blocked (m, pos.x, pos.y))	1989	if (ob->blocked (pos.m, pos.x, pos.y))
2004	continue;	1990	continue;
2005		1991
2006	altern [index++] = i;	1992	altern [index++] = i;
2007	}	1993	}
2008		1994
…		…
2076	* there is capable of.	2062	* there is capable of.
2077	*/	2063	*/
2078	int	2064	int
2079	find_dir (maptile *m, int x, int y, object *exclude)	2065	find_dir (maptile *m, int x, int y, object *exclude)
2080	{	2066	{
2081	int i, max = SIZEOFFREE, mflags;	2067	int max = SIZEOFFREE, mflags;
2082
2083	sint16 nx, ny;
2084	object *tmp;
2085	maptile *mp;
2086
2087	MoveType blocked, move_type;	2068	MoveType move_type;
2088		2069
2089	if (exclude && exclude->head_ () != exclude)	2070	if (exclude && exclude->head_ () != exclude)
2090	{	2071	{
2091	exclude = exclude->head;	2072	exclude = exclude->head;
2092	move_type = exclude->move_type;	2073	move_type = exclude->move_type;
…		…
2095	{	2076	{
2096	/* If we don't have anything, presume it can use all movement types. */	2077	/* If we don't have anything, presume it can use all movement types. */
2097	move_type = MOVE_ALL;	2078	move_type = MOVE_ALL;
2098	}	2079	}
2099		2080
2100	for (i = 1; i < max; i++)	2081	for (int i = 1; i < max; i++)
2101	{	2082	{
2102	mp = m;	2083	mapxy pos (m, x, y);
2103	nx = x + freearr_x[i];	2084	pos.move (i);
2104	ny = y + freearr_y[i];
2105		2085
2106	mflags = get_map_flags (m, &mp, nx, ny, &nx, &ny);	2086	if (!pos.normalise ())
2107
2108	if (mflags & P_OUT_OF_MAP)
2109	max = maxfree[i];	2087	max = maxfree[i];
2110	else	2088	else
2111	{	2089	{
2112	mapspace &ms = mp->at (nx, ny);	2090	mapspace &ms = *pos;
2113		2091
2114	blocked = ms.move_block;
2115
2116	if ((move_type & blocked) == move_type)	2092	if ((move_type & ms.move_block) == move_type)
2117	max = maxfree[i];	2093	max = maxfree [i];
2118	else if (mflags & P_IS_ALIVE)	2094	else if (ms.flags () & P_IS_ALIVE)
2119	{	2095	{
2120	for (tmp = ms.bot; tmp; tmp = tmp->above)	2096	for (object *tmp = ms.bot; tmp; tmp = tmp->above)
2121	if ((tmp->flag [FLAG_MONSTER] || tmp->type == PLAYER)	2097	if ((tmp->flag [FLAG_MONSTER] || tmp->is_player ())
2122	&& (tmp != exclude || (tmp->head_ () != tmp && tmp->head_ () != exclude)))	2098	&& (tmp != exclude || (tmp->head_ () != tmp && tmp->head_ () != exclude)))
2123	break;
2124
2125	if (tmp)
2126	return freedir[i];	2099	return freedir [i];
2127	}	2100	}
2128	}	2101	}
2129	}	2102	}
2130		2103
2131	return 0;	2104	return 0;
…		…
2206	* This basically means that if direction is 15, then it could either go	2179	* This basically means that if direction is 15, then it could either go
2207	* direction 4, 14, or 16 to get back to where we are.	2180	* direction 4, 14, or 16 to get back to where we are.
2208	* Moved from spell_util.c to object.c with the other related direction	2181	* Moved from spell_util.c to object.c with the other related direction
2209	* functions.	2182	* functions.
2210	*/	2183	*/
2211	int reduction_dir[SIZEOFFREE][3] = {	2184	const int reduction_dir[SIZEOFFREE][3] = {
2212	{0, 0, 0}, /* 0 */	2185	{0, 0, 0}, /* 0 */
2213	{0, 0, 0}, /* 1 */	2186	{0, 0, 0}, /* 1 */
2214	{0, 0, 0}, /* 2 */	2187	{0, 0, 0}, /* 2 */
2215	{0, 0, 0}, /* 3 */	2188	{0, 0, 0}, /* 3 */
2216	{0, 0, 0}, /* 4 */	2189	{0, 0, 0}, /* 4 */
…		…
2312	int	2285	int
2313	can_pick (const object *who, const object *item)	2286	can_pick (const object *who, const object *item)
2314	{	2287	{
2315	return /*QUERY_FLAG(who,FLAG_WIZ)|| */	2288	return /*QUERY_FLAG(who,FLAG_WIZ)|| */
2316	(item->weight > 0 && !QUERY_FLAG (item, FLAG_NO_PICK) &&	2289	(item->weight > 0 && !QUERY_FLAG (item, FLAG_NO_PICK) &&
2317	!QUERY_FLAG (item, FLAG_ALIVE) && !item->invisible && (who->type == PLAYER || item->weight < who->weight / 3));	2290	!QUERY_FLAG (item, FLAG_ALIVE) && !item->invisible && (who->is_player () || item->weight < who->weight / 3));
2318	}	2291	}
2319		2292
2320	/*	2293	/*
2321	* create clone from object to another	2294	* create clone from object to another
2322	*/	2295	*/
2323	object *	2296	object *
2324	object_create_clone (object *asrc)	2297	object::deep_clone ()
2325	{	2298	{
2326	object *dst = 0, *tmp, *src, *prev, *item;	2299	assert (("deep_clone called on non-head object", is_head ()));
2327		2300
2328	if (!asrc)	2301	object *dst = clone ();
2329	return 0;
2330		2302
2331	src = asrc->head_ ();	2303	object *prev = dst;
2332
2333	prev = 0;
2334	for (object *part = src; part; part = part->more)	2304	for (object *part = this->more; part; part = part->more)
2335	{	2305	{
2336	tmp = part->clone ();	2306	object *tmp = part->clone ();
2337	tmp->x -= src->x;
2338	tmp->y -= src->y;
2339
2340	if (!part->head)
2341	{
2342	dst = tmp;
2343	tmp->head = 0;
2344	}
2345	else
2346	tmp->head = dst;	2307	tmp->head = dst;
2347
2348	tmp->more = 0;
2349
2350	if (prev)
2351	prev->more = tmp;	2308	prev->more = tmp;
2352
2353	prev = tmp;	2309	prev = tmp;
2354	}	2310	}
2355		2311
2356	for (item = src->inv; item; item = item->below)	2312	for (object *item = inv; item; item = item->below)
2357	insert_ob_in_ob (object_create_clone (item), dst);	2313	insert_ob_in_ob (item->deep_clone (), dst);
2358		2314
2359	return dst;	2315	return dst;
2360	}	2316	}
2361		2317
2362	/* This returns the first object in who's inventory that	2318	/* This returns the first object in who's inventory that
…		…
2371	return tmp;	2327	return tmp;
2372		2328
2373	return 0;	2329	return 0;
2374	}	2330	}
2375		2331
2376	/* If ob has a field named key, return the link from the list,	2332	shstr_tmp
2377	* otherwise return NULL.	2333	object::kv_get (shstr_tmp key) const
2378	*
2379	* key must be a passed in shared string - otherwise, this won't
2380	* do the desired thing.
2381	*/
2382	key_value *
2383	get_ob_key_link (const object *ob, const char *key)
2384	{	2334	{
2385	for (key_value *link = ob->key_values; link; link = link->next)	2335	for (key_value *kv = key_values; kv; kv = kv->next)
2386	if (link->key == key)	2336	if (kv->key == key)
2387	return link;
2388
2389	return 0;
2390	}
2391
2392	/*
2393	* Returns the value of op has an extra_field for key, or NULL.
2394	*
2395	* The argument doesn't need to be a shared string.
2396	*
2397	* The returned string is shared.
2398	*/
2399	const char *
2400	get_ob_key_value (const object *op, const char *const key)
2401	{
2402	key_value *link;
2403	shstr_cmp canonical_key (key);
2404
2405	if (!canonical_key)
2406	{
2407	/* 1. There being a field named key on any object
2408	* implies there'd be a shared string to find.
2409	* 2. Since there isn't, no object has this field.
2410	* 3. Therefore, *this* object doesn't have this field.
2411	*/
2412	return 0;
2413	}
2414
2415	/* This is copied from get_ob_key_link() above -
2416	* only 4 lines, and saves the function call overhead.
2417	*/
2418	for (link = op->key_values; link; link = link->next)
2419	if (link->key == canonical_key)
2420	return link->value;	2337	return kv->value;
2421		2338
2422	return 0;	2339	return shstr ();
2423	}	2340	}
2424		2341
2425	/*	2342	void
2426	* Updates the canonical_key in op to value.	2343	object::kv_set (shstr_tmp key, shstr_tmp value)
2427	*
2428	* canonical_key is a shared string (value doesn't have to be).
2429	*
2430	* Unless add_key is TRUE, it won't add fields, only change the value of existing
2431	* keys.
2432	*
2433	* Returns TRUE on success.
2434	*/
2435	int
2436	set_ob_key_value_s (object *op, const shstr & canonical_key, const char *value, int add_key)
2437	{	2344	{
2438	key_value *field = NULL, *last = NULL;	2345	for (key_value *kv = key_values; kv; kv = kv->next)
2439		2346	if (kv->key == key)
2440	for (field = op->key_values; field != NULL; field = field->next)
2441	{
2442	if (field->key != canonical_key)
2443	{	2347	{
2444	last = field;	2348	kv->value = value;
2445	continue;	2349	return;
2446	}	2350	}
2447		2351
2448	if (value)	2352	key_value *kv = new key_value;
2449	field->value = value;	2353
2450	else	2354	kv->next = key_values;
		2355	kv->key = key;
		2356	kv->value = value;
		2357
		2358	key_values = kv;
		2359	}
		2360
		2361	void
		2362	object::kv_del (shstr_tmp key)
		2363	{
		2364	for (key_value **kvp = &key_values; *kvp; kvp = &(*kvp)->next)
		2365	if ((*kvp)->key == key)
2451	{	2366	{
2452	/* Basically, if the archetype has this key set,	2367	key_value *kv = *kvp;
2453	* we need to store the null value so when we save	2368	*kvp = (*kvp)->next;
2454	* it, we save the empty value so that when we load,	2369	delete kv;
2455	* we get this value back again.	2370	return;
2456	*/
2457	if (get_ob_key_link (op->arch, canonical_key))
2458	field->value = 0;
2459	else
2460	{
2461	if (last)
2462	last->next = field->next;
2463	else
2464	op->key_values = field->next;
2465
2466	delete field;
2467	}
2468	}	2371	}
2469	return TRUE;
2470	}
2471	/* IF we get here, key doesn't exist */
2472
2473	/* No field, we'll have to add it. */
2474
2475	if (!add_key)
2476	return FALSE;
2477
2478	/* There isn't any good reason to store a null
2479	* value in the key/value list. If the archetype has
2480	* this key, then we should also have it, so shouldn't
2481	* be here. If user wants to store empty strings,
2482	* should pass in ""
2483	*/
2484	if (value == NULL)
2485	return TRUE;
2486
2487	field = new key_value;
2488
2489	field->key = canonical_key;
2490	field->value = value;
2491	/* Usual prepend-addition. */
2492	field->next = op->key_values;
2493	op->key_values = field;
2494
2495	return TRUE;
2496	}
2497
2498	/*
2499	* Updates the key in op to value.
2500	*
2501	* If add_key is FALSE, this will only update existing keys,
2502	* and not add new ones.
2503	* In general, should be little reason FALSE is ever passed in for add_key
2504	*
2505	* Returns TRUE on success.
2506	*/
2507	int
2508	set_ob_key_value (object *op, const char *key, const char *value, int add_key)
2509	{
2510	shstr key_ (key);
2511
2512	return set_ob_key_value_s (op, key_, value, add_key);
2513	}	2372	}
2514		2373
2515	object::depth_iterator::depth_iterator (object *container)	2374	object::depth_iterator::depth_iterator (object *container)
2516	: iterator_base (container)	2375	: iterator_base (container)
2517	{	2376	{
…		…
2567	{	2426	{
2568	char flagdesc[512];	2427	char flagdesc[512];
2569	char info2[256 * 4];	2428	char info2[256 * 4];
2570	char *p = info;	2429	char *p = info;
2571		2430
2572	p += snprintf (p, 512, "{cnt:%d,uuid:%s,name:\"%s\"%s%s,flags:[%s],type:%d}",	2431	p += snprintf (p, 512, "{cnt:%d,uuid:%s,name:\"%s\"%s%s%s,flags:[%s],type:%d}",
2573	count,	2432	count,
2574	uuid.c_str (),	2433	uuid.c_str (),
2575	&name,	2434	&name,
2576	title ? "\",title:\"" : "",	2435	title ? ",title:\"" : "",
2577	title ? (const char *)title : "",	2436	title ? (const char *)title : "",
		2437	title ? "\"" : "",
2578	flag_desc (flagdesc, 512), type);	2438	flag_desc (flagdesc, 512), type);
2579		2439
2580	if (!flag[FLAG_REMOVED] && env)	2440	if (!flag[FLAG_REMOVED] && env)
2581	p += snprintf (p, 256, "(in %s)", env->debug_desc (info2));	2441	p += snprintf (p, 256, "(in %s)", env->debug_desc (info2));
2582		2442
…		…
2615	object::open_container (object *new_container)	2475	object::open_container (object *new_container)
2616	{	2476	{
2617	if (container == new_container)	2477	if (container == new_container)
2618	return;	2478	return;
2619		2479
2620	if (object *old_container = container)	2480	object *old_container = container;
		2481
		2482	if (old_container)
2621	{	2483	{
2622	if (INVOKE_OBJECT (CLOSE, old_container, ARG_OBJECT (this)))	2484	if (INVOKE_OBJECT (CLOSE, old_container, ARG_OBJECT (this)))
2623	return;	2485	return;
2624		2486
2625	#if 0	2487	#if 0
…		…
2627	if (object *closer = old_container->inv)	2489	if (object *closer = old_container->inv)
2628	if (closer->type == CLOSE_CON)	2490	if (closer->type == CLOSE_CON)
2629	closer->destroy ();	2491	closer->destroy ();
2630	#endif	2492	#endif
2631		2493
		2494	// make sure the container is available
		2495	esrv_send_item (this, old_container);
		2496
2632	old_container->flag [FLAG_APPLIED] = false;	2497	old_container->flag [FLAG_APPLIED] = false;
2633	container = 0;	2498	container = 0;
2634		2499
		2500	// client needs item update to make it work, client bug requires this to be separate
2635	esrv_update_item (UPD_FLAGS, this, old_container);	2501	esrv_update_item (UPD_FLAGS, this, old_container);
		2502
2636	new_draw_info_format (NDI_UNIQUE, 0, this, "You close %s.", query_name (old_container));	2503	new_draw_info_format (NDI_UNIQUE, 0, this, "You close %s.", old_container->query_name ());
2637	play_sound (sound_find ("chest_close"));	2504	play_sound (sound_find ("chest_close"));
2638	}	2505	}
2639		2506
2640	if (new_container)	2507	if (new_container)
2641	{	2508	{
…		…
2651	closer->flag [FLAG_NO_MAP_SAVE] = 1;	2518	closer->flag [FLAG_NO_MAP_SAVE] = 1;
2652	new_container->insert (closer);	2519	new_container->insert (closer);
2653	}	2520	}
2654	#endif	2521	#endif
2655		2522
2656	new_draw_info_format (NDI_UNIQUE, 0, this, "You open %s.", query_name (new_container));	2523	new_draw_info_format (NDI_UNIQUE, 0, this, "You open %s.", new_container->query_name ());
		2524
		2525	// make sure the container is available, client bug requires this to be separate
		2526	esrv_send_item (this, new_container);
2657		2527
2658	new_container->flag [FLAG_APPLIED] = true;	2528	new_container->flag [FLAG_APPLIED] = true;
2659	container = new_container;	2529	container = new_container;
2660		2530
		2531	// client needs flag change
2661	esrv_update_item (UPD_FLAGS, this, new_container);	2532	esrv_update_item (UPD_FLAGS, this, new_container);
2662	esrv_send_inventory (this, new_container);	2533	esrv_send_inventory (this, new_container);
2663	play_sound (sound_find ("chest_open"));	2534	play_sound (sound_find ("chest_open"));
2664	}	2535	}
		2536	// else if (!old_container->env && contr && contr->ns)
		2537	// contr->ns->floorbox_reset ();
2665	}	2538	}
2666		2539
2667	object *	2540	object *
2668	object::force_find (const shstr name)	2541	object::force_find (shstr_tmp name)
2669	{	2542	{
2670	/* cycle through his inventory to look for the MARK we want to	2543	/* cycle through his inventory to look for the MARK we want to
2671	* place	2544	* place
2672	*/	2545	*/
2673	for (object *tmp = inv; tmp; tmp = tmp->below)	2546	for (object *tmp = inv; tmp; tmp = tmp->below)
…		…
2676		2549
2677	return 0;	2550	return 0;
2678	}	2551	}
2679		2552
2680	void	2553	void
		2554	object::force_set_timer (int duration)
		2555	{
		2556	this->duration = 1;
		2557	this->speed_left = -1.f;
		2558
		2559	this->set_speed (duration ? 1.f / duration : 0.f);
		2560	}
		2561
		2562	object *
2681	object::force_add (const shstr name, int duration)	2563	object::force_add (shstr_tmp name, int duration)
2682	{	2564	{
2683	if (object *force = force_find (name))	2565	if (object *force = force_find (name))
2684	force->destroy ();	2566	force->destroy ();
2685		2567
2686	object *force = get_archetype (FORCE_NAME);	2568	object *force = get_archetype (FORCE_NAME);
2687		2569
2688	force->slaying = name;	2570	force->slaying = name;
2689	force->stats.food = 1;
2690	force->speed_left = -1.f;
2691		2571
2692	force->set_speed (duration ? 1.f / duration : 0.f);	2572	force->force_set_timer (duration);
2693	force->flag [FLAG_IS_USED_UP] = true;	2573
2694	force->flag [FLAG_APPLIED] = true;	2574	force->flag [FLAG_APPLIED] = true;
2695		2575
2696	insert (force);	2576	return insert (force);
2697	}	2577	}
2698		2578
2699	void	2579	void
2700	object::play_sound (faceidx sound)	2580	object::play_sound (faceidx sound) const
2701	{	2581	{
2702	if (!sound)	2582	if (!sound)
2703	return;	2583	return;
2704		2584
2705	if (flag [FLAG_REMOVED])	2585	if (is_on_map ())
		2586	map->play_sound (sound, x, y);
		2587	else if (object *pl = in_player ())
		2588	pl->contr->play_sound (sound);
		2589	}
		2590
		2591	void
		2592	object::say_msg (const char *msg) const
		2593	{
		2594	if (is_on_map ())
		2595	map->say_msg (msg, x, y);
		2596	else if (object *pl = in_player ())
		2597	pl->contr->play_sound (sound);
		2598	}
		2599
		2600	void
		2601	object::make_noise ()
		2602	{
		2603	// we do not model noise in the map, so instead put
		2604	// a temporary light into the noise source
		2605	// could use the map instead, but that's less reliable for our
		2606	// goal, which is to make invisibility a bit harder to exploit
		2607
		2608	// currently only works sensibly for players
		2609	if (!is_player ())
2706	return;	2610	return;
2707		2611
2708	if (env)	2612	// find old force, or create new one
2709	{	2613	object *force = force_find (shstr_noise_force);
2710	if (object *pl = in_player ())	2614
2711	pl->contr->play_sound (sound);	2615	if (force)
2712	}	2616	force->speed_left = -1.f; // patch old speed up
2713	else	2617	else
2714	map->play_sound (sound, x, y);	2618	{
2715	}	2619	force = archetype::get (shstr_noise_force);
2716		2620
		2621	force->slaying = shstr_noise_force;
		2622	force->stats.food = 1;
		2623	force->speed_left = -1.f;
		2624
		2625	force->set_speed (1.f / 4.f);
		2626	force->flag [FLAG_IS_USED_UP] = true;
		2627	force->flag [FLAG_APPLIED] = true;
		2628
		2629	insert (force);
		2630	}
		2631	}
		2632

Diff Legend

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