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.56 by root, Sun Dec 3 02:04:57 2006 UTC vs.
Revision 1.86 by root, Tue Dec 26 08:17:58 2006 UTC

…		…
29	#include <sys/types.h>	29	#include <sys/types.h>
30	#include <sys/uio.h>	30	#include <sys/uio.h>
31	#include <object.h>	31	#include <object.h>
32	#include <funcpoint.h>	32	#include <funcpoint.h>
33	#include <loader.h>	33	#include <loader.h>
		34
		35	#include <bitset>
34		36
35	int nrofallocobjects = 0;	37	int nrofallocobjects = 0;
36	static UUID uuid;	38	static UUID uuid;
37	const uint64 UUID_SKIP = 1<<19;	39	const uint64 UUID_SKIP = 1<<19;
38		40
…		…
104	_exit (1);	106	_exit (1);
105	}	107	}
106		108
107	uuid.seq = uid;	109	uuid.seq = uid;
108	write_uuid ();	110	write_uuid ();
109	LOG (llevDebug, "read UID: %lld\n", uid);	111	LOG (llevDebug, "read UID: %" PRId64 "\n", uid);
110	fclose (fp);	112	fclose (fp);
111	}	113	}
112		114
113	UUID	115	UUID
114	gen_uuid ()	116	gen_uuid ()
…		…
139	* objects with lists are rare, and lists stay short. If not, use a	141	* objects with lists are rare, and lists stay short. If not, use a
140	* different structure or at least keep the lists sorted...	142	* different structure or at least keep the lists sorted...
141	*/	143	*/
142		144
143	/* For each field in wants, */	145	/* For each field in wants, */
144	for (wants_field = wants->key_values; wants_field != NULL; wants_field = wants_field->next)	146	for (wants_field = wants->key_values; wants_field; wants_field = wants_field->next)
145	{	147	{
146	key_value *has_field;	148	key_value *has_field;
147		149
148	/* Look for a field in has with the same key. */	150	/* Look for a field in has with the same key. */
149	has_field = get_ob_key_link (has, wants_field->key);	151	has_field = get_ob_key_link (has, wants_field->key);
…		…
183	*	185	*
184	* Note that this function appears a lot longer than the macro it	186	* Note that this function appears a lot longer than the macro it
185	* replaces - this is mostly for clarity - a decent compiler should hopefully	187	* replaces - this is mostly for clarity - a decent compiler should hopefully
186	* reduce this to the same efficiency.	188	* reduce this to the same efficiency.
187	*	189	*
188	* Check nrof variable *before* calling CAN_MERGE()	190	* Check nrof variable *before* calling can_merge()
189	*	191	*
190	* Improvements made with merge: Better checking on potion, and also	192	* Improvements made with merge: Better checking on potion, and also
191	* check weight	193	* check weight
192	*/	194	*/
193		195
194	bool object::can_merge (object *ob1, object *ob2)	196	bool object::can_merge_slow (object *ob1, object *ob2)
195	{	197	{
196	/* A couple quicksanity checks */	198	/* A couple quicksanity checks */
197	if ((ob1 == ob2) || (ob1->type != ob2->type))	199	if (ob1 == ob2
		200	|| ob1->type != ob2->type
		201	|| ob1->speed != ob2->speed
		202	|| ob1->value != ob2->value
		203	|| ob1->name != ob2->name)
198	return 0;	204	return 0;
199		205
200	if (ob1->speed != ob2->speed)	206	//TODO: this ain't working well, use nicer and correct overflow check
201	return 0;
202
203	/* Do not merge objects if nrof would overflow. We use 1UL<<31 since that	207	/* Do not merge objects if nrof would overflow. We use 1UL<<31 since that
204	* value could not be stored in a sint32 (which unfortunately sometimes is	208	* value could not be stored in a sint32 (which unfortunately sometimes is
205	* used to store nrof).	209	* used to store nrof).
206	*/	210	*/
207	if (ob1->nrof + ob2->nrof >= 1UL << 31)	211	if (ob1->nrof + ob2->nrof >= 1UL << 31)
…		…
217	SET_FLAG (ob1, FLAG_BEEN_APPLIED);	221	SET_FLAG (ob1, FLAG_BEEN_APPLIED);
218		222
219	if (QUERY_FLAG (ob2, FLAG_IDENTIFIED))	223	if (QUERY_FLAG (ob2, FLAG_IDENTIFIED))
220	SET_FLAG (ob2, FLAG_BEEN_APPLIED);	224	SET_FLAG (ob2, FLAG_BEEN_APPLIED);
221		225
222		226	if ((ob1->flag ^ ob2->flag).reset (FLAG_INV_LOCKED).reset (FLAG_CLIENT_SENT).any ()
223	/* the 0x400000 on flags2 is FLAG_INV_LOCK. I don't think something	227	|| ob1->arch != ob2->arch
224	* being locked in inventory should prevent merging.
225	* 0x4 in flags3 is CLIENT_SENT
226	*/
227	if ((ob1->arch != ob2->arch) ||
228	(ob1->flags[0] != ob2->flags[0]) ||
229	(ob1->flags[1] != ob2->flags[1]) ||
230	((ob1->flags[2] & ~0x400000) != (ob2->flags[2] & ~0x400000)) ||
231	((ob1->flags[3] & ~0x4) != (ob2->flags[3] & ~0x4)) ||
232	(ob1->name != ob2->name) ||	228	|| ob1->name != ob2->name
233	(ob1->title != ob2->title) ||	229	|| ob1->title != ob2->title
234	(ob1->msg != ob2->msg) ||	230	|| ob1->msg != ob2->msg
235	(ob1->weight != ob2->weight) ||	231	|| ob1->weight != ob2->weight
236	(memcmp (&ob1->resist, &ob2->resist, sizeof (ob1->resist)) != 0) ||	232	|| memcmp (&ob1->resist, &ob2->resist, sizeof (ob1->resist))
237	(memcmp (&ob1->stats, &ob2->stats, sizeof (ob1->stats)) != 0) ||	233	|| memcmp (&ob1->stats , &ob2->stats , sizeof (ob1->stats) )
238	(ob1->attacktype != ob2->attacktype) ||	234	|| ob1->attacktype != ob2->attacktype
239	(ob1->magic != ob2->magic) ||	235	|| ob1->magic != ob2->magic
240	(ob1->slaying != ob2->slaying) ||	236	|| ob1->slaying != ob2->slaying
241	(ob1->skill != ob2->skill) ||	237	|| ob1->skill != ob2->skill
242	(ob1->value != ob2->value) ||	238	|| ob1->value != ob2->value
243	(ob1->animation_id != ob2->animation_id) ||	239	|| ob1->animation_id != ob2->animation_id
244	(ob1->client_type != ob2->client_type) ||	240	|| ob1->client_type != ob2->client_type
245	(ob1->materialname != ob2->materialname) ||	241	|| ob1->materialname != ob2->materialname
246	(ob1->lore != ob2->lore) ||	242	|| ob1->lore != ob2->lore
247	(ob1->subtype != ob2->subtype) ||	243	|| ob1->subtype != ob2->subtype
248	(ob1->move_type != ob2->move_type) ||	244	|| ob1->move_type != ob2->move_type
249	(ob1->move_block != ob2->move_block) ||	245	|| ob1->move_block != ob2->move_block
250	(ob1->move_allow != ob2->move_allow) ||	246	|| ob1->move_allow != ob2->move_allow
251	(ob1->move_on != ob2->move_on) ||	247	|| ob1->move_on != ob2->move_on
252	(ob1->move_off != ob2->move_off) || (ob1->move_slow != ob2->move_slow) || (ob1->move_slow_penalty != ob2->move_slow_penalty))	248	|| ob1->move_off != ob2->move_off
		249	|| ob1->move_slow != ob2->move_slow
		250	|| ob1->move_slow_penalty != ob2->move_slow_penalty)
253	return 0;	251	return 0;
254		252
255	/* This is really a spellbook check - really, we should	253	/* This is really a spellbook check - really, we should
256	* check all objects in the inventory.	254	* check all objects in the inventory.
257	*/	255	*/
…		…
260	/* if one object has inventory but the other doesn't, not equiv */	258	/* if one object has inventory but the other doesn't, not equiv */
261	if ((ob1->inv && !ob2->inv) || (ob2->inv && !ob1->inv))	259	if ((ob1->inv && !ob2->inv) || (ob2->inv && !ob1->inv))
262	return 0;	260	return 0;
263		261
264	/* Now check to see if the two inventory objects could merge */	262	/* Now check to see if the two inventory objects could merge */
265	if (!CAN_MERGE (ob1->inv, ob2->inv))	263	if (!object::can_merge (ob1->inv, ob2->inv))
266	return 0;	264	return 0;
267		265
268	/* inventory ok - still need to check rest of this object to see	266	/* inventory ok - still need to check rest of this object to see
269	* if it is valid.	267	* if it is valid.
270	*/	268	*/
…		…
354	op = op->env;	352	op = op->env;
355	return op;	353	return op;
356	}	354	}
357		355
358	/*	356	/*
359	* Eneq(@csd.uu.se): Since we can have items buried in a character we need
360	* a better check. We basically keeping traversing up until we can't
361	* or find a player.
362	*/
363
364	object *
365	is_player_inv (object *op)
366	{
367	for (; op != NULL && op->type != PLAYER; op = op->env)
368	if (op->env == op)
369	op->env = NULL;
370	return op;
371	}
372
373	/*
374	* Used by: Crossedit: dump. Server DM commands: dumpbelow, dump.	357	* Used by: Crossedit: dump. Server DM commands: dumpbelow, dump.
375	* Some error messages.	358	* Some error messages.
376	* The result of the dump is stored in the static global errmsg array.	359	* The result of the dump is stored in the static global errmsg array.
377	*/	360	*/
378		361
…		…
381	{	364	{
382	if (!op)	365	if (!op)
383	return strdup ("[NULLOBJ]");	366	return strdup ("[NULLOBJ]");
384		367
385	object_freezer freezer;	368	object_freezer freezer;
386	save_object (freezer, op, 3);	369	save_object (freezer, op, 1);
387	return freezer.as_string ();	370	return freezer.as_string ();
388	}	371	}
389		372
390	/*	373	/*
391	* get_nearest_part(multi-object, object 2) returns the part of the	374	* get_nearest_part(multi-object, object 2) returns the part of the
…		…
484	}	467	}
485		468
486	op->key_values = 0;	469	op->key_values = 0;
487	}	470	}
488		471
489	void object::clear ()
490	{
491	attachable_base::clear ();
492
493	free_key_values (this);
494
495	owner = 0;
496	name = 0;
497	name_pl = 0;
498	title = 0;
499	race = 0;
500	slaying = 0;
501	skill = 0;
502	msg = 0;
503	lore = 0;
504	custom_name = 0;
505	materialname = 0;
506	contr = 0;
507	below = 0;
508	above = 0;
509	inv = 0;
510	container = 0;
511	env = 0;
512	more = 0;
513	head = 0;
514	map = 0;
515	active_next = 0;
516	active_prev = 0;
517
518	memset (static_cast<object_pod *>(this), 0, sizeof (object_pod));
519
520	SET_FLAG (this, FLAG_REMOVED);
521
522	/* What is not cleared is next, prev, and count */
523
524	expmul = 1.0;
525	face = blank_face;
526
527	if (settings.casting_time)
528	casting_time = -1;
529	}
530
531	void object::clone (object *destination)
532	{
533	*(object_copy *)destination = *this;
534	*(object_pod *)destination = *this;
535
536	if (self || cb)
537	INVOKE_OBJECT (CLONE, this, ARG_OBJECT (destination));
538	}
539
540	/*	472	/*
541	* copy object first frees everything allocated by the second object,	473	* copy_to first frees everything allocated by the dst object,
542	* and then copies the contends of the first object into the second	474	* and then copies the contents of itself into the second
543	* object, allocating what needs to be allocated. Basically, any	475	* object, allocating what needs to be allocated. Basically, any
544	* data that is malloc'd needs to be re-malloc/copied. Otherwise,	476	* data that is malloc'd needs to be re-malloc/copied. Otherwise,
545	* if the first object is freed, the pointers in the new object	477	* if the first object is freed, the pointers in the new object
546	* will point at garbage.	478	* will point at garbage.
547	*/	479	*/
548	void	480	void
549	copy_object (object *op2, object *op)	481	object::copy_to (object *dst)
550	{	482	{
551	bool is_freed = QUERY_FLAG (op, FLAG_FREED);	483	bool is_freed = QUERY_FLAG (dst, FLAG_FREED);
552	bool is_removed = QUERY_FLAG (op, FLAG_REMOVED);	484	bool is_removed = QUERY_FLAG (dst, FLAG_REMOVED);
553		485
554	op2->clone (op);	486	*(object_copy *)dst = *this;
555		487
556	if (is_freed)	488	if (is_freed)
557	SET_FLAG (op, FLAG_FREED);	489	SET_FLAG (dst, FLAG_FREED);
		490
558	if (is_removed)	491	if (is_removed)
559	SET_FLAG (op, FLAG_REMOVED);	492	SET_FLAG (dst, FLAG_REMOVED);
560		493
561	if (op2->speed < 0)	494	if (speed < 0)
562	op->speed_left = op2->speed_left - RANDOM () % 200 / 100.0;	495	dst->speed_left = speed_left - RANDOM () % 200 / 100.0;
563		496
564	/* Copy over key_values, if any. */	497	/* Copy over key_values, if any. */
565	if (op2->key_values)	498	if (key_values)
566	{	499	{
567	key_value *tail = 0;	500	key_value *tail = 0;
568	key_value *i;	501	key_value *i;
569		502
570	op->key_values = 0;	503	dst->key_values = 0;
571		504
572	for (i = op2->key_values; i; i = i->next)	505	for (i = key_values; i; i = i->next)
573	{	506	{
574	key_value *new_link = new key_value;	507	key_value *new_link = new key_value;
575		508
576	new_link->next = 0;	509	new_link->next = 0;
577	new_link->key = i->key;	510	new_link->key = i->key;
578	new_link->value = i->value;	511	new_link->value = i->value;
579		512
580	/* Try and be clever here, too. */	513	/* Try and be clever here, too. */
581	if (!op->key_values)	514	if (!dst->key_values)
582	{	515	{
583	op->key_values = new_link;	516	dst->key_values = new_link;
584	tail = new_link;	517	tail = new_link;
585	}	518	}
586	else	519	else
587	{	520	{
588	tail->next = new_link;	521	tail->next = new_link;
589	tail = new_link;	522	tail = new_link;
590	}	523	}
591	}	524	}
592	}	525	}
593		526
594	update_ob_speed (op);	527	update_ob_speed (dst);
		528	}
		529
		530	object *
		531	object::clone ()
		532	{
		533	object *neu = create ();
		534	copy_to (neu);
		535	return neu;
595	}	536	}
596		537
597	/*	538	/*
598	* If an object with the IS_TURNABLE() flag needs to be turned due	539	* If an object with the IS_TURNABLE() flag needs to be turned due
599	* to the closest player being on the other side, this function can	540	* to the closest player being on the other side, this function can
…		…
620	extern int arch_init;	561	extern int arch_init;
621		562
622	/* No reason putting the archetypes objects on the speed list,	563	/* No reason putting the archetypes objects on the speed list,
623	* since they never really need to be updated.	564	* since they never really need to be updated.
624	*/	565	*/
625
626	if (QUERY_FLAG (op, FLAG_FREED) && op->speed)	566	if (QUERY_FLAG (op, FLAG_FREED) && op->speed)
627	{	567	{
628	LOG (llevError, "Object %s is freed but has speed.\n", &op->name);	568	LOG (llevError, "Object %s is freed but has speed.\n", &op->name);
629	#ifdef MANY_CORES
630	abort ();
631	#else
632	op->speed = 0;	569	op->speed = 0;
633	#endif
634	}	570	}
635		571
636	if (arch_init)	572	if (arch_init)
637	return;	573	return;
638		574
…		…
707	op->active_next = NULL;	643	op->active_next = NULL;
708	op->active_prev = NULL;	644	op->active_prev = NULL;
709	}	645	}
710		646
711	/*	647	/*
712	* update_object() updates the array which represents the map.	648	* update_object() updates the the map.
713	* It takes into account invisible objects (and represent squares covered	649	* It takes into account invisible objects (and represent squares covered
714	* by invisible objects by whatever is below them (unless it's another	650	* by invisible objects by whatever is below them (unless it's another
715	* invisible object, etc...)	651	* invisible object, etc...)
716	* If the object being updated is beneath a player, the look-window	652	* If the object being updated is beneath a player, the look-window
717	* of that player is updated (this might be a suboptimal way of	653	* of that player is updated (this might be a suboptimal way of
718	* updating that window, though, since update_object() is called _often_)	654	* updating that window, though, since update_object() is called _often_)
719	*	655	*
720	* action is a hint of what the caller believes need to be done.	656	* action is a hint of what the caller believes need to be done.
721	* For example, if the only thing that has changed is the face (due to
722	* an animation), we don't need to call update_position until that actually
723	* comes into view of a player. OTOH, many other things, like addition/removal
724	* of walls or living creatures may need us to update the flags now.
725	* current action are:	657	* current action are:
726	* UP_OBJ_INSERT: op was inserted	658	* UP_OBJ_INSERT: op was inserted
727	* UP_OBJ_REMOVE: op was removed	659	* UP_OBJ_REMOVE: op was removed
728	* UP_OBJ_CHANGE: object has somehow changed. In this case, we always update	660	* UP_OBJ_CHANGE: object has somehow changed. In this case, we always update
729	* as that is easier than trying to look at what may have changed.	661	* as that is easier than trying to look at what may have changed.
730	* UP_OBJ_FACE: only the objects face has changed.	662	* UP_OBJ_FACE: only the objects face has changed.
731	*/	663	*/
732
733	void	664	void
734	update_object (object *op, int action)	665	update_object (object *op, int action)
735	{	666	{
736	int update_now = 0, flags;
737	MoveType move_on, move_off, move_block, move_slow;	667	MoveType move_on, move_off, move_block, move_slow;
738		668
739	if (op == NULL)	669	if (op == NULL)
740	{	670	{
741	/* this should never happen */	671	/* this should never happen */
742	LOG (llevDebug, "update_object() called for NULL object.\n");	672	LOG (llevDebug, "update_object() called for NULL object.\n");
743	return;	673	return;
744	}	674	}
745		675
746	if (op->env != NULL)	676	if (op->env)
747	{	677	{
748	/* Animation is currently handled by client, so nothing	678	/* Animation is currently handled by client, so nothing
749	* to do in this case.	679	* to do in this case.
750	*/	680	*/
751	return;	681	return;
…		…
756	*/	686	*/
757	if (!op->map || op->map->in_memory == MAP_SAVING)	687	if (!op->map || op->map->in_memory == MAP_SAVING)
758	return;	688	return;
759		689
760	/* make sure the object is within map boundaries */	690	/* make sure the object is within map boundaries */
761	if (op->x < 0 || op->x >= MAP_WIDTH (op->map) || op->y < 0 || op->y >= MAP_HEIGHT (op->map))	691	if (op->x < 0 || op->x >= op->map->width || op->y < 0 || op->y >= op->map->height)
762	{	692	{
763	LOG (llevError, "update_object() called for object out of map!\n");	693	LOG (llevError, "update_object() called for object out of map!\n");
764	#ifdef MANY_CORES	694	#ifdef MANY_CORES
765	abort ();	695	abort ();
766	#endif	696	#endif
767	return;	697	return;
768	}	698	}
769		699
770	flags = GET_MAP_FLAGS (op->map, op->x, op->y);	700	mapspace &m = op->ms ();
771	SET_MAP_FLAGS (op->map, op->x, op->y, flags | P_NEED_UPDATE);
772	move_slow = GET_MAP_MOVE_SLOW (op->map, op->x, op->y);
773	move_on = GET_MAP_MOVE_ON (op->map, op->x, op->y);
774	move_block = GET_MAP_MOVE_BLOCK (op->map, op->x, op->y);
775	move_off = GET_MAP_MOVE_OFF (op->map, op->x, op->y);
776		701
		702	if (m.flags_ & P_NEED_UPDATE)
		703	/* nop */;
777	if (action == UP_OBJ_INSERT)	704	else if (action == UP_OBJ_INSERT)
778	{	705	{
		706	// this is likely overkill, TODO: revisit (schmorp)
779	if (QUERY_FLAG (op, FLAG_BLOCKSVIEW) && !(flags & P_BLOCKSVIEW))	707	if ((QUERY_FLAG (op, FLAG_BLOCKSVIEW) && !(m.flags_ & P_BLOCKSVIEW))
780	update_now = 1;
781
782	if (QUERY_FLAG (op, FLAG_NO_MAGIC) && !(flags & P_NO_MAGIC))	708	|| (QUERY_FLAG (op, FLAG_NO_MAGIC) && !(m.flags_ & P_NO_MAGIC))
783	update_now = 1;	709	|| (op->type == PLAYER && !(m.flags_ & P_PLAYER))
784		710	|| (op->type == SAFE_GROUND && !(m.flags_ & P_SAFE))
		711	|| (QUERY_FLAG (op, FLAG_ALIVE) && !(m.flags_ & P_IS_ALIVE))
785	if (QUERY_FLAG (op, FLAG_DAMNED) && !(flags & P_NO_CLERIC))	712	|| (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC))
786	update_now = 1;
787
788	if (QUERY_FLAG (op, FLAG_ALIVE) && !(flags & P_IS_ALIVE))
789	update_now = 1;
790
791	if (op->type == SAFE_GROUND && !(flags & P_SAFE))
792	update_now = 1;
793
794	if ((move_on | op->move_on) != move_on)	713	|| (m.move_on | op->move_on ) != m.move_on
795	update_now = 1;
796
797	if ((move_off | op->move_off) != move_off)	714	|| (m.move_off | op->move_off ) != m.move_off
798	update_now = 1;	715	|| (m.move_slow | op->move_slow) != m.move_slow
799
800	/* This isn't perfect, but I don't expect a lot of objects to	716	/* This isn't perfect, but I don't expect a lot of objects to
801	* to have move_allow right now.	717	* to have move_allow right now.
802	*/	718	*/
803	if (((move_block | op->move_block) & ~op->move_allow) != move_block)	719	|| ((m.move_block | op->move_block) & ~op->move_allow) != m.move_block
804	update_now = 1;	720	|| 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp)
805		721	m.flags_ = P_NEED_UPDATE;
806	if ((move_slow | op->move_slow) != move_slow)
807	update_now = 1;
808	}	722	}
809	/* if the object is being removed, we can't make intelligent	723	/* if the object is being removed, we can't make intelligent
810	* decisions, because remove_ob can't really pass the object	724	* decisions, because remove_ob can't really pass the object
811	* that is being removed.	725	* that is being removed.
812	*/	726	*/
813	else if (action == UP_OBJ_CHANGE || action == UP_OBJ_REMOVE)	727	else if (action == UP_OBJ_CHANGE || action == UP_OBJ_REMOVE)
814	update_now = 1;	728	m.flags_ = P_NEED_UPDATE;
815	else if (action == UP_OBJ_FACE)	729	else if (action == UP_OBJ_FACE)
816	/* Nothing to do for that case */ ;	730	/* Nothing to do for that case */ ;
817	else	731	else
818	LOG (llevError, "update_object called with invalid action: %d\n", action);	732	LOG (llevError, "update_object called with invalid action: %d\n", action);
819		733
820	if (update_now)
821	{
822	SET_MAP_FLAGS (op->map, op->x, op->y, flags | P_NO_ERROR | P_NEED_UPDATE);
823	update_position (op->map, op->x, op->y);
824	}
825
826	if (op->more != NULL)	734	if (op->more)
827	update_object (op->more, action);	735	update_object (op->more, action);
828	}	736	}
829		737
830	object::vector object::mortals;
831	object::vector object::objects; // not yet used	738	object::vector object::objects; // not yet used
832	object *object::first;	739	object *object::first;
833
834	void object::free_mortals ()
835	{
836	for (AUTODECL (i, mortals.begin ()); i != mortals.end ();)
837	if ((*i)->refcnt)
838	++i; // further delay freeing
839	else
840	{
841	delete *i;
842	mortals.erase (i);
843	}
844
845	static int lastmortals = 0;//D
846
847	if (mortals.size() != lastmortals && lastmortals > 100)//D
848	{
849	lastmortals = mortals.size ();//D
850	LOG (llevDebug, "%d objects in mortal queue\n", lastmortals);//D
851	}
852	}
853		740
854	object::object ()	741	object::object ()
855	{	742	{
856	SET_FLAG (this, FLAG_REMOVED);	743	SET_FLAG (this, FLAG_REMOVED);
857		744
…		…
906	* this function to succeed.	793	* this function to succeed.
907	*	794	*
908	* If destroy_inventory is set, free inventory as well. Else drop items in	795	* If destroy_inventory is set, free inventory as well. Else drop items in
909	* inventory to the ground.	796	* inventory to the ground.
910	*/	797	*/
911	void object::destroy (bool destroy_inventory)	798	void
		799	object::do_destroy ()
912	{	800	{
913	if (QUERY_FLAG (this, FLAG_FREED))	801	attachable::do_destroy ();
		802
		803	if (flag [FLAG_IS_LINKED])
		804	remove_button_link (this);
		805
		806	if (flag [FLAG_FRIENDLY])
		807	remove_friendly_object (this);
		808
		809	if (!flag [FLAG_REMOVED])
		810	remove ();
		811
		812	if (flag [FLAG_FREED])
914	return;	813	return;
915		814
916	if (QUERY_FLAG (this, FLAG_FRIENDLY))	815	flag [FLAG_FREED] = 1;
917	remove_friendly_object (this);
918
919	if (!QUERY_FLAG (this, FLAG_REMOVED))
920	remove_ob (this);
921
922	SET_FLAG (this, FLAG_FREED);
923		816
924	// hack to ensure that freed objects still have a valid map	817	// hack to ensure that freed objects still have a valid map
925	{	818	{
926	static maptile *freed_map; // freed objects are moved here to avoid crashes	819	static maptile *freed_map; // freed objects are moved here to avoid crashes
927		820
…		…
939	map = freed_map;	832	map = freed_map;
940	x = 1;	833	x = 1;
941	y = 1;	834	y = 1;
942	}	835	}
943		836
944	if (more)
945	{
946	more->destroy (destroy_inventory);
947	more = 0;	837	more = 0;
948	}	838	head = 0;
949		839	inv = 0;
950	if (inv)
951	{
952	/* Only if the space blocks everything do we not process -
953	* if some form of movement is allowed, let objects
954	* drop on that space.
955	*/
956	if (destroy_inventory || !map || map->in_memory != MAP_IN_MEMORY || GET_MAP_MOVE_BLOCK (map, x, y) == MOVE_ALL)
957	{
958	object *op = inv;
959
960	while (op)
961	{
962	object *tmp = op->below;
963	op->destroy (destroy_inventory);
964	op = tmp;
965	}
966	}
967	else
968	{ /* Put objects in inventory onto this space */
969	object *op = inv;
970
971	while (op)
972	{
973	object *tmp = op->below;
974
975	remove_ob (op);
976
977	if (QUERY_FLAG (op, FLAG_STARTEQUIP)
978	|| QUERY_FLAG (op, FLAG_NO_DROP) || op->type == RUNE || op->type == TRAP || QUERY_FLAG (op, FLAG_IS_A_TEMPLATE))
979	free_object (op);
980	else
981	{
982	op->x = x;
983	op->y = y;
984	insert_ob_in_map (op, map, 0, 0); /* Insert in same map as the envir */
985	}
986
987	op = tmp;
988	}
989	}
990	}
991		840
992	// clear those pointers that likely might have circular references to us	841	// clear those pointers that likely might have circular references to us
993	owner = 0;	842	owner = 0;
994	enemy = 0;	843	enemy = 0;
995	attacked_by = 0;	844	attacked_by = 0;
…		…
1000	/* Remove object from the active list */	849	/* Remove object from the active list */
1001	speed = 0;	850	speed = 0;
1002	update_ob_speed (this);	851	update_ob_speed (this);
1003		852
1004	unlink ();	853	unlink ();
		854	}
1005		855
1006	mortals.push_back (this);	856	/*
		857	* Remove and free all objects in the inventory of the given object.
		858	* object.c ?
		859	*/
		860	void
		861	object::destroy_inv (bool drop_to_ground)
		862	{
		863	if (!inv)
		864	return;
		865
		866	/* Only if the space blocks everything do we not process -
		867	* if some form of movement is allowed, let objects
		868	* drop on that space.
		869	*/
		870	if (!drop_to_ground || !map || map->in_memory != MAP_IN_MEMORY || GET_MAP_MOVE_BLOCK (map, x, y) == MOVE_ALL)
		871	{
		872	while (inv)
		873	{
		874	inv->destroy_inv (drop_to_ground);
		875	inv->destroy ();
		876	}
		877	}
		878	else
		879	{ /* Put objects in inventory onto this space */
		880	while (inv)
		881	{
		882	object *op = inv;
		883
		884	if (op->flag [FLAG_STARTEQUIP]
		885	|| op->flag [FLAG_NO_DROP]
		886	|| op->type == RUNE
		887	|| op->type == TRAP
		888	|| op->flag [FLAG_IS_A_TEMPLATE])
		889	op->destroy ();
		890	else
		891	{
		892	op->remove ();
		893	op->x = x;
		894	op->y = y;
		895	insert_ob_in_map (op, map, 0, 0); /* Insert in same map as the envir */
		896	}
		897	}
		898	}
		899	}
		900
		901	void
		902	object::destroy (bool destroy_inventory)
		903	{
		904	if (destroyed ())
		905	return;
		906
		907	if (more)
		908	{
		909	//TODO: non-head objects must not have inventory
		910	more->destroy (destroy_inventory);
		911	more = 0;
		912	}
		913
		914	if (destroy_inventory)
		915	destroy_inv (true);
		916
		917	attachable::destroy ();
1007	}	918	}
1008		919
1009	/*	920	/*
1010	* sub_weight() recursively (outwards) subtracts a number from the	921	* sub_weight() recursively (outwards) subtracts a number from the
1011	* weight of an object (and what is carried by it's environment(s)).	922	* weight of an object (and what is carried by it's environment(s)).
1012	*/	923	*/
1013
1014	void	924	void
1015	sub_weight (object *op, signed long weight)	925	sub_weight (object *op, signed long weight)
1016	{	926	{
1017	while (op != NULL)	927	while (op != NULL)
1018	{	928	{
…		…
1022	op->carrying -= weight;	932	op->carrying -= weight;
1023	op = op->env;	933	op = op->env;
1024	}	934	}
1025	}	935	}
1026		936
1027	/* remove_ob(op):	937	/* op->remove ():
1028	* This function removes the object op from the linked list of objects	938	* This function removes the object op from the linked list of objects
1029	* which it is currently tied to. When this function is done, the	939	* which it is currently tied to. When this function is done, the
1030	* object will have no environment. If the object previously had an	940	* object will have no environment. If the object previously had an
1031	* environment, the x and y coordinates will be updated to	941	* environment, the x and y coordinates will be updated to
1032	* the previous environment.	942	* the previous environment.
1033	* Beware: This function is called from the editor as well!	943	* Beware: This function is called from the editor as well!
1034	*/	944	*/
1035
1036	void	945	void
1037	remove_ob (object *op)	946	object::remove ()
1038	{	947	{
1039	object *tmp, *last = 0;	948	object *tmp, *last = 0;
1040	object *otmp;	949	object *otmp;
1041		950
1042	int check_walk_off;
1043	maptile *m;
1044
1045	sint16 x, y;
1046
1047	if (QUERY_FLAG (op, FLAG_REMOVED))	951	if (QUERY_FLAG (this, FLAG_REMOVED))
1048	return;	952	return;
1049		953
1050	SET_FLAG (op, FLAG_REMOVED);	954	SET_FLAG (this, FLAG_REMOVED);
		955	INVOKE_OBJECT (REMOVE, this);
1051		956
1052	if (op->more != NULL)	957	if (more)
1053	remove_ob (op->more);	958	more->remove ();
1054		959
1055	/*	960	/*
1056	* In this case, the object to be removed is in someones	961	* In this case, the object to be removed is in someones
1057	* inventory.	962	* inventory.
1058	*/	963	*/
1059	if (op->env != NULL)	964	if (env)
1060	{	965	{
1061	if (op->nrof)	966	if (nrof)
1062	sub_weight (op->env, op->weight * op->nrof);	967	sub_weight (env, weight * nrof);
1063	else	968	else
1064	sub_weight (op->env, op->weight + op->carrying);	969	sub_weight (env, weight + carrying);
1065		970
1066	/* NO_FIX_PLAYER is set when a great many changes are being	971	/* NO_FIX_PLAYER is set when a great many changes are being
1067	* made to players inventory. If set, avoiding the call	972	* made to players inventory. If set, avoiding the call
1068	* to save cpu time.	973	* to save cpu time.
1069	*/	974	*/
1070	if ((otmp = is_player_inv (op->env)) != NULL && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))	975	if ((otmp = in_player ()) && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
1071	fix_player (otmp);	976	otmp->update_stats ();
1072		977
1073	if (op->above != NULL)	978	if (above != NULL)
1074	op->above->below = op->below;	979	above->below = below;
1075	else	980	else
1076	op->env->inv = op->below;	981	env->inv = below;
1077		982
1078	if (op->below != NULL)	983	if (below != NULL)
1079	op->below->above = op->above;	984	below->above = above;
1080		985
1081	/* we set up values so that it could be inserted into	986	/* we set up values so that it could be inserted into
1082	* the map, but we don't actually do that - it is up	987	* the map, but we don't actually do that - it is up
1083	* to the caller to decide what we want to do.	988	* to the caller to decide what we want to do.
1084	*/	989	*/
1085	op->x = op->env->x, op->y = op->env->y;	990	x = env->x, y = env->y;
1086	op->map = op->env->map;	991	map = env->map;
1087	op->above = NULL, op->below = NULL;	992	above = 0, below = 0;
1088	op->env = NULL;	993	env = 0;
1089	}	994	}
1090	else if (op->map)	995	else if (map)
1091	{	996	{
1092	x = op->x;
1093	y = op->y;
1094	m = get_map_from_coord (op->map, &x, &y);
1095
1096	if (!m)
1097	{
1098	LOG (llevError, "remove_ob called when object was on map but appears to not be within valid coordinates? %s (%d,%d)\n",
1099	op->map->path, op->x, op->y);
1100	/* in old days, we used to set x and y to 0 and continue.
1101	* it seems if we get into this case, something is probablye
1102	* screwed up and should be fixed.
1103	*/
1104	abort ();
1105	}
1106
1107	if (op->map != m)
1108	LOG (llevDebug, "remove_ob: Object not really on map it claimed to be on? %s != %s, %d,%d != %d,%d\n",
1109	op->map->path, m->path, op->x, op->y, x, y);
1110
1111	/* Re did the following section of code - it looks like it had	997	/* Re did the following section of code - it looks like it had
1112	* lots of logic for things we no longer care about	998	* lots of logic for things we no longer care about
1113	*/	999	*/
1114		1000
1115	/* link the object above us */	1001	/* link the object above us */
1116	if (op->above)	1002	if (above)
1117	op->above->below = op->below;	1003	above->below = below;
1118	else	1004	else
1119	SET_MAP_TOP (m, x, y, op->below); /* we were top, set new top */	1005	map->at (x, y).top = below; /* we were top, set new top */
1120		1006
1121	/* Relink the object below us, if there is one */	1007	/* Relink the object below us, if there is one */
1122	if (op->below)	1008	if (below)
1123	op->below->above = op->above;	1009	below->above = above;
1124	else	1010	else
1125	{	1011	{
1126	/* Nothing below, which means we need to relink map object for this space	1012	/* Nothing below, which means we need to relink map object for this space
1127	* use translated coordinates in case some oddness with map tiling is	1013	* use translated coordinates in case some oddness with map tiling is
1128	* evident	1014	* evident
1129	*/	1015	*/
1130	if (GET_MAP_OB (m, x, y) != op)	1016	if (GET_MAP_OB (map, x, y) != this)
1131	{	1017	{
1132	char *dump = dump_object (op);	1018	char *dump = dump_object (this);
1133	LOG (llevError,	1019	LOG (llevError,
1134	"remove_ob: GET_MAP_OB does not return object to be removed even though it appears to be on the bottom?\n%s\n", dump);	1020	"remove_ob: GET_MAP_OB does not return object to be removed even though it appears to be on the bottom?\n%s\n", dump);
1135	free (dump);	1021	free (dump);
1136	dump = dump_object (GET_MAP_OB (m, x, y));	1022	dump = dump_object (GET_MAP_OB (map, x, y));
1137	LOG (llevError, "%s\n", dump);	1023	LOG (llevError, "%s\n", dump);
1138	free (dump);	1024	free (dump);
1139	}	1025	}
1140		1026
1141	SET_MAP_OB (m, x, y, op->above); /* goes on above it. */	1027	map->at (x, y).bot = above; /* goes on above it. */
1142	}	1028	}
1143		1029
1144	op->above = 0;	1030	above = 0;
1145	op->below = 0;	1031	below = 0;
1146		1032
1147	if (op->map->in_memory == MAP_SAVING)	1033	if (map->in_memory == MAP_SAVING)
1148	return;	1034	return;
1149		1035
1150	check_walk_off = !QUERY_FLAG (op, FLAG_NO_APPLY);	1036	int check_walk_off = !flag [FLAG_NO_APPLY];
1151		1037
1152	for (tmp = GET_MAP_OB (m, x, y); tmp != NULL; tmp = tmp->above)	1038	for (tmp = GET_MAP_OB (map, x, y); tmp; tmp = tmp->above)
1153	{	1039	{
1154	/* No point updating the players look faces if he is the object	1040	/* No point updating the players look faces if he is the object
1155	* being removed.	1041	* being removed.
1156	*/	1042	*/
1157		1043
1158	if (tmp->type == PLAYER && tmp != op)	1044	if (tmp->type == PLAYER && tmp != this)
1159	{	1045	{
1160	/* If a container that the player is currently using somehow gets	1046	/* If a container that the player is currently using somehow gets
1161	* removed (most likely destroyed), update the player view	1047	* removed (most likely destroyed), update the player view
1162	* appropriately.	1048	* appropriately.
1163	*/	1049	*/
1164	if (tmp->container == op)	1050	if (tmp->container == this)
1165	{	1051	{
1166	CLEAR_FLAG (op, FLAG_APPLIED);	1052	flag [FLAG_APPLIED] = 0;
1167	tmp->container = NULL;	1053	tmp->container = 0;
1168	}	1054	}
1169		1055
1170	tmp->contr->socket.update_look = 1;	1056	if (tmp->contr->ns)
		1057	tmp->contr->ns->floorbox_update ();
1171	}	1058	}
1172		1059
1173	/* See if player moving off should effect something */	1060	/* See if player moving off should effect something */
1174	if (check_walk_off	1061	if (check_walk_off
1175	&& ((op->move_type & tmp->move_off)	1062	&& ((move_type & tmp->move_off)
1176	&& (op->move_type & ~tmp->move_off & ~tmp->move_block) == 0))	1063	&& (move_type & ~tmp->move_off & ~tmp->move_block) == 0))
1177	{	1064	{
1178	move_apply (tmp, op, NULL);	1065	move_apply (tmp, this, 0);
1179		1066
1180	if (op->destroyed ())	1067	if (destroyed ())
1181	LOG (llevError, "BUG: remove_ob(): name %s, destroyed leaving object\n", tmp->debug_desc ());	1068	LOG (llevError, "BUG: remove_ob(): name %s, destroyed leaving object\n", tmp->debug_desc ());
1182	}	1069	}
1183		1070
1184	/* Eneq(@csd.uu.se): Fixed this to skip tmp->above=tmp */	1071	/* Eneq(@csd.uu.se): Fixed this to skip tmp->above=tmp */
1185		1072
1186	if (tmp->above == tmp)	1073	if (tmp->above == tmp)
1187	tmp->above = NULL;	1074	tmp->above = 0;
1188		1075
1189	last = tmp;	1076	last = tmp;
1190	}	1077	}
1191		1078
1192	/* last == NULL of there are no objects on this space */	1079	/* last == NULL of there are no objects on this space */
1193	if (last == NULL)	1080	if (!last)
1194	{	1081	map->at (x, y).flags_ = P_NEED_UPDATE;
1195	/* set P_NEED_UPDATE, otherwise update_position will complain. In theory,
1196	* we could preserve the flags (GET_MAP_FLAGS), but update_position figures
1197	* those out anyways, and if there are any flags set right now, they won't
1198	* be correct anyways.
1199	*/
1200	SET_MAP_FLAGS (op->map, op->x, op->y, P_NEED_UPDATE);
1201	update_position (op->map, op->x, op->y);
1202	}
1203	else	1082	else
1204	update_object (last, UP_OBJ_REMOVE);	1083	update_object (last, UP_OBJ_REMOVE);
1205		1084
1206	if (QUERY_FLAG (op, FLAG_BLOCKSVIEW) || (op->glow_radius != 0))	1085	if (flag [FLAG_BLOCKSVIEW] || glow_radius)
1207	update_all_los (op->map, op->x, op->y);	1086	update_all_los (map, x, y);
1208	}	1087	}
1209	}	1088	}
1210		1089
1211	/*	1090	/*
1212	* merge_ob(op,top):	1091	* merge_ob(op,top):
…		…
1220	merge_ob (object *op, object *top)	1099	merge_ob (object *op, object *top)
1221	{	1100	{
1222	if (!op->nrof)	1101	if (!op->nrof)
1223	return 0;	1102	return 0;
1224		1103
1225	if (top == NULL)	1104	if (top)
1226	for (top = op; top != NULL && top->above != NULL; top = top->above);	1105	for (top = op; top && top->above; top = top->above)
		1106	;
1227		1107
1228	for (; top != NULL; top = top->below)	1108	for (; top; top = top->below)
1229	{	1109	{
1230	if (top == op)	1110	if (top == op)
1231	continue;	1111	continue;
1232	if (CAN_MERGE (op, top))	1112
		1113	if (object::can_merge (op, top))
1233	{	1114	{
1234	top->nrof += op->nrof;	1115	top->nrof += op->nrof;
1235		1116
1236	/* CLEAR_FLAG(top,FLAG_STARTEQUIP);*/	1117	/* CLEAR_FLAG(top,FLAG_STARTEQUIP);*/
1237	op->weight = 0; /* Don't want any adjustements now */	1118	op->weight = 0; /* Don't want any adjustements now */
1238	remove_ob (op);	1119	op->destroy ();
1239	free_object (op);
1240	return top;	1120	return top;
1241	}	1121	}
1242	}	1122	}
1243		1123
1244	return 0;	1124	return 0;
…		…
1283	* Return value:	1163	* Return value:
1284	* new object if 'op' was merged with other object	1164	* new object if 'op' was merged with other object
1285	* NULL if 'op' was destroyed	1165	* NULL if 'op' was destroyed
1286	* just 'op' otherwise	1166	* just 'op' otherwise
1287	*/	1167	*/
1288
1289	object *	1168	object *
1290	insert_ob_in_map (object *op, maptile *m, object *originator, int flag)	1169	insert_ob_in_map (object *op, maptile *m, object *originator, int flag)
1291	{	1170	{
1292	object *tmp, *top, *floor = NULL;	1171	object *tmp, *top, *floor = NULL;
1293	sint16 x, y;	1172	sint16 x, y;
…		…
1327	LOG (llevError, "Trying to insert (map) inserted object.\n%s\n", dump);	1206	LOG (llevError, "Trying to insert (map) inserted object.\n%s\n", dump);
1328	free (dump);	1207	free (dump);
1329	return op;	1208	return op;
1330	}	1209	}
1331		1210
1332	if (op->more != NULL)	1211	if (op->more)
1333	{	1212	{
1334	/* The part may be on a different map. */	1213	/* The part may be on a different map. */
1335		1214
1336	object *more = op->more;	1215	object *more = op->more;
1337		1216
…		…
1353	if (insert_ob_in_map (more, more->map, originator, flag) == NULL)	1232	if (insert_ob_in_map (more, more->map, originator, flag) == NULL)
1354	{	1233	{
1355	if (!op->head)	1234	if (!op->head)
1356	LOG (llevError, "BUG: insert_ob_in_map(): inserting op->more killed op\n");	1235	LOG (llevError, "BUG: insert_ob_in_map(): inserting op->more killed op\n");
1357		1236
1358	return NULL;	1237	return 0;
1359	}	1238	}
1360	}	1239	}
1361		1240
1362	CLEAR_FLAG (op, FLAG_REMOVED);	1241	CLEAR_FLAG (op, FLAG_REMOVED);
1363		1242
…		…
1371		1250
1372	/* this has to be done after we translate the coordinates.	1251	/* this has to be done after we translate the coordinates.
1373	*/	1252	*/
1374	if (op->nrof && !(flag & INS_NO_MERGE))	1253	if (op->nrof && !(flag & INS_NO_MERGE))
1375	for (tmp = GET_MAP_OB (op->map, x, y); tmp != NULL; tmp = tmp->above)	1254	for (tmp = GET_MAP_OB (op->map, x, y); tmp != NULL; tmp = tmp->above)
1376	if (CAN_MERGE (op, tmp))	1255	if (object::can_merge (op, tmp))
1377	{	1256	{
1378	op->nrof += tmp->nrof;	1257	op->nrof += tmp->nrof;
1379	remove_ob (tmp);	1258	tmp->destroy ();
1380	free_object (tmp);
1381	}	1259	}
1382		1260
1383	CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */	1261	CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */
1384	CLEAR_FLAG (op, FLAG_INV_LOCKED);	1262	CLEAR_FLAG (op, FLAG_INV_LOCKED);
1385		1263
…		…
1398	op->below = originator->below;	1276	op->below = originator->below;
1399		1277
1400	if (op->below)	1278	if (op->below)
1401	op->below->above = op;	1279	op->below->above = op;
1402	else	1280	else
1403	SET_MAP_OB (op->map, op->x, op->y, op);	1281	op->ms ().bot = op;
1404		1282
1405	/* since *below* originator, no need to update top */	1283	/* since *below* originator, no need to update top */
1406	originator->below = op;	1284	originator->below = op;
1407	}	1285	}
1408	else	1286	else
…		…
1454	* If INS_ON_TOP is used, don't do this processing	1332	* If INS_ON_TOP is used, don't do this processing
1455	* Need to find the object that in fact blocks view, otherwise	1333	* Need to find the object that in fact blocks view, otherwise
1456	* stacking is a bit odd.	1334	* stacking is a bit odd.
1457	*/	1335	*/
1458	if (!(flag & INS_ON_TOP) &&	1336	if (!(flag & INS_ON_TOP) &&
1459	(get_map_flags (op->map, NULL, op->x, op->y, NULL, NULL) & P_BLOCKSVIEW) && (op->face && !op->face->visibility))	1337	(get_map_flags (op->map, 0, op->x, op->y, 0, 0) & P_BLOCKSVIEW) && (op->face && !op->face->visibility))
1460	{	1338	{
1461	for (last = top; last != floor; last = last->below)	1339	for (last = top; last != floor; last = last->below)
1462	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))	1340	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))
1463	break;	1341	break;
1464	/* Check to see if we found the object that blocks view,	1342	/* Check to see if we found the object that blocks view,
…		…
1487		1365
1488	if (op->above)	1366	if (op->above)
1489	op->above->below = op;	1367	op->above->below = op;
1490		1368
1491	op->below = NULL;	1369	op->below = NULL;
1492	SET_MAP_OB (op->map, op->x, op->y, op);	1370	op->ms ().bot = op;
1493	}	1371	}
1494	else	1372	else
1495	{ /* get inserted into the stack above top */	1373	{ /* get inserted into the stack above top */
1496	op->above = top->above;	1374	op->above = top->above;
1497		1375
…		…
1501	op->below = top;	1379	op->below = top;
1502	top->above = op;	1380	top->above = op;
1503	}	1381	}
1504		1382
1505	if (op->above == NULL)	1383	if (op->above == NULL)
1506	SET_MAP_TOP (op->map, op->x, op->y, op);	1384	op->ms ().top = op;
1507	} /* else not INS_BELOW_ORIGINATOR */	1385	} /* else not INS_BELOW_ORIGINATOR */
1508		1386
1509	if (op->type == PLAYER)	1387	if (op->type == PLAYER)
1510	op->contr->do_los = 1;	1388	op->contr->do_los = 1;
1511		1389
1512	/* If we have a floor, we know the player, if any, will be above	1390	/* If we have a floor, we know the player, if any, will be above
1513	* it, so save a few ticks and start from there.	1391	* it, so save a few ticks and start from there.
1514	*/	1392	*/
1515	if (!(flag & INS_MAP_LOAD))	1393	if (!(flag & INS_MAP_LOAD))
1516	for (tmp = floor ? floor : GET_MAP_OB (op->map, op->x, op->y); tmp != NULL; tmp = tmp->above)	1394	if (object *pl = op->ms ().player ())
1517	if (tmp->type == PLAYER)	1395	if (pl->contr->ns)
1518	tmp->contr->socket.update_look = 1;	1396	pl->contr->ns->floorbox_update ();
1519		1397
1520	/* If this object glows, it may affect lighting conditions that are	1398	/* If this object glows, it may affect lighting conditions that are
1521	* visible to others on this map. But update_all_los is really	1399	* visible to others on this map. But update_all_los is really
1522	* an inefficient way to do this, as it means los for all players	1400	* an inefficient way to do this, as it means los for all players
1523	* on the map will get recalculated. The players could very well	1401	* on the map will get recalculated. The players could very well
1524	* be far away from this change and not affected in any way -	1402	* be far away from this change and not affected in any way -
1525	* this should get redone to only look for players within range,	1403	* this should get redone to only look for players within range,
1526	* or just updating the P_NEED_UPDATE for spaces within this area	1404	* or just updating the P_NEED_UPDATE for spaces within this area
1527	* of effect may be sufficient.	1405	* of effect may be sufficient.
1528	*/	1406	*/
1529	if (MAP_DARKNESS (op->map) && (op->glow_radius != 0))	1407	if (op->map->darkness && (op->glow_radius != 0))
1530	update_all_los (op->map, op->x, op->y);	1408	update_all_los (op->map, op->x, op->y);
1531		1409
1532	/* updates flags (blocked, alive, no magic, etc) for this map space */	1410	/* updates flags (blocked, alive, no magic, etc) for this map space */
1533	update_object (op, UP_OBJ_INSERT);	1411	update_object (op, UP_OBJ_INSERT);
1534		1412
		1413	INVOKE_OBJECT (INSERT, op);
		1414
1535	/* Don't know if moving this to the end will break anything. However,	1415	/* Don't know if moving this to the end will break anything. However,
1536	* we want to have update_look set above before calling this.	1416	* we want to have floorbox_update called before calling this.
1537	*	1417	*
1538	* check_move_on() must be after this because code called from	1418	* check_move_on() must be after this because code called from
1539	* check_move_on() depends on correct map flags (so functions like	1419	* check_move_on() depends on correct map flags (so functions like
1540	* blocked() and wall() work properly), and these flags are updated by	1420	* blocked() and wall() work properly), and these flags are updated by
1541	* update_object().	1421	* update_object().
…		…
1543		1423
1544	/* if this is not the head or flag has been passed, don't check walk on status */	1424	/* if this is not the head or flag has been passed, don't check walk on status */
1545	if (!(flag & INS_NO_WALK_ON) && !op->head)	1425	if (!(flag & INS_NO_WALK_ON) && !op->head)
1546	{	1426	{
1547	if (check_move_on (op, originator))	1427	if (check_move_on (op, originator))
1548	return NULL;	1428	return 0;
1549		1429
1550	/* If we are a multi part object, lets work our way through the check	1430	/* If we are a multi part object, lets work our way through the check
1551	* walk on's.	1431	* walk on's.
1552	*/	1432	*/
1553	for (tmp = op->more; tmp != NULL; tmp = tmp->more)	1433	for (tmp = op->more; tmp != NULL; tmp = tmp->more)
1554	if (check_move_on (tmp, originator))	1434	if (check_move_on (tmp, originator))
1555	return NULL;	1435	return 0;
1556	}	1436	}
1557		1437
1558	return op;	1438	return op;
1559	}	1439	}
1560		1440
1561	/* this function inserts an object in the map, but if it	1441	/* this function inserts an object in the map, but if it
1562	* finds an object of its own type, it'll remove that one first.	1442	* finds an object of its own type, it'll remove that one first.
1563	* op is the object to insert it under: supplies x and the map.	1443	* op is the object to insert it under: supplies x and the map.
1564	*/	1444	*/
1565	void	1445	void
1566	replace_insert_ob_in_map (const char *arch_string, object *op)	1446	replace_insert_ob_in_map (const char *arch_string, object *op)
1567	{	1447	{
1568	object *	1448	object *tmp, *tmp1;
1569	tmp;
1570	object *
1571	tmp1;
1572		1449
1573	/* first search for itself and remove any old instances */	1450	/* first search for itself and remove any old instances */
1574		1451
1575	for (tmp = GET_MAP_OB (op->map, op->x, op->y); tmp != NULL; tmp = tmp->above)	1452	for (tmp = GET_MAP_OB (op->map, op->x, op->y); tmp != NULL; tmp = tmp->above)
1576	{
1577	if (!strcmp (tmp->arch->name, arch_string)) /* same archetype */	1453	if (!strcmp (tmp->arch->name, arch_string)) /* same archetype */
1578	{	1454	tmp->destroy ();
1579	remove_ob (tmp);
1580	free_object (tmp);
1581	}
1582	}
1583		1455
1584	tmp1 = arch_to_object (archetype::find (arch_string));	1456	tmp1 = arch_to_object (archetype::find (arch_string));
1585		1457
1586	tmp1->x = op->x;	1458	tmp1->x = op->x;
1587	tmp1->y = op->y;	1459	tmp1->y = op->y;
…		…
1597	*/	1469	*/
1598		1470
1599	object *	1471	object *
1600	get_split_ob (object *orig_ob, uint32 nr)	1472	get_split_ob (object *orig_ob, uint32 nr)
1601	{	1473	{
1602	object *	1474	object *newob;
1603	newob;
1604	int
1605	is_removed = (QUERY_FLAG (orig_ob, FLAG_REMOVED) != 0);	1475	int is_removed = (QUERY_FLAG (orig_ob, FLAG_REMOVED) != 0);
1606		1476
1607	if (orig_ob->nrof < nr)	1477	if (orig_ob->nrof < nr)
1608	{	1478	{
1609	sprintf (errmsg, "There are only %d %ss.", orig_ob->nrof ? orig_ob->nrof : 1, &orig_ob->name);	1479	sprintf (errmsg, "There are only %d %ss.", orig_ob->nrof ? orig_ob->nrof : 1, &orig_ob->name);
1610	return NULL;	1480	return NULL;
1611	}	1481	}
1612		1482
1613	newob = object_create_clone (orig_ob);	1483	newob = object_create_clone (orig_ob);
1614		1484
1615	if ((orig_ob->nrof -= nr) < 1)	1485	if ((orig_ob->nrof -= nr) < 1)
1616	{	1486	orig_ob->destroy (1);
1617	if (!is_removed)
1618	remove_ob (orig_ob);
1619	free_object2 (orig_ob, 1);
1620	}
1621	else if (!is_removed)	1487	else if (!is_removed)
1622	{	1488	{
1623	if (orig_ob->env != NULL)	1489	if (orig_ob->env != NULL)
1624	sub_weight (orig_ob->env, orig_ob->weight * nr);	1490	sub_weight (orig_ob->env, orig_ob->weight * nr);
1625	if (orig_ob->env == NULL && orig_ob->map->in_memory != MAP_IN_MEMORY)	1491	if (orig_ob->env == NULL && orig_ob->map->in_memory != MAP_IN_MEMORY)
…		…
1645		1511
1646	object *	1512	object *
1647	decrease_ob_nr (object *op, uint32 i)	1513	decrease_ob_nr (object *op, uint32 i)
1648	{	1514	{
1649	object *tmp;	1515	object *tmp;
1650	player *pl;
1651		1516
1652	if (i == 0) /* objects with op->nrof require this check */	1517	if (i == 0) /* objects with op->nrof require this check */
1653	return op;	1518	return op;
1654		1519
1655	if (i > op->nrof)	1520	if (i > op->nrof)
1656	i = op->nrof;	1521	i = op->nrof;
1657		1522
1658	if (QUERY_FLAG (op, FLAG_REMOVED))	1523	if (QUERY_FLAG (op, FLAG_REMOVED))
1659	op->nrof -= i;	1524	op->nrof -= i;
1660	else if (op->env != NULL)	1525	else if (op->env)
1661	{	1526	{
1662	/* is this object in the players inventory, or sub container	1527	/* is this object in the players inventory, or sub container
1663	* therein?	1528	* therein?
1664	*/	1529	*/
1665	tmp = is_player_inv (op->env);	1530	tmp = op->in_player ();
1666	/* nope. Is this a container the player has opened?	1531	/* nope. Is this a container the player has opened?
1667	* If so, set tmp to that player.	1532	* If so, set tmp to that player.
1668	* IMO, searching through all the players will mostly	1533	* IMO, searching through all the players will mostly
1669	* likely be quicker than following op->env to the map,	1534	* likely be quicker than following op->env to the map,
1670	* and then searching the map for a player.	1535	* and then searching the map for a player.
1671	*/	1536	*/
1672	if (!tmp)	1537	if (!tmp)
1673	{	1538	for_all_players (pl)
1674	for (pl = first_player; pl; pl = pl->next)
1675	if (pl->ob->container == op->env)	1539	if (pl->ob->container == op->env)
		1540	{
		1541	tmp = pl->ob;
1676	break;	1542	break;
1677	if (pl)
1678	tmp = pl->ob;
1679	else
1680	tmp = NULL;
1681	}	1543	}
1682		1544
1683	if (i < op->nrof)	1545	if (i < op->nrof)
1684	{	1546	{
1685	sub_weight (op->env, op->weight * i);	1547	sub_weight (op->env, op->weight * i);
1686	op->nrof -= i;	1548	op->nrof -= i;
1687	if (tmp)	1549	if (tmp)
1688	{
1689	esrv_send_item (tmp, op);	1550	esrv_send_item (tmp, op);
1690	}
1691	}	1551	}
1692	else	1552	else
1693	{	1553	{
1694	remove_ob (op);	1554	op->remove ();
1695	op->nrof = 0;	1555	op->nrof = 0;
1696	if (tmp)	1556	if (tmp)
1697	{
1698	esrv_del_item (tmp->contr, op->count);	1557	esrv_del_item (tmp->contr, op->count);
1699	}
1700	}	1558	}
1701	}	1559	}
1702	else	1560	else
1703	{	1561	{
1704	object *above = op->above;	1562	object *above = op->above;
1705		1563
1706	if (i < op->nrof)	1564	if (i < op->nrof)
1707	op->nrof -= i;	1565	op->nrof -= i;
1708	else	1566	else
1709	{	1567	{
1710	remove_ob (op);	1568	op->remove ();
1711	op->nrof = 0;	1569	op->nrof = 0;
1712	}	1570	}
1713		1571
1714	/* Since we just removed op, op->above is null */	1572	/* Since we just removed op, op->above is null */
1715	for (tmp = above; tmp != NULL; tmp = tmp->above)	1573	for (tmp = above; tmp; tmp = tmp->above)
1716	if (tmp->type == PLAYER)	1574	if (tmp->type == PLAYER)
1717	{	1575	{
1718	if (op->nrof)	1576	if (op->nrof)
1719	esrv_send_item (tmp, op);	1577	esrv_send_item (tmp, op);
1720	else	1578	else
…		…
1724		1582
1725	if (op->nrof)	1583	if (op->nrof)
1726	return op;	1584	return op;
1727	else	1585	else
1728	{	1586	{
1729	free_object (op);	1587	op->destroy ();
1730	return NULL;	1588	return 0;
1731	}	1589	}
1732	}	1590	}
1733		1591
1734	/*	1592	/*
1735	* add_weight(object, weight) adds the specified weight to an object,	1593	* add_weight(object, weight) adds the specified weight to an object,
…		…
1747	op->carrying += weight;	1605	op->carrying += weight;
1748	op = op->env;	1606	op = op->env;
1749	}	1607	}
1750	}	1608	}
1751		1609
1752	/*
1753	* insert_ob_in_ob(op,environment):
1754	* This function inserts the object op in the linked list
1755	* inside the object environment.
1756	*
1757	* Eneq(@csd.uu.se): Altered insert_ob_in_ob to make things picked up enter
1758	* the inventory at the last position or next to other objects of the same
1759	* type.
1760	* Frank: Now sorted by type, archetype and magic!
1761	*
1762	* The function returns now pointer to inserted item, and return value can
1763	* be != op, if items are merged. -Tero
1764	*/
1765
1766	object *	1610	object *
1767	insert_ob_in_ob (object *op, object *where)	1611	insert_ob_in_ob (object *op, object *where)
1768	{	1612	{
1769	object *	1613	if (!where)
1770	tmp, *
1771	otmp;
1772
1773	if (!QUERY_FLAG (op, FLAG_REMOVED))
1774	{
1775	char *dump = dump_object (op);
1776	LOG (llevError, "Trying to insert (ob) inserted object.\n%s\n", dump);
1777	free (dump);
1778	return op;
1779	}
1780
1781	if (where == NULL)
1782	{	1614	{
1783	char *dump = dump_object (op);	1615	char *dump = dump_object (op);
1784	LOG (llevError, "Trying to put object in NULL.\n%s\n", dump);	1616	LOG (llevError, "Trying to put object in NULL.\n%s\n", dump);
1785	free (dump);	1617	free (dump);
1786	return op;	1618	return op;
1787	}	1619	}
1788		1620
1789	if (where->head)	1621	if (where->head)
1790	{	1622	{
1791	LOG (llevDebug, "Warning: Tried to insert object wrong part of multipart object.\n");	1623	LOG (llevDebug, "Warning: Tried to insert object into wrong part of multipart object.\n");
1792	where = where->head;	1624	where = where->head;
1793	}	1625	}
		1626
		1627	return where->insert (op);
		1628	}
		1629
		1630	/*
		1631	* env->insert (op)
		1632	* This function inserts the object op in the linked list
		1633	* inside the object environment.
		1634	*
		1635	* The function returns now pointer to inserted item, and return value can
		1636	* be != op, if items are merged. -Tero
		1637	*/
		1638
		1639	object *
		1640	object::insert (object *op)
		1641	{
		1642	object *tmp, *otmp;
		1643
		1644	if (!QUERY_FLAG (op, FLAG_REMOVED))
		1645	op->remove ();
1794		1646
1795	if (op->more)	1647	if (op->more)
1796	{	1648	{
1797	LOG (llevError, "Tried to insert multipart object %s (%d)\n", &op->name, op->count);	1649	LOG (llevError, "Tried to insert multipart object %s (%d)\n", &op->name, op->count);
1798	return op;	1650	return op;
…		…
1800		1652
1801	CLEAR_FLAG (op, FLAG_OBJ_ORIGINAL);	1653	CLEAR_FLAG (op, FLAG_OBJ_ORIGINAL);
1802	CLEAR_FLAG (op, FLAG_REMOVED);	1654	CLEAR_FLAG (op, FLAG_REMOVED);
1803	if (op->nrof)	1655	if (op->nrof)
1804	{	1656	{
1805	for (tmp = where->inv; tmp != NULL; tmp = tmp->below)	1657	for (tmp = inv; tmp != NULL; tmp = tmp->below)
1806	if (CAN_MERGE (tmp, op))	1658	if (object::can_merge (tmp, op))
1807	{	1659	{
1808	/* return the original object and remove inserted object	1660	/* return the original object and remove inserted object
1809	(client needs the original object) */	1661	(client needs the original object) */
1810	tmp->nrof += op->nrof;	1662	tmp->nrof += op->nrof;
1811	/* Weight handling gets pretty funky. Since we are adding to	1663	/* Weight handling gets pretty funky. Since we are adding to
1812	* tmp->nrof, we need to increase the weight.	1664	* tmp->nrof, we need to increase the weight.
1813	*/	1665	*/
1814	add_weight (where, op->weight * op->nrof);	1666	add_weight (this, op->weight * op->nrof);
1815	SET_FLAG (op, FLAG_REMOVED);	1667	SET_FLAG (op, FLAG_REMOVED);
1816	free_object (op); /* free the inserted object */	1668	op->destroy (); /* free the inserted object */
1817	op = tmp;	1669	op = tmp;
1818	remove_ob (op); /* and fix old object's links */	1670	op->remove (); /* and fix old object's links */
1819	CLEAR_FLAG (op, FLAG_REMOVED);	1671	CLEAR_FLAG (op, FLAG_REMOVED);
1820	break;	1672	break;
1821	}	1673	}
1822		1674
1823	/* I assume combined objects have no inventory	1675	/* I assume combined objects have no inventory
1824	* We add the weight - this object could have just been removed	1676	* We add the weight - this object could have just been removed
1825	* (if it was possible to merge). calling remove_ob will subtract	1677	* (if it was possible to merge). calling remove_ob will subtract
1826	* the weight, so we need to add it in again, since we actually do	1678	* the weight, so we need to add it in again, since we actually do
1827	* the linking below	1679	* the linking below
1828	*/	1680	*/
1829	add_weight (where, op->weight * op->nrof);	1681	add_weight (this, op->weight * op->nrof);
1830	}	1682	}
1831	else	1683	else
1832	add_weight (where, (op->weight + op->carrying));	1684	add_weight (this, (op->weight + op->carrying));
1833		1685
1834	otmp = is_player_inv (where);	1686	otmp = this->in_player ();
1835	if (otmp && otmp->contr != NULL)	1687	if (otmp && otmp->contr)
1836	{
1837	if (!QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))	1688	if (!QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
1838	fix_player (otmp);	1689	otmp->update_stats ();
1839	}
1840		1690
1841	op->map = NULL;	1691	op->map = 0;
1842	op->env = where;	1692	op->env = this;
1843	op->above = NULL;	1693	op->above = 0;
1844	op->below = NULL;	1694	op->below = 0;
1845	op->x = 0, op->y = 0;	1695	op->x = 0, op->y = 0;
1846		1696
1847	/* reset the light list and los of the players on the map */	1697	/* reset the light list and los of the players on the map */
1848	if ((op->glow_radius != 0) && where->map)	1698	if ((op->glow_radius != 0) && map)
1849	{	1699	{
1850	#ifdef DEBUG_LIGHTS	1700	#ifdef DEBUG_LIGHTS
1851	LOG (llevDebug, " insert_ob_in_ob(): got %s to insert in map/op\n", op->name);	1701	LOG (llevDebug, " insert_ob_in_ob(): got %s to insert in map/op\n", op->name);
1852	#endif /* DEBUG_LIGHTS */	1702	#endif /* DEBUG_LIGHTS */
1853	if (MAP_DARKNESS (where->map))	1703	if (map->darkness)
1854	update_all_los (where->map, where->x, where->y);	1704	update_all_los (map, x, y);
1855	}	1705	}
1856		1706
1857	/* Client has no idea of ordering so lets not bother ordering it here.	1707	/* Client has no idea of ordering so lets not bother ordering it here.
1858	* It sure simplifies this function...	1708	* It sure simplifies this function...
1859	*/	1709	*/
1860	if (where->inv == NULL)	1710	if (!inv)
1861	where->inv = op;	1711	inv = op;
1862	else	1712	else
1863	{	1713	{
1864	op->below = where->inv;	1714	op->below = inv;
1865	op->below->above = op;	1715	op->below->above = op;
1866	where->inv = op;	1716	inv = op;
1867	}	1717	}
		1718
		1719	INVOKE_OBJECT (INSERT, this);
		1720
1868	return op;	1721	return op;
1869	}	1722	}
1870		1723
1871	/*	1724	/*
1872	* Checks if any objects has a move_type that matches objects	1725	* Checks if any objects has a move_type that matches objects
…		…
1923		1776
1924	/* The objects have to be checked from top to bottom.	1777	/* The objects have to be checked from top to bottom.
1925	* Hence, we first go to the top:	1778	* Hence, we first go to the top:
1926	*/	1779	*/
1927		1780
1928	for (tmp = GET_MAP_OB (op->map, op->x, op->y); tmp != NULL && tmp->above != NULL; tmp = tmp->above)	1781	for (tmp = GET_MAP_OB (op->map, op->x, op->y); tmp && tmp->above; tmp = tmp->above)
1929	{	1782	{
1930	/* Trim the search when we find the first other spell effect	1783	/* Trim the search when we find the first other spell effect
1931	* this helps performance so that if a space has 50 spell objects,	1784	* this helps performance so that if a space has 50 spell objects,
1932	* we don't need to check all of them.	1785	* we don't need to check all of them.
1933	*/	1786	*/
…		…
1988	/*	1841	/*
1989	* present_arch(arch, map, x, y) searches for any objects with	1842	* present_arch(arch, map, x, y) searches for any objects with
1990	* a matching archetype at the given map and coordinates.	1843	* a matching archetype at the given map and coordinates.
1991	* The first matching object is returned, or NULL if none.	1844	* The first matching object is returned, or NULL if none.
1992	*/	1845	*/
1993
1994	object *	1846	object *
1995	present_arch (const archetype *at, maptile *m, int x, int y)	1847	present_arch (const archetype *at, maptile *m, int x, int y)
1996	{	1848	{
1997	object *
1998	tmp;
1999
2000	if (m == NULL || out_of_map (m, x, y))	1849	if (m == NULL || out_of_map (m, x, y))
2001	{	1850	{
2002	LOG (llevError, "Present_arch called outside map.\n");	1851	LOG (llevError, "Present_arch called outside map.\n");
2003	return NULL;	1852	return NULL;
2004	}	1853	}
		1854
2005	for (tmp = GET_MAP_OB (m, x, y); tmp != NULL; tmp = tmp->above)	1855	for (object *tmp = GET_MAP_OB (m, x, y); tmp != NULL; tmp = tmp->above)
2006	if (tmp->arch == at)	1856	if (tmp->arch == at)
2007	return tmp;	1857	return tmp;
		1858
2008	return NULL;	1859	return NULL;
2009	}	1860	}
2010		1861
2011	/*	1862	/*
2012	* present(type, map, x, y) searches for any objects with	1863	* present(type, map, x, y) searches for any objects with
2013	* a matching type variable at the given map and coordinates.	1864	* a matching type variable at the given map and coordinates.
2014	* The first matching object is returned, or NULL if none.	1865	* The first matching object is returned, or NULL if none.
2015	*/	1866	*/
2016
2017	object *	1867	object *
2018	present (unsigned char type, maptile *m, int x, int y)	1868	present (unsigned char type, maptile *m, int x, int y)
2019	{	1869	{
2020	object *
2021	tmp;
2022
2023	if (out_of_map (m, x, y))	1870	if (out_of_map (m, x, y))
2024	{	1871	{
2025	LOG (llevError, "Present called outside map.\n");	1872	LOG (llevError, "Present called outside map.\n");
2026	return NULL;	1873	return NULL;
2027	}	1874	}
		1875
2028	for (tmp = GET_MAP_OB (m, x, y); tmp != NULL; tmp = tmp->above)	1876	for (object *tmp = GET_MAP_OB (m, x, y); tmp != NULL; tmp = tmp->above)
2029	if (tmp->type == type)	1877	if (tmp->type == type)
2030	return tmp;	1878	return tmp;
		1879
2031	return NULL;	1880	return NULL;
2032	}	1881	}
2033		1882
2034	/*	1883	/*
2035	* present_in_ob(type, object) searches for any objects with	1884	* present_in_ob(type, object) searches for any objects with
2036	* a matching type variable in the inventory of the given object.	1885	* a matching type variable in the inventory of the given object.
2037	* The first matching object is returned, or NULL if none.	1886	* The first matching object is returned, or NULL if none.
2038	*/	1887	*/
2039
2040	object *	1888	object *
2041	present_in_ob (unsigned char type, const object *op)	1889	present_in_ob (unsigned char type, const object *op)
2042	{	1890	{
2043	object *
2044	tmp;
2045
2046	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)	1891	for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below)
2047	if (tmp->type == type)	1892	if (tmp->type == type)
2048	return tmp;	1893	return tmp;
		1894
2049	return NULL;	1895	return NULL;
2050	}	1896	}
2051		1897
2052	/*	1898	/*
2053	* present_in_ob (type, str, object) searches for any objects with	1899	* present_in_ob (type, str, object) searches for any objects with
…		…
2061	* str is the string to match against. Note that we match against	1907	* str is the string to match against. Note that we match against
2062	* the object name, not the archetype name. this is so that the	1908	* the object name, not the archetype name. this is so that the
2063	* spell code can use one object type (force), but change it's name	1909	* spell code can use one object type (force), but change it's name
2064	* to be unique.	1910	* to be unique.
2065	*/	1911	*/
2066
2067	object *	1912	object *
2068	present_in_ob_by_name (int type, const char *str, const object *op)	1913	present_in_ob_by_name (int type, const char *str, const object *op)
2069	{	1914	{
2070	object *
2071	tmp;
2072
2073	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)	1915	for (object *tmp = op->inv; tmp; tmp = tmp->below)
2074	{
2075	if ((type == -1 || tmp->type == type) && (!strcmp (str, tmp->name)))	1916	if ((type == -1 || tmp->type == type) && (!strcmp (str, tmp->name)))
2076	return tmp;	1917	return tmp;
2077	}	1918
2078	return NULL;	1919	return 0;
2079	}	1920	}
2080		1921
2081	/*	1922	/*
2082	* present_arch_in_ob(archetype, object) searches for any objects with	1923	* present_arch_in_ob(archetype, object) searches for any objects with
2083	* a matching archetype in the inventory of the given object.	1924	* a matching archetype in the inventory of the given object.
2084	* The first matching object is returned, or NULL if none.	1925	* The first matching object is returned, or NULL if none.
2085	*/	1926	*/
2086
2087	object *	1927	object *
2088	present_arch_in_ob (const archetype *at, const object *op)	1928	present_arch_in_ob (const archetype *at, const object *op)
2089	{	1929	{
2090	object *
2091	tmp;
2092
2093	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)	1930	for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below)
2094	if (tmp->arch == at)	1931	if (tmp->arch == at)
2095	return tmp;	1932	return tmp;
		1933
2096	return NULL;	1934	return NULL;
2097	}	1935	}
2098		1936
2099	/*	1937	/*
2100	* activate recursively a flag on an object inventory	1938	* activate recursively a flag on an object inventory
2101	*/	1939	*/
2102	void	1940	void
2103	flag_inv (object *op, int flag)	1941	flag_inv (object *op, int flag)
2104	{	1942	{
2105	object *
2106	tmp;
2107
2108	if (op->inv)	1943	if (op->inv)
2109	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)	1944	for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below)
2110	{	1945	{
2111	SET_FLAG (tmp, flag);	1946	SET_FLAG (tmp, flag);
2112	flag_inv (tmp, flag);	1947	flag_inv (tmp, flag);
2113	}	1948	}
2114	} /*	1949	}
		1950
		1951	/*
2115	* desactivate recursively a flag on an object inventory	1952	* deactivate recursively a flag on an object inventory
2116	*/	1953	*/
2117	void	1954	void
2118	unflag_inv (object *op, int flag)	1955	unflag_inv (object *op, int flag)
2119	{	1956	{
2120	object *
2121	tmp;
2122
2123	if (op->inv)	1957	if (op->inv)
2124	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)	1958	for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below)
2125	{	1959	{
2126	CLEAR_FLAG (tmp, flag);	1960	CLEAR_FLAG (tmp, flag);
2127	unflag_inv (tmp, flag);	1961	unflag_inv (tmp, flag);
2128	}	1962	}
2129	}	1963	}
…		…
2132	* set_cheat(object) sets the cheat flag (WAS_WIZ) in the object and in	1966	* set_cheat(object) sets the cheat flag (WAS_WIZ) in the object and in
2133	* all it's inventory (recursively).	1967	* all it's inventory (recursively).
2134	* If checksums are used, a player will get set_cheat called for	1968	* If checksums are used, a player will get set_cheat called for
2135	* him/her-self and all object carried by a call to this function.	1969	* him/her-self and all object carried by a call to this function.
2136	*/	1970	*/
2137
2138	void	1971	void
2139	set_cheat (object *op)	1972	set_cheat (object *op)
2140	{	1973	{
2141	SET_FLAG (op, FLAG_WAS_WIZ);	1974	SET_FLAG (op, FLAG_WAS_WIZ);
2142	flag_inv (op, FLAG_WAS_WIZ);	1975	flag_inv (op, FLAG_WAS_WIZ);
…		…
2161	* because arch_blocked (now ob_blocked) needs to know the movement type	1994	* because arch_blocked (now ob_blocked) needs to know the movement type
2162	* to know if the space in question will block the object. We can't use	1995	* to know if the space in question will block the object. We can't use
2163	* the archetype because that isn't correct if the monster has been	1996	* the archetype because that isn't correct if the monster has been
2164	* customized, changed states, etc.	1997	* customized, changed states, etc.
2165	*/	1998	*/
2166
2167	int	1999	int
2168	find_free_spot (const object *ob, maptile *m, int x, int y, int start, int stop)	2000	find_free_spot (const object *ob, maptile *m, int x, int y, int start, int stop)
2169	{	2001	{
2170	int
2171	i,
2172	index = 0, flag;	2002	int index = 0, flag;
2173	static int
2174	altern[SIZEOFFREE];	2003	int altern[SIZEOFFREE];
2175		2004
2176	for (i = start; i < stop; i++)	2005	for (int i = start; i < stop; i++)
2177	{	2006	{
2178	flag = ob_blocked (ob, m, x + freearr_x[i], y + freearr_y[i]);	2007	flag = ob_blocked (ob, m, x + freearr_x[i], y + freearr_y[i]);
2179	if (!flag)	2008	if (!flag)
2180	altern[index++] = i;	2009	altern [index++] = i;
2181		2010
2182	/* Basically, if we find a wall on a space, we cut down the search size.	2011	/* Basically, if we find a wall on a space, we cut down the search size.
2183	* In this way, we won't return spaces that are on another side of a wall.	2012	* In this way, we won't return spaces that are on another side of a wall.
2184	* This mostly work, but it cuts down the search size in all directions -	2013	* This mostly work, but it cuts down the search size in all directions -
2185	* if the space being examined only has a wall to the north and empty	2014	* if the space being examined only has a wall to the north and empty
2186	* spaces in all the other directions, this will reduce the search space	2015	* spaces in all the other directions, this will reduce the search space
2187	* to only the spaces immediately surrounding the target area, and	2016	* to only the spaces immediately surrounding the target area, and
2188	* won't look 2 spaces south of the target space.	2017	* won't look 2 spaces south of the target space.
2189	*/	2018	*/
2190	else if ((flag & AB_NO_PASS) && maxfree[i] < stop)	2019	else if ((flag & P_NO_PASS) && maxfree[i] < stop)
2191	stop = maxfree[i];	2020	stop = maxfree[i];
2192	}	2021	}
		2022
2193	if (!index)	2023	if (!index)
2194	return -1;	2024	return -1;
		2025
2195	return altern[RANDOM () % index];	2026	return altern[RANDOM () % index];
2196	}	2027	}
2197		2028
2198	/*	2029	/*
2199	* find_first_free_spot(archetype, maptile, x, y) works like	2030	* find_first_free_spot(archetype, maptile, x, y) works like
2200	* find_free_spot(), but it will search max number of squares.	2031	* find_free_spot(), but it will search max number of squares.
2201	* But it will return the first available spot, not a random choice.	2032	* But it will return the first available spot, not a random choice.
2202	* Changed 0.93.2: Have it return -1 if there is no free spot available.	2033	* Changed 0.93.2: Have it return -1 if there is no free spot available.
2203	*/	2034	*/
2204
2205	int	2035	int
2206	find_first_free_spot (const object *ob, maptile *m, int x, int y)	2036	find_first_free_spot (const object *ob, maptile *m, int x, int y)
2207	{	2037	{
2208	int
2209	i;
2210
2211	for (i = 0; i < SIZEOFFREE; i++)	2038	for (int i = 0; i < SIZEOFFREE; i++)
2212	{
2213	if (!ob_blocked (ob, m, x + freearr_x[i], y + freearr_y[i]))	2039	if (!ob_blocked (ob, m, x + freearr_x[i], y + freearr_y[i]))
2214	return i;	2040	return i;
2215	}	2041
2216	return -1;	2042	return -1;
2217	}	2043	}
2218		2044
2219	/*	2045	/*
2220	* The function permute(arr, begin, end) randomly reorders the array	2046	* The function permute(arr, begin, end) randomly reorders the array
2221	* arr[begin..end-1].	2047	* arr[begin..end-1].
		2048	* now uses a fisher-yates shuffle, old permute was broken
2222	*/	2049	*/
2223	static void	2050	static void
2224	permute (int *arr, int begin, int end)	2051	permute (int *arr, int begin, int end)
2225	{	2052	{
2226	int	2053	arr += begin;
2227	i,
2228	j,
2229	tmp,
2230	len;
2231
2232	len = end - begin;	2054	end -= begin;
2233	for (i = begin; i < end; i++)
2234	{
2235	j = begin + RANDOM () % len;
2236		2055
2237	tmp = arr[i];	2056	while (--end)
2238	arr[i] = arr[j];	2057	swap (arr [end], arr [RANDOM () % (end + 1)]);
2239	arr[j] = tmp;
2240	}
2241	}	2058	}
2242		2059
2243	/* new function to make monster searching more efficient, and effective!	2060	/* new function to make monster searching more efficient, and effective!
2244	* This basically returns a randomized array (in the passed pointer) of	2061	* This basically returns a randomized array (in the passed pointer) of
2245	* the spaces to find monsters. In this way, it won't always look for	2062	* the spaces to find monsters. In this way, it won't always look for
…		…
2248	* the 3x3 area will be searched, just not in a predictable order.	2065	* the 3x3 area will be searched, just not in a predictable order.
2249	*/	2066	*/
2250	void	2067	void
2251	get_search_arr (int *search_arr)	2068	get_search_arr (int *search_arr)
2252	{	2069	{
2253	int	2070	int i;
2254	i;
2255		2071
2256	for (i = 0; i < SIZEOFFREE; i++)	2072	for (i = 0; i < SIZEOFFREE; i++)
2257	{
2258	search_arr[i] = i;	2073	search_arr[i] = i;
2259	}
2260		2074
2261	permute (search_arr, 1, SIZEOFFREE1 + 1);	2075	permute (search_arr, 1, SIZEOFFREE1 + 1);
2262	permute (search_arr, SIZEOFFREE1 + 1, SIZEOFFREE2 + 1);	2076	permute (search_arr, SIZEOFFREE1 + 1, SIZEOFFREE2 + 1);
2263	permute (search_arr, SIZEOFFREE2 + 1, SIZEOFFREE);	2077	permute (search_arr, SIZEOFFREE2 + 1, SIZEOFFREE);
2264	}	2078	}
…		…
2273	* Perhaps incorrectly, but I'm making the assumption that exclude	2087	* Perhaps incorrectly, but I'm making the assumption that exclude
2274	* is actually want is going to try and move there. We need this info	2088	* is actually want is going to try and move there. We need this info
2275	* because we have to know what movement the thing looking to move	2089	* because we have to know what movement the thing looking to move
2276	* there is capable of.	2090	* there is capable of.
2277	*/	2091	*/
2278
2279	int	2092	int
2280	find_dir (maptile *m, int x, int y, object *exclude)	2093	find_dir (maptile *m, int x, int y, object *exclude)
2281	{	2094	{
2282	int
2283	i,
2284	max = SIZEOFFREE, mflags;	2095	int i, max = SIZEOFFREE, mflags;
2285		2096
2286	sint16 nx, ny;	2097	sint16 nx, ny;
2287	object *	2098	object *tmp;
2288	tmp;
2289	maptile *	2099	maptile *mp;
2290	mp;
2291		2100
2292	MoveType blocked, move_type;	2101	MoveType blocked, move_type;
2293		2102
2294	if (exclude && exclude->head)	2103	if (exclude && exclude->head)
2295	{	2104	{
…		…
2307	mp = m;	2116	mp = m;
2308	nx = x + freearr_x[i];	2117	nx = x + freearr_x[i];
2309	ny = y + freearr_y[i];	2118	ny = y + freearr_y[i];
2310		2119
2311	mflags = get_map_flags (m, &mp, nx, ny, &nx, &ny);	2120	mflags = get_map_flags (m, &mp, nx, ny, &nx, &ny);
		2121
2312	if (mflags & P_OUT_OF_MAP)	2122	if (mflags & P_OUT_OF_MAP)
2313	{
2314	max = maxfree[i];	2123	max = maxfree[i];
2315	}
2316	else	2124	else
2317	{	2125	{
2318	blocked = GET_MAP_MOVE_BLOCK (mp, nx, ny);	2126	mapspace &ms = mp->at (nx, ny);
		2127
		2128	blocked = ms.move_block;
2319		2129
2320	if ((move_type & blocked) == move_type)	2130	if ((move_type & blocked) == move_type)
2321	{
2322	max = maxfree[i];	2131	max = maxfree[i];
2323	}
2324	else if (mflags & P_IS_ALIVE)	2132	else if (mflags & P_IS_ALIVE)
2325	{	2133	{
2326	for (tmp = GET_MAP_OB (mp, nx, ny); tmp != NULL; tmp = tmp->above)	2134	for (tmp = ms.bot; tmp; tmp = tmp->above)
2327	{	2135	if ((tmp->flag [FLAG_MONSTER] || tmp->type == PLAYER)
2328	if ((QUERY_FLAG (tmp, FLAG_MONSTER) || tmp->type == PLAYER) && (tmp != exclude || (tmp->head && tmp->head != exclude)))	2136	&& (tmp != exclude || (tmp->head && tmp->head != exclude)))
2329	{
2330	break;	2137	break;
2331	}	2138
2332	}
2333	if (tmp)	2139	if (tmp)
2334	{
2335	return freedir[i];	2140	return freedir[i];
2336	}
2337	}	2141	}
2338	}	2142	}
2339	}	2143	}
		2144
2340	return 0;	2145	return 0;
2341	}	2146	}
2342		2147
2343	/*	2148	/*
2344	* distance(object 1, object 2) will return the square of the	2149	* distance(object 1, object 2) will return the square of the
2345	* distance between the two given objects.	2150	* distance between the two given objects.
2346	*/	2151	*/
2347
2348	int	2152	int
2349	distance (const object *ob1, const object *ob2)	2153	distance (const object *ob1, const object *ob2)
2350	{	2154	{
2351	int
2352	i;
2353
2354	i = (ob1->x - ob2->x) * (ob1->x - ob2->x) + (ob1->y - ob2->y) * (ob1->y - ob2->y);	2155	return (ob1->x - ob2->x) * (ob1->x - ob2->x) + (ob1->y - ob2->y) * (ob1->y - ob2->y);
2355	return i;
2356	}	2156	}
2357		2157
2358	/*	2158	/*
2359	* find_dir_2(delta-x,delta-y) will return a direction in which	2159	* find_dir_2(delta-x,delta-y) will return a direction in which
2360	* an object which has subtracted the x and y coordinates of another	2160	* an object which has subtracted the x and y coordinates of another
2361	* object, needs to travel toward it.	2161	* object, needs to travel toward it.
2362	*/	2162	*/
2363
2364	int	2163	int
2365	find_dir_2 (int x, int y)	2164	find_dir_2 (int x, int y)
2366	{	2165	{
2367	int	2166	int q;
2368	q;
2369		2167
2370	if (y)	2168	if (y)
2371	q = x * 100 / y;	2169	q = x * 100 / y;
2372	else if (x)	2170	else if (x)
2373	q = -300 * x;	2171	q = -300 * x;
…		…
2408	int	2206	int
2409	absdir (int d)	2207	absdir (int d)
2410	{	2208	{
2411	while (d < 1)	2209	while (d < 1)
2412	d += 8;	2210	d += 8;
		2211
2413	while (d > 8)	2212	while (d > 8)
2414	d -= 8;	2213	d -= 8;
		2214
2415	return d;	2215	return d;
2416	}	2216	}
2417		2217
2418	/*	2218	/*
2419	* dirdiff(dir1, dir2) returns how many 45-degrees differences there is	2219	* dirdiff(dir1, dir2) returns how many 45-degrees differences there is
…		…
2421	*/	2221	*/
2422		2222
2423	int	2223	int
2424	dirdiff (int dir1, int dir2)	2224	dirdiff (int dir1, int dir2)
2425	{	2225	{
2426	int	2226	int d;
2427	d;
2428		2227
2429	d = abs (dir1 - dir2);	2228	d = abs (dir1 - dir2);
2430	if (d > 4)	2229	if (d > 4)
2431	d = 8 - d;	2230	d = 8 - d;
		2231
2432	return d;	2232	return d;
2433	}	2233	}
2434		2234
2435	/* peterm:	2235	/* peterm:
2436	* do LOS stuff for ball lightning. Go after the closest VISIBLE monster.	2236	* do LOS stuff for ball lightning. Go after the closest VISIBLE monster.
…		…
2439	* This basically means that if direction is 15, then it could either go	2239	* This basically means that if direction is 15, then it could either go
2440	* direction 4, 14, or 16 to get back to where we are.	2240	* direction 4, 14, or 16 to get back to where we are.
2441	* Moved from spell_util.c to object.c with the other related direction	2241	* Moved from spell_util.c to object.c with the other related direction
2442	* functions.	2242	* functions.
2443	*/	2243	*/
2444
2445	int
2446	reduction_dir[SIZEOFFREE][3] = {	2244	int reduction_dir[SIZEOFFREE][3] = {
2447	{0, 0, 0}, /* 0 */	2245	{0, 0, 0}, /* 0 */
2448	{0, 0, 0}, /* 1 */	2246	{0, 0, 0}, /* 1 */
2449	{0, 0, 0}, /* 2 */	2247	{0, 0, 0}, /* 2 */
2450	{0, 0, 0}, /* 3 */	2248	{0, 0, 0}, /* 3 */
2451	{0, 0, 0}, /* 4 */	2249	{0, 0, 0}, /* 4 */
…		…
2499	* find a path to that monster that we found. If not,	2297	* find a path to that monster that we found. If not,
2500	* we don't bother going toward it. Returns 1 if we	2298	* we don't bother going toward it. Returns 1 if we
2501	* can see a direct way to get it	2299	* can see a direct way to get it
2502	* Modified to be map tile aware -.MSW	2300	* Modified to be map tile aware -.MSW
2503	*/	2301	*/
2504
2505
2506	int	2302	int
2507	can_see_monsterP (maptile *m, int x, int y, int dir)	2303	can_see_monsterP (maptile *m, int x, int y, int dir)
2508	{	2304	{
2509	sint16 dx, dy;	2305	sint16 dx, dy;
2510	int
2511	mflags;	2306	int mflags;
2512		2307
2513	if (dir < 0)	2308	if (dir < 0)
2514	return 0; /* exit condition: invalid direction */	2309	return 0; /* exit condition: invalid direction */
2515		2310
2516	dx = x + freearr_x[dir];	2311	dx = x + freearr_x[dir];
…		…
2529	return 0;	2324	return 0;
2530		2325
2531	/* yes, can see. */	2326	/* yes, can see. */
2532	if (dir < 9)	2327	if (dir < 9)
2533	return 1;	2328	return 1;
		2329
2534	return can_see_monsterP (m, x, y, reduction_dir[dir][0]) |	2330	return can_see_monsterP (m, x, y, reduction_dir[dir][0])
2535	can_see_monsterP (m, x, y, reduction_dir[dir][1]) | can_see_monsterP (m, x, y, reduction_dir[dir][2]);	2331	| can_see_monsterP (m, x, y, reduction_dir[dir][1])
		2332	| can_see_monsterP (m, x, y, reduction_dir[dir][2]);
2536	}	2333	}
2537
2538
2539		2334
2540	/*	2335	/*
2541	* can_pick(picker, item): finds out if an object is possible to be	2336	* can_pick(picker, item): finds out if an object is possible to be
2542	* picked up by the picker. Returnes 1 if it can be	2337	* picked up by the picker. Returnes 1 if it can be
2543	* picked up, otherwise 0.	2338	* picked up, otherwise 0.
…		…
2554	return /*QUERY_FLAG(who,FLAG_WIZ)|| */	2349	return /*QUERY_FLAG(who,FLAG_WIZ)|| */
2555	(item->weight > 0 && !QUERY_FLAG (item, FLAG_NO_PICK) &&	2350	(item->weight > 0 && !QUERY_FLAG (item, FLAG_NO_PICK) &&
2556	!QUERY_FLAG (item, FLAG_ALIVE) && !item->invisible && (who->type == PLAYER || item->weight < who->weight / 3));	2351	!QUERY_FLAG (item, FLAG_ALIVE) && !item->invisible && (who->type == PLAYER || item->weight < who->weight / 3));
2557	}	2352	}
2558		2353
2559
2560	/*	2354	/*
2561	* create clone from object to another	2355	* create clone from object to another
2562	*/	2356	*/
2563	object *	2357	object *
2564	object_create_clone (object *asrc)	2358	object_create_clone (object *asrc)
2565	{	2359	{
2566	object *
2567	dst = NULL, *tmp, *src, *part, *prev, *item;	2360	object *dst = 0, *tmp, *src, *part, *prev, *item;
2568		2361
2569	if (!asrc)	2362	if (!asrc)
2570	return NULL;	2363	return 0;
		2364
2571	src = asrc;	2365	src = asrc;
2572	if (src->head)	2366	if (src->head)
2573	src = src->head;	2367	src = src->head;
2574		2368
2575	prev = NULL;	2369	prev = 0;
2576	for (part = src; part; part = part->more)	2370	for (part = src; part; part = part->more)
2577	{	2371	{
2578	tmp = get_object ();	2372	tmp = part->clone ();
2579	copy_object (part, tmp);
2580	tmp->x -= src->x;	2373	tmp->x -= src->x;
2581	tmp->y -= src->y;	2374	tmp->y -= src->y;
		2375
2582	if (!part->head)	2376	if (!part->head)
2583	{	2377	{
2584	dst = tmp;	2378	dst = tmp;
2585	tmp->head = NULL;	2379	tmp->head = 0;
2586	}	2380	}
2587	else	2381	else
2588	{
2589	tmp->head = dst;	2382	tmp->head = dst;
2590	}	2383
2591	tmp->more = NULL;	2384	tmp->more = 0;
		2385
2592	if (prev)	2386	if (prev)
2593	prev->more = tmp;	2387	prev->more = tmp;
		2388
2594	prev = tmp;	2389	prev = tmp;
2595	}	2390	}
2596		2391
2597	for (item = src->inv; item; item = item->below)	2392	for (item = src->inv; item; item = item->below)
2598	insert_ob_in_ob (object_create_clone (item), dst);	2393	insert_ob_in_ob (object_create_clone (item), dst);
…		…
2604	/* Basically, we save the content of the string to a temp file, then call */	2399	/* Basically, we save the content of the string to a temp file, then call */
2605	/* load_object on it. I admit it is a highly inefficient way to make things, */	2400	/* load_object on it. I admit it is a highly inefficient way to make things, */
2606	/* but it was simple to make and allows reusing the load_object function. */	2401	/* but it was simple to make and allows reusing the load_object function. */
2607	/* Remember not to use load_object_str in a time-critical situation. */	2402	/* Remember not to use load_object_str in a time-critical situation. */
2608	/* Also remember that multiparts objects are not supported for now. */	2403	/* Also remember that multiparts objects are not supported for now. */
2609
2610	object *	2404	object *
2611	load_object_str (const char *obstr)	2405	load_object_str (const char *obstr)
2612	{	2406	{
2613	object *op;	2407	object *op;
2614	char filename[MAX_BUF];	2408	char filename[MAX_BUF];
…		…
2624	}	2418	}
2625		2419
2626	fprintf (tempfile, obstr);	2420	fprintf (tempfile, obstr);
2627	fclose (tempfile);	2421	fclose (tempfile);
2628		2422
2629	op = get_object ();	2423	op = object::create ();
2630		2424
2631	object_thawer thawer (filename);	2425	object_thawer thawer (filename);
2632		2426
2633	if (thawer)	2427	if (thawer)
2634	load_object (thawer, op, 0);	2428	load_object (thawer, op, 0);
…		…
2644	* returns NULL if no match.	2438	* returns NULL if no match.
2645	*/	2439	*/
2646	object *	2440	object *
2647	find_obj_by_type_subtype (const object *who, int type, int subtype)	2441	find_obj_by_type_subtype (const object *who, int type, int subtype)
2648	{	2442	{
2649	object *tmp;
2650
2651	for (tmp = who->inv; tmp; tmp = tmp->below)	2443	for (object *tmp = who->inv; tmp; tmp = tmp->below)
2652	if (tmp->type == type && tmp->subtype == subtype)	2444	if (tmp->type == type && tmp->subtype == subtype)
2653	return tmp;	2445	return tmp;
2654		2446
2655	return NULL;	2447	return 0;
2656	}	2448	}
2657		2449
2658	/* If ob has a field named key, return the link from the list,	2450	/* If ob has a field named key, return the link from the list,
2659	* otherwise return NULL.	2451	* otherwise return NULL.
2660	*	2452	*
…		…
2662	* do the desired thing.	2454	* do the desired thing.
2663	*/	2455	*/
2664	key_value *	2456	key_value *
2665	get_ob_key_link (const object *ob, const char *key)	2457	get_ob_key_link (const object *ob, const char *key)
2666	{	2458	{
2667	key_value *link;
2668
2669	for (link = ob->key_values; link != NULL; link = link->next)	2459	for (key_value *link = ob->key_values; link; link = link->next)
2670	if (link->key == key)	2460	if (link->key == key)
2671	return link;	2461	return link;
2672		2462
2673	return NULL;	2463	return 0;
2674	}	2464	}
2675		2465
2676	/*	2466	/*
2677	* Returns the value of op has an extra_field for key, or NULL.	2467	* Returns the value of op has an extra_field for key, or NULL.
2678	*	2468	*
…		…
2718	* Returns TRUE on success.	2508	* Returns TRUE on success.
2719	*/	2509	*/
2720	int	2510	int
2721	set_ob_key_value_s (object *op, const shstr & canonical_key, const char *value, int add_key)	2511	set_ob_key_value_s (object *op, const shstr & canonical_key, const char *value, int add_key)
2722	{	2512	{
2723	key_value *
2724	field = NULL, *last = NULL;	2513	key_value *field = NULL, *last = NULL;
2725		2514
2726	for (field = op->key_values; field != NULL; field = field->next)	2515	for (field = op->key_values; field != NULL; field = field->next)
2727	{	2516	{
2728	if (field->key != canonical_key)	2517	if (field->key != canonical_key)
2729	{	2518	{
…		…
2757	/* IF we get here, key doesn't exist */	2546	/* IF we get here, key doesn't exist */
2758		2547
2759	/* No field, we'll have to add it. */	2548	/* No field, we'll have to add it. */
2760		2549
2761	if (!add_key)	2550	if (!add_key)
2762	{
2763	return FALSE;	2551	return FALSE;
2764	}	2552
2765	/* There isn't any good reason to store a null	2553	/* There isn't any good reason to store a null
2766	* value in the key/value list. If the archetype has	2554	* value in the key/value list. If the archetype has
2767	* this key, then we should also have it, so shouldn't	2555	* this key, then we should also have it, so shouldn't
2768	* be here. If user wants to store empty strings,	2556	* be here. If user wants to store empty strings,
2769	* should pass in ""	2557	* should pass in ""

Diff Legend

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