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.55 by root, Sun Dec 3 00:41:33 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
928	if (!freed_map)	821	if (!freed_map)
929	{	822	{
930	freed_map = get_empty_map (3, 3);	823	freed_map = new maptile;
		824
931	freed_map->name = "freed objects map";	825	freed_map->name = "/internal/freed_objects_map";
932	allocate_map (freed_map);	826	freed_map->width = 3;
		827	freed_map->height = 3;
		828
		829	freed_map->allocate ();
933	}	830	}
934		831
935	map = freed_map;	832	map = freed_map;
936	x = 1;	833	x = 1;
937	y = 1;	834	y = 1;
938	}	835	}
939		836
940	if (more)
941	{
942	more->destroy (destroy_inventory);
943	more = 0;	837	more = 0;
944	}	838	head = 0;
945		839	inv = 0;
946	if (inv)
947	{
948	/* Only if the space blocks everything do we not process -
949	* if some form of movement is allowed, let objects
950	* drop on that space.
951	*/
952	if (destroy_inventory || !map || map->in_memory != MAP_IN_MEMORY || GET_MAP_MOVE_BLOCK (map, x, y) == MOVE_ALL)
953	{
954	object *op = inv;
955
956	while (op)
957	{
958	object *tmp = op->below;
959	op->destroy (destroy_inventory);
960	op = tmp;
961	}
962	}
963	else
964	{ /* Put objects in inventory onto this space */
965	object *op = inv;
966
967	while (op)
968	{
969	object *tmp = op->below;
970
971	remove_ob (op);
972
973	if (QUERY_FLAG (op, FLAG_STARTEQUIP)
974	|| QUERY_FLAG (op, FLAG_NO_DROP) || op->type == RUNE || op->type == TRAP || QUERY_FLAG (op, FLAG_IS_A_TEMPLATE))
975	free_object (op);
976	else
977	{
978	op->x = x;
979	op->y = y;
980	insert_ob_in_map (op, map, 0, 0); /* Insert in same map as the envir */
981	}
982
983	op = tmp;
984	}
985	}
986	}
987		840
988	// clear those pointers that likely might have circular references to us	841	// clear those pointers that likely might have circular references to us
989	owner = 0;	842	owner = 0;
990	enemy = 0;	843	enemy = 0;
991	attacked_by = 0;	844	attacked_by = 0;
…		…
996	/* Remove object from the active list */	849	/* Remove object from the active list */
997	speed = 0;	850	speed = 0;
998	update_ob_speed (this);	851	update_ob_speed (this);
999		852
1000	unlink ();	853	unlink ();
		854	}
1001		855
1002	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 ();
1003	}	918	}
1004		919
1005	/*	920	/*
1006	* sub_weight() recursively (outwards) subtracts a number from the	921	* sub_weight() recursively (outwards) subtracts a number from the
1007	* 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)).
1008	*/	923	*/
1009
1010	void	924	void
1011	sub_weight (object *op, signed long weight)	925	sub_weight (object *op, signed long weight)
1012	{	926	{
1013	while (op != NULL)	927	while (op != NULL)
1014	{	928	{
…		…
1018	op->carrying -= weight;	932	op->carrying -= weight;
1019	op = op->env;	933	op = op->env;
1020	}	934	}
1021	}	935	}
1022		936
1023	/* remove_ob(op):	937	/* op->remove ():
1024	* 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
1025	* 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
1026	* object will have no environment. If the object previously had an	940	* object will have no environment. If the object previously had an
1027	* environment, the x and y coordinates will be updated to	941	* environment, the x and y coordinates will be updated to
1028	* the previous environment.	942	* the previous environment.
1029	* Beware: This function is called from the editor as well!	943	* Beware: This function is called from the editor as well!
1030	*/	944	*/
1031
1032	void	945	void
1033	remove_ob (object *op)	946	object::remove ()
1034	{	947	{
1035	object *tmp, *last = 0;	948	object *tmp, *last = 0;
1036	object *otmp;	949	object *otmp;
1037		950
1038	int check_walk_off;
1039	maptile *m;
1040
1041	sint16 x, y;
1042
1043	if (QUERY_FLAG (op, FLAG_REMOVED))	951	if (QUERY_FLAG (this, FLAG_REMOVED))
1044	return;	952	return;
1045		953
1046	SET_FLAG (op, FLAG_REMOVED);	954	SET_FLAG (this, FLAG_REMOVED);
		955	INVOKE_OBJECT (REMOVE, this);
1047		956
1048	if (op->more != NULL)	957	if (more)
1049	remove_ob (op->more);	958	more->remove ();
1050		959
1051	/*	960	/*
1052	* In this case, the object to be removed is in someones	961	* In this case, the object to be removed is in someones
1053	* inventory.	962	* inventory.
1054	*/	963	*/
1055	if (op->env != NULL)	964	if (env)
1056	{	965	{
1057	if (op->nrof)	966	if (nrof)
1058	sub_weight (op->env, op->weight * op->nrof);	967	sub_weight (env, weight * nrof);
1059	else	968	else
1060	sub_weight (op->env, op->weight + op->carrying);	969	sub_weight (env, weight + carrying);
1061		970
1062	/* 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
1063	* made to players inventory. If set, avoiding the call	972	* made to players inventory. If set, avoiding the call
1064	* to save cpu time.	973	* to save cpu time.
1065	*/	974	*/
1066	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))
1067	fix_player (otmp);	976	otmp->update_stats ();
1068		977
1069	if (op->above != NULL)	978	if (above != NULL)
1070	op->above->below = op->below;	979	above->below = below;
1071	else	980	else
1072	op->env->inv = op->below;	981	env->inv = below;
1073		982
1074	if (op->below != NULL)	983	if (below != NULL)
1075	op->below->above = op->above;	984	below->above = above;
1076		985
1077	/* we set up values so that it could be inserted into	986	/* we set up values so that it could be inserted into
1078	* 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
1079	* to the caller to decide what we want to do.	988	* to the caller to decide what we want to do.
1080	*/	989	*/
1081	op->x = op->env->x, op->y = op->env->y;	990	x = env->x, y = env->y;
1082	op->map = op->env->map;	991	map = env->map;
1083	op->above = NULL, op->below = NULL;	992	above = 0, below = 0;
1084	op->env = NULL;	993	env = 0;
1085	}	994	}
1086	else if (op->map)	995	else if (map)
1087	{	996	{
1088	x = op->x;
1089	y = op->y;
1090	m = get_map_from_coord (op->map, &x, &y);
1091
1092	if (!m)
1093	{
1094	LOG (llevError, "remove_ob called when object was on map but appears to not be within valid coordinates? %s (%d,%d)\n",
1095	op->map->path, op->x, op->y);
1096	/* in old days, we used to set x and y to 0 and continue.
1097	* it seems if we get into this case, something is probablye
1098	* screwed up and should be fixed.
1099	*/
1100	abort ();
1101	}
1102
1103	if (op->map != m)
1104	LOG (llevDebug, "remove_ob: Object not really on map it claimed to be on? %s != %s, %d,%d != %d,%d\n",
1105	op->map->path, m->path, op->x, op->y, x, y);
1106
1107	/* 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
1108	* lots of logic for things we no longer care about	998	* lots of logic for things we no longer care about
1109	*/	999	*/
1110		1000
1111	/* link the object above us */	1001	/* link the object above us */
1112	if (op->above)	1002	if (above)
1113	op->above->below = op->below;	1003	above->below = below;
1114	else	1004	else
1115	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 */
1116		1006
1117	/* Relink the object below us, if there is one */	1007	/* Relink the object below us, if there is one */
1118	if (op->below)	1008	if (below)
1119	op->below->above = op->above;	1009	below->above = above;
1120	else	1010	else
1121	{	1011	{
1122	/* 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
1123	* use translated coordinates in case some oddness with map tiling is	1013	* use translated coordinates in case some oddness with map tiling is
1124	* evident	1014	* evident
1125	*/	1015	*/
1126	if (GET_MAP_OB (m, x, y) != op)	1016	if (GET_MAP_OB (map, x, y) != this)
1127	{	1017	{
1128	char *dump = dump_object (op);	1018	char *dump = dump_object (this);
1129	LOG (llevError,	1019	LOG (llevError,
1130	"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);
1131	free (dump);	1021	free (dump);
1132	dump = dump_object (GET_MAP_OB (m, x, y));	1022	dump = dump_object (GET_MAP_OB (map, x, y));
1133	LOG (llevError, "%s\n", dump);	1023	LOG (llevError, "%s\n", dump);
1134	free (dump);	1024	free (dump);
1135	}	1025	}
1136		1026
1137	SET_MAP_OB (m, x, y, op->above); /* goes on above it. */	1027	map->at (x, y).bot = above; /* goes on above it. */
1138	}	1028	}
1139		1029
1140	op->above = 0;	1030	above = 0;
1141	op->below = 0;	1031	below = 0;
1142		1032
1143	if (op->map->in_memory == MAP_SAVING)	1033	if (map->in_memory == MAP_SAVING)
1144	return;	1034	return;
1145		1035
1146	check_walk_off = !QUERY_FLAG (op, FLAG_NO_APPLY);	1036	int check_walk_off = !flag [FLAG_NO_APPLY];
1147		1037
1148	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)
1149	{	1039	{
1150	/* 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
1151	* being removed.	1041	* being removed.
1152	*/	1042	*/
1153		1043
1154	if (tmp->type == PLAYER && tmp != op)	1044	if (tmp->type == PLAYER && tmp != this)
1155	{	1045	{
1156	/* If a container that the player is currently using somehow gets	1046	/* If a container that the player is currently using somehow gets
1157	* removed (most likely destroyed), update the player view	1047	* removed (most likely destroyed), update the player view
1158	* appropriately.	1048	* appropriately.
1159	*/	1049	*/
1160	if (tmp->container == op)	1050	if (tmp->container == this)
1161	{	1051	{
1162	CLEAR_FLAG (op, FLAG_APPLIED);	1052	flag [FLAG_APPLIED] = 0;
1163	tmp->container = NULL;	1053	tmp->container = 0;
1164	}	1054	}
1165		1055
1166	tmp->contr->socket.update_look = 1;	1056	if (tmp->contr->ns)
		1057	tmp->contr->ns->floorbox_update ();
1167	}	1058	}
1168		1059
1169	/* See if player moving off should effect something */	1060	/* See if player moving off should effect something */
1170	if (check_walk_off	1061	if (check_walk_off
1171	&& ((op->move_type & tmp->move_off)	1062	&& ((move_type & tmp->move_off)
1172	&& (op->move_type & ~tmp->move_off & ~tmp->move_block) == 0))	1063	&& (move_type & ~tmp->move_off & ~tmp->move_block) == 0))
1173	{	1064	{
1174	move_apply (tmp, op, NULL);	1065	move_apply (tmp, this, 0);
1175		1066
1176	if (op->destroyed ())	1067	if (destroyed ())
1177	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 ());
1178	}	1069	}
1179		1070
1180	/* Eneq(@csd.uu.se): Fixed this to skip tmp->above=tmp */	1071	/* Eneq(@csd.uu.se): Fixed this to skip tmp->above=tmp */
1181		1072
1182	if (tmp->above == tmp)	1073	if (tmp->above == tmp)
1183	tmp->above = NULL;	1074	tmp->above = 0;
1184		1075
1185	last = tmp;	1076	last = tmp;
1186	}	1077	}
1187		1078
1188	/* last == NULL of there are no objects on this space */	1079	/* last == NULL of there are no objects on this space */
1189	if (last == NULL)	1080	if (!last)
1190	{	1081	map->at (x, y).flags_ = P_NEED_UPDATE;
1191	/* set P_NEED_UPDATE, otherwise update_position will complain. In theory,
1192	* we could preserve the flags (GET_MAP_FLAGS), but update_position figures
1193	* those out anyways, and if there are any flags set right now, they won't
1194	* be correct anyways.
1195	*/
1196	SET_MAP_FLAGS (op->map, op->x, op->y, P_NEED_UPDATE);
1197	update_position (op->map, op->x, op->y);
1198	}
1199	else	1082	else
1200	update_object (last, UP_OBJ_REMOVE);	1083	update_object (last, UP_OBJ_REMOVE);
1201		1084
1202	if (QUERY_FLAG (op, FLAG_BLOCKSVIEW) || (op->glow_radius != 0))	1085	if (flag [FLAG_BLOCKSVIEW] || glow_radius)
1203	update_all_los (op->map, op->x, op->y);	1086	update_all_los (map, x, y);
1204	}	1087	}
1205	}	1088	}
1206		1089
1207	/*	1090	/*
1208	* merge_ob(op,top):	1091	* merge_ob(op,top):
…		…
1216	merge_ob (object *op, object *top)	1099	merge_ob (object *op, object *top)
1217	{	1100	{
1218	if (!op->nrof)	1101	if (!op->nrof)
1219	return 0;	1102	return 0;
1220		1103
1221	if (top == NULL)	1104	if (top)
1222	for (top = op; top != NULL && top->above != NULL; top = top->above);	1105	for (top = op; top && top->above; top = top->above)
		1106	;
1223		1107
1224	for (; top != NULL; top = top->below)	1108	for (; top; top = top->below)
1225	{	1109	{
1226	if (top == op)	1110	if (top == op)
1227	continue;	1111	continue;
1228	if (CAN_MERGE (op, top))	1112
		1113	if (object::can_merge (op, top))
1229	{	1114	{
1230	top->nrof += op->nrof;	1115	top->nrof += op->nrof;
1231		1116
1232	/* CLEAR_FLAG(top,FLAG_STARTEQUIP);*/	1117	/* CLEAR_FLAG(top,FLAG_STARTEQUIP);*/
1233	op->weight = 0; /* Don't want any adjustements now */	1118	op->weight = 0; /* Don't want any adjustements now */
1234	remove_ob (op);	1119	op->destroy ();
1235	free_object (op);
1236	return top;	1120	return top;
1237	}	1121	}
1238	}	1122	}
1239		1123
1240	return 0;	1124	return 0;
…		…
1279	* Return value:	1163	* Return value:
1280	* new object if 'op' was merged with other object	1164	* new object if 'op' was merged with other object
1281	* NULL if 'op' was destroyed	1165	* NULL if 'op' was destroyed
1282	* just 'op' otherwise	1166	* just 'op' otherwise
1283	*/	1167	*/
1284
1285	object *	1168	object *
1286	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)
1287	{	1170	{
1288	object *tmp, *top, *floor = NULL;	1171	object *tmp, *top, *floor = NULL;
1289	sint16 x, y;	1172	sint16 x, y;
…		…
1323	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);
1324	free (dump);	1207	free (dump);
1325	return op;	1208	return op;
1326	}	1209	}
1327		1210
1328	if (op->more != NULL)	1211	if (op->more)
1329	{	1212	{
1330	/* The part may be on a different map. */	1213	/* The part may be on a different map. */
1331		1214
1332	object *more = op->more;	1215	object *more = op->more;
1333		1216
…		…
1349	if (insert_ob_in_map (more, more->map, originator, flag) == NULL)	1232	if (insert_ob_in_map (more, more->map, originator, flag) == NULL)
1350	{	1233	{
1351	if (!op->head)	1234	if (!op->head)
1352	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");
1353		1236
1354	return NULL;	1237	return 0;
1355	}	1238	}
1356	}	1239	}
1357		1240
1358	CLEAR_FLAG (op, FLAG_REMOVED);	1241	CLEAR_FLAG (op, FLAG_REMOVED);
1359		1242
…		…
1367		1250
1368	/* this has to be done after we translate the coordinates.	1251	/* this has to be done after we translate the coordinates.
1369	*/	1252	*/
1370	if (op->nrof && !(flag & INS_NO_MERGE))	1253	if (op->nrof && !(flag & INS_NO_MERGE))
1371	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)
1372	if (CAN_MERGE (op, tmp))	1255	if (object::can_merge (op, tmp))
1373	{	1256	{
1374	op->nrof += tmp->nrof;	1257	op->nrof += tmp->nrof;
1375	remove_ob (tmp);	1258	tmp->destroy ();
1376	free_object (tmp);
1377	}	1259	}
1378		1260
1379	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 */
1380	CLEAR_FLAG (op, FLAG_INV_LOCKED);	1262	CLEAR_FLAG (op, FLAG_INV_LOCKED);
1381		1263
…		…
1394	op->below = originator->below;	1276	op->below = originator->below;
1395		1277
1396	if (op->below)	1278	if (op->below)
1397	op->below->above = op;	1279	op->below->above = op;
1398	else	1280	else
1399	SET_MAP_OB (op->map, op->x, op->y, op);	1281	op->ms ().bot = op;
1400		1282
1401	/* since *below* originator, no need to update top */	1283	/* since *below* originator, no need to update top */
1402	originator->below = op;	1284	originator->below = op;
1403	}	1285	}
1404	else	1286	else
…		…
1450	* If INS_ON_TOP is used, don't do this processing	1332	* If INS_ON_TOP is used, don't do this processing
1451	* Need to find the object that in fact blocks view, otherwise	1333	* Need to find the object that in fact blocks view, otherwise
1452	* stacking is a bit odd.	1334	* stacking is a bit odd.
1453	*/	1335	*/
1454	if (!(flag & INS_ON_TOP) &&	1336	if (!(flag & INS_ON_TOP) &&
1455	(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))
1456	{	1338	{
1457	for (last = top; last != floor; last = last->below)	1339	for (last = top; last != floor; last = last->below)
1458	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))	1340	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))
1459	break;	1341	break;
1460	/* Check to see if we found the object that blocks view,	1342	/* Check to see if we found the object that blocks view,
…		…
1483		1365
1484	if (op->above)	1366	if (op->above)
1485	op->above->below = op;	1367	op->above->below = op;
1486		1368
1487	op->below = NULL;	1369	op->below = NULL;
1488	SET_MAP_OB (op->map, op->x, op->y, op);	1370	op->ms ().bot = op;
1489	}	1371	}
1490	else	1372	else
1491	{ /* get inserted into the stack above top */	1373	{ /* get inserted into the stack above top */
1492	op->above = top->above;	1374	op->above = top->above;
1493		1375
…		…
1497	op->below = top;	1379	op->below = top;
1498	top->above = op;	1380	top->above = op;
1499	}	1381	}
1500		1382
1501	if (op->above == NULL)	1383	if (op->above == NULL)
1502	SET_MAP_TOP (op->map, op->x, op->y, op);	1384	op->ms ().top = op;
1503	} /* else not INS_BELOW_ORIGINATOR */	1385	} /* else not INS_BELOW_ORIGINATOR */
1504		1386
1505	if (op->type == PLAYER)	1387	if (op->type == PLAYER)
1506	op->contr->do_los = 1;	1388	op->contr->do_los = 1;
1507		1389
1508	/* 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
1509	* it, so save a few ticks and start from there.	1391	* it, so save a few ticks and start from there.
1510	*/	1392	*/
1511	if (!(flag & INS_MAP_LOAD))	1393	if (!(flag & INS_MAP_LOAD))
1512	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 ())
1513	if (tmp->type == PLAYER)	1395	if (pl->contr->ns)
1514	tmp->contr->socket.update_look = 1;	1396	pl->contr->ns->floorbox_update ();
1515		1397
1516	/* If this object glows, it may affect lighting conditions that are	1398	/* If this object glows, it may affect lighting conditions that are
1517	* 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
1518	* 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
1519	* on the map will get recalculated. The players could very well	1401	* on the map will get recalculated. The players could very well
1520	* 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 -
1521	* this should get redone to only look for players within range,	1403	* this should get redone to only look for players within range,
1522	* 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
1523	* of effect may be sufficient.	1405	* of effect may be sufficient.
1524	*/	1406	*/
1525	if (MAP_DARKNESS (op->map) && (op->glow_radius != 0))	1407	if (op->map->darkness && (op->glow_radius != 0))
1526	update_all_los (op->map, op->x, op->y);	1408	update_all_los (op->map, op->x, op->y);
1527		1409
1528	/* updates flags (blocked, alive, no magic, etc) for this map space */	1410	/* updates flags (blocked, alive, no magic, etc) for this map space */
1529	update_object (op, UP_OBJ_INSERT);	1411	update_object (op, UP_OBJ_INSERT);
1530		1412
		1413	INVOKE_OBJECT (INSERT, op);
		1414
1531	/* 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,
1532	* we want to have update_look set above before calling this.	1416	* we want to have floorbox_update called before calling this.
1533	*	1417	*
1534	* check_move_on() must be after this because code called from	1418	* check_move_on() must be after this because code called from
1535	* check_move_on() depends on correct map flags (so functions like	1419	* check_move_on() depends on correct map flags (so functions like
1536	* blocked() and wall() work properly), and these flags are updated by	1420	* blocked() and wall() work properly), and these flags are updated by
1537	* update_object().	1421	* update_object().
…		…
1539		1423
1540	/* 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 */
1541	if (!(flag & INS_NO_WALK_ON) && !op->head)	1425	if (!(flag & INS_NO_WALK_ON) && !op->head)
1542	{	1426	{
1543	if (check_move_on (op, originator))	1427	if (check_move_on (op, originator))
1544	return NULL;	1428	return 0;
1545		1429
1546	/* 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
1547	* walk on's.	1431	* walk on's.
1548	*/	1432	*/
1549	for (tmp = op->more; tmp != NULL; tmp = tmp->more)	1433	for (tmp = op->more; tmp != NULL; tmp = tmp->more)
1550	if (check_move_on (tmp, originator))	1434	if (check_move_on (tmp, originator))
1551	return NULL;	1435	return 0;
1552	}	1436	}
1553		1437
1554	return op;	1438	return op;
1555	}	1439	}
1556		1440
1557	/* this function inserts an object in the map, but if it	1441	/* this function inserts an object in the map, but if it
1558	* 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.
1559	* 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.
1560	*/	1444	*/
1561	void	1445	void
1562	replace_insert_ob_in_map (const char *arch_string, object *op)	1446	replace_insert_ob_in_map (const char *arch_string, object *op)
1563	{	1447	{
1564	object *	1448	object *tmp, *tmp1;
1565	tmp;
1566	object *
1567	tmp1;
1568		1449
1569	/* first search for itself and remove any old instances */	1450	/* first search for itself and remove any old instances */
1570		1451
1571	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)
1572	{
1573	if (!strcmp (tmp->arch->name, arch_string)) /* same archetype */	1453	if (!strcmp (tmp->arch->name, arch_string)) /* same archetype */
1574	{	1454	tmp->destroy ();
1575	remove_ob (tmp);
1576	free_object (tmp);
1577	}
1578	}
1579		1455
1580	tmp1 = arch_to_object (archetype::find (arch_string));	1456	tmp1 = arch_to_object (archetype::find (arch_string));
1581		1457
1582	tmp1->x = op->x;	1458	tmp1->x = op->x;
1583	tmp1->y = op->y;	1459	tmp1->y = op->y;
…		…
1593	*/	1469	*/
1594		1470
1595	object *	1471	object *
1596	get_split_ob (object *orig_ob, uint32 nr)	1472	get_split_ob (object *orig_ob, uint32 nr)
1597	{	1473	{
1598	object *	1474	object *newob;
1599	newob;
1600	int
1601	is_removed = (QUERY_FLAG (orig_ob, FLAG_REMOVED) != 0);	1475	int is_removed = (QUERY_FLAG (orig_ob, FLAG_REMOVED) != 0);
1602		1476
1603	if (orig_ob->nrof < nr)	1477	if (orig_ob->nrof < nr)
1604	{	1478	{
1605	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);
1606	return NULL;	1480	return NULL;
1607	}	1481	}
1608		1482
1609	newob = object_create_clone (orig_ob);	1483	newob = object_create_clone (orig_ob);
1610		1484
1611	if ((orig_ob->nrof -= nr) < 1)	1485	if ((orig_ob->nrof -= nr) < 1)
1612	{	1486	orig_ob->destroy (1);
1613	if (!is_removed)
1614	remove_ob (orig_ob);
1615	free_object2 (orig_ob, 1);
1616	}
1617	else if (!is_removed)	1487	else if (!is_removed)
1618	{	1488	{
1619	if (orig_ob->env != NULL)	1489	if (orig_ob->env != NULL)
1620	sub_weight (orig_ob->env, orig_ob->weight * nr);	1490	sub_weight (orig_ob->env, orig_ob->weight * nr);
1621	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)
…		…
1641		1511
1642	object *	1512	object *
1643	decrease_ob_nr (object *op, uint32 i)	1513	decrease_ob_nr (object *op, uint32 i)
1644	{	1514	{
1645	object *tmp;	1515	object *tmp;
1646	player *pl;
1647		1516
1648	if (i == 0) /* objects with op->nrof require this check */	1517	if (i == 0) /* objects with op->nrof require this check */
1649	return op;	1518	return op;
1650		1519
1651	if (i > op->nrof)	1520	if (i > op->nrof)
1652	i = op->nrof;	1521	i = op->nrof;
1653		1522
1654	if (QUERY_FLAG (op, FLAG_REMOVED))	1523	if (QUERY_FLAG (op, FLAG_REMOVED))
1655	op->nrof -= i;	1524	op->nrof -= i;
1656	else if (op->env != NULL)	1525	else if (op->env)
1657	{	1526	{
1658	/* is this object in the players inventory, or sub container	1527	/* is this object in the players inventory, or sub container
1659	* therein?	1528	* therein?
1660	*/	1529	*/
1661	tmp = is_player_inv (op->env);	1530	tmp = op->in_player ();
1662	/* nope. Is this a container the player has opened?	1531	/* nope. Is this a container the player has opened?
1663	* If so, set tmp to that player.	1532	* If so, set tmp to that player.
1664	* IMO, searching through all the players will mostly	1533	* IMO, searching through all the players will mostly
1665	* likely be quicker than following op->env to the map,	1534	* likely be quicker than following op->env to the map,
1666	* and then searching the map for a player.	1535	* and then searching the map for a player.
1667	*/	1536	*/
1668	if (!tmp)	1537	if (!tmp)
1669	{	1538	for_all_players (pl)
1670	for (pl = first_player; pl; pl = pl->next)
1671	if (pl->ob->container == op->env)	1539	if (pl->ob->container == op->env)
		1540	{
		1541	tmp = pl->ob;
1672	break;	1542	break;
1673	if (pl)
1674	tmp = pl->ob;
1675	else
1676	tmp = NULL;
1677	}	1543	}
1678		1544
1679	if (i < op->nrof)	1545	if (i < op->nrof)
1680	{	1546	{
1681	sub_weight (op->env, op->weight * i);	1547	sub_weight (op->env, op->weight * i);
1682	op->nrof -= i;	1548	op->nrof -= i;
1683	if (tmp)	1549	if (tmp)
1684	{
1685	esrv_send_item (tmp, op);	1550	esrv_send_item (tmp, op);
1686	}
1687	}	1551	}
1688	else	1552	else
1689	{	1553	{
1690	remove_ob (op);	1554	op->remove ();
1691	op->nrof = 0;	1555	op->nrof = 0;
1692	if (tmp)	1556	if (tmp)
1693	{
1694	esrv_del_item (tmp->contr, op->count);	1557	esrv_del_item (tmp->contr, op->count);
1695	}
1696	}	1558	}
1697	}	1559	}
1698	else	1560	else
1699	{	1561	{
1700	object *above = op->above;	1562	object *above = op->above;
1701		1563
1702	if (i < op->nrof)	1564	if (i < op->nrof)
1703	op->nrof -= i;	1565	op->nrof -= i;
1704	else	1566	else
1705	{	1567	{
1706	remove_ob (op);	1568	op->remove ();
1707	op->nrof = 0;	1569	op->nrof = 0;
1708	}	1570	}
1709		1571
1710	/* Since we just removed op, op->above is null */	1572	/* Since we just removed op, op->above is null */
1711	for (tmp = above; tmp != NULL; tmp = tmp->above)	1573	for (tmp = above; tmp; tmp = tmp->above)
1712	if (tmp->type == PLAYER)	1574	if (tmp->type == PLAYER)
1713	{	1575	{
1714	if (op->nrof)	1576	if (op->nrof)
1715	esrv_send_item (tmp, op);	1577	esrv_send_item (tmp, op);
1716	else	1578	else
…		…
1720		1582
1721	if (op->nrof)	1583	if (op->nrof)
1722	return op;	1584	return op;
1723	else	1585	else
1724	{	1586	{
1725	free_object (op);	1587	op->destroy ();
1726	return NULL;	1588	return 0;
1727	}	1589	}
1728	}	1590	}
1729		1591
1730	/*	1592	/*
1731	* add_weight(object, weight) adds the specified weight to an object,	1593	* add_weight(object, weight) adds the specified weight to an object,
…		…
1743	op->carrying += weight;	1605	op->carrying += weight;
1744	op = op->env;	1606	op = op->env;
1745	}	1607	}
1746	}	1608	}
1747		1609
1748	/*
1749	* insert_ob_in_ob(op,environment):
1750	* This function inserts the object op in the linked list
1751	* inside the object environment.
1752	*
1753	* Eneq(@csd.uu.se): Altered insert_ob_in_ob to make things picked up enter
1754	* the inventory at the last position or next to other objects of the same
1755	* type.
1756	* Frank: Now sorted by type, archetype and magic!
1757	*
1758	* The function returns now pointer to inserted item, and return value can
1759	* be != op, if items are merged. -Tero
1760	*/
1761
1762	object *	1610	object *
1763	insert_ob_in_ob (object *op, object *where)	1611	insert_ob_in_ob (object *op, object *where)
1764	{	1612	{
1765	object *	1613	if (!where)
1766	tmp, *
1767	otmp;
1768
1769	if (!QUERY_FLAG (op, FLAG_REMOVED))
1770	{
1771	char *dump = dump_object (op);
1772	LOG (llevError, "Trying to insert (ob) inserted object.\n%s\n", dump);
1773	free (dump);
1774	return op;
1775	}
1776
1777	if (where == NULL)
1778	{	1614	{
1779	char *dump = dump_object (op);	1615	char *dump = dump_object (op);
1780	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);
1781	free (dump);	1617	free (dump);
1782	return op;	1618	return op;
1783	}	1619	}
1784		1620
1785	if (where->head)	1621	if (where->head)
1786	{	1622	{
1787	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");
1788	where = where->head;	1624	where = where->head;
1789	}	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 ();
1790		1646
1791	if (op->more)	1647	if (op->more)
1792	{	1648	{
1793	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);
1794	return op;	1650	return op;
…		…
1796		1652
1797	CLEAR_FLAG (op, FLAG_OBJ_ORIGINAL);	1653	CLEAR_FLAG (op, FLAG_OBJ_ORIGINAL);
1798	CLEAR_FLAG (op, FLAG_REMOVED);	1654	CLEAR_FLAG (op, FLAG_REMOVED);
1799	if (op->nrof)	1655	if (op->nrof)
1800	{	1656	{
1801	for (tmp = where->inv; tmp != NULL; tmp = tmp->below)	1657	for (tmp = inv; tmp != NULL; tmp = tmp->below)
1802	if (CAN_MERGE (tmp, op))	1658	if (object::can_merge (tmp, op))
1803	{	1659	{
1804	/* return the original object and remove inserted object	1660	/* return the original object and remove inserted object
1805	(client needs the original object) */	1661	(client needs the original object) */
1806	tmp->nrof += op->nrof;	1662	tmp->nrof += op->nrof;
1807	/* Weight handling gets pretty funky. Since we are adding to	1663	/* Weight handling gets pretty funky. Since we are adding to
1808	* tmp->nrof, we need to increase the weight.	1664	* tmp->nrof, we need to increase the weight.
1809	*/	1665	*/
1810	add_weight (where, op->weight * op->nrof);	1666	add_weight (this, op->weight * op->nrof);
1811	SET_FLAG (op, FLAG_REMOVED);	1667	SET_FLAG (op, FLAG_REMOVED);
1812	free_object (op); /* free the inserted object */	1668	op->destroy (); /* free the inserted object */
1813	op = tmp;	1669	op = tmp;
1814	remove_ob (op); /* and fix old object's links */	1670	op->remove (); /* and fix old object's links */
1815	CLEAR_FLAG (op, FLAG_REMOVED);	1671	CLEAR_FLAG (op, FLAG_REMOVED);
1816	break;	1672	break;
1817	}	1673	}
1818		1674
1819	/* I assume combined objects have no inventory	1675	/* I assume combined objects have no inventory
1820	* We add the weight - this object could have just been removed	1676	* We add the weight - this object could have just been removed
1821	* (if it was possible to merge). calling remove_ob will subtract	1677	* (if it was possible to merge). calling remove_ob will subtract
1822	* 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
1823	* the linking below	1679	* the linking below
1824	*/	1680	*/
1825	add_weight (where, op->weight * op->nrof);	1681	add_weight (this, op->weight * op->nrof);
1826	}	1682	}
1827	else	1683	else
1828	add_weight (where, (op->weight + op->carrying));	1684	add_weight (this, (op->weight + op->carrying));
1829		1685
1830	otmp = is_player_inv (where);	1686	otmp = this->in_player ();
1831	if (otmp && otmp->contr != NULL)	1687	if (otmp && otmp->contr)
1832	{
1833	if (!QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))	1688	if (!QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
1834	fix_player (otmp);	1689	otmp->update_stats ();
1835	}
1836		1690
1837	op->map = NULL;	1691	op->map = 0;
1838	op->env = where;	1692	op->env = this;
1839	op->above = NULL;	1693	op->above = 0;
1840	op->below = NULL;	1694	op->below = 0;
1841	op->x = 0, op->y = 0;	1695	op->x = 0, op->y = 0;
1842		1696
1843	/* 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 */
1844	if ((op->glow_radius != 0) && where->map)	1698	if ((op->glow_radius != 0) && map)
1845	{	1699	{
1846	#ifdef DEBUG_LIGHTS	1700	#ifdef DEBUG_LIGHTS
1847	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);
1848	#endif /* DEBUG_LIGHTS */	1702	#endif /* DEBUG_LIGHTS */
1849	if (MAP_DARKNESS (where->map))	1703	if (map->darkness)
1850	update_all_los (where->map, where->x, where->y);	1704	update_all_los (map, x, y);
1851	}	1705	}
1852		1706
1853	/* 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.
1854	* It sure simplifies this function...	1708	* It sure simplifies this function...
1855	*/	1709	*/
1856	if (where->inv == NULL)	1710	if (!inv)
1857	where->inv = op;	1711	inv = op;
1858	else	1712	else
1859	{	1713	{
1860	op->below = where->inv;	1714	op->below = inv;
1861	op->below->above = op;	1715	op->below->above = op;
1862	where->inv = op;	1716	inv = op;
1863	}	1717	}
		1718
		1719	INVOKE_OBJECT (INSERT, this);
		1720
1864	return op;	1721	return op;
1865	}	1722	}
1866		1723
1867	/*	1724	/*
1868	* Checks if any objects has a move_type that matches objects	1725	* Checks if any objects has a move_type that matches objects
…		…
1919		1776
1920	/* The objects have to be checked from top to bottom.	1777	/* The objects have to be checked from top to bottom.
1921	* Hence, we first go to the top:	1778	* Hence, we first go to the top:
1922	*/	1779	*/
1923		1780
1924	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)
1925	{	1782	{
1926	/* Trim the search when we find the first other spell effect	1783	/* Trim the search when we find the first other spell effect
1927	* 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,
1928	* we don't need to check all of them.	1785	* we don't need to check all of them.
1929	*/	1786	*/
…		…
1984	/*	1841	/*
1985	* present_arch(arch, map, x, y) searches for any objects with	1842	* present_arch(arch, map, x, y) searches for any objects with
1986	* a matching archetype at the given map and coordinates.	1843	* a matching archetype at the given map and coordinates.
1987	* The first matching object is returned, or NULL if none.	1844	* The first matching object is returned, or NULL if none.
1988	*/	1845	*/
1989
1990	object *	1846	object *
1991	present_arch (const archetype *at, maptile *m, int x, int y)	1847	present_arch (const archetype *at, maptile *m, int x, int y)
1992	{	1848	{
1993	object *
1994	tmp;
1995
1996	if (m == NULL || out_of_map (m, x, y))	1849	if (m == NULL || out_of_map (m, x, y))
1997	{	1850	{
1998	LOG (llevError, "Present_arch called outside map.\n");	1851	LOG (llevError, "Present_arch called outside map.\n");
1999	return NULL;	1852	return NULL;
2000	}	1853	}
		1854
2001	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)
2002	if (tmp->arch == at)	1856	if (tmp->arch == at)
2003	return tmp;	1857	return tmp;
		1858
2004	return NULL;	1859	return NULL;
2005	}	1860	}
2006		1861
2007	/*	1862	/*
2008	* present(type, map, x, y) searches for any objects with	1863	* present(type, map, x, y) searches for any objects with
2009	* a matching type variable at the given map and coordinates.	1864	* a matching type variable at the given map and coordinates.
2010	* The first matching object is returned, or NULL if none.	1865	* The first matching object is returned, or NULL if none.
2011	*/	1866	*/
2012
2013	object *	1867	object *
2014	present (unsigned char type, maptile *m, int x, int y)	1868	present (unsigned char type, maptile *m, int x, int y)
2015	{	1869	{
2016	object *
2017	tmp;
2018
2019	if (out_of_map (m, x, y))	1870	if (out_of_map (m, x, y))
2020	{	1871	{
2021	LOG (llevError, "Present called outside map.\n");	1872	LOG (llevError, "Present called outside map.\n");
2022	return NULL;	1873	return NULL;
2023	}	1874	}
		1875
2024	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)
2025	if (tmp->type == type)	1877	if (tmp->type == type)
2026	return tmp;	1878	return tmp;
		1879
2027	return NULL;	1880	return NULL;
2028	}	1881	}
2029		1882
2030	/*	1883	/*
2031	* present_in_ob(type, object) searches for any objects with	1884	* present_in_ob(type, object) searches for any objects with
2032	* a matching type variable in the inventory of the given object.	1885	* a matching type variable in the inventory of the given object.
2033	* The first matching object is returned, or NULL if none.	1886	* The first matching object is returned, or NULL if none.
2034	*/	1887	*/
2035
2036	object *	1888	object *
2037	present_in_ob (unsigned char type, const object *op)	1889	present_in_ob (unsigned char type, const object *op)
2038	{	1890	{
2039	object *
2040	tmp;
2041
2042	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)	1891	for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below)
2043	if (tmp->type == type)	1892	if (tmp->type == type)
2044	return tmp;	1893	return tmp;
		1894
2045	return NULL;	1895	return NULL;
2046	}	1896	}
2047		1897
2048	/*	1898	/*
2049	* present_in_ob (type, str, object) searches for any objects with	1899	* present_in_ob (type, str, object) searches for any objects with
…		…
2057	* 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
2058	* 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
2059	* 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
2060	* to be unique.	1910	* to be unique.
2061	*/	1911	*/
2062
2063	object *	1912	object *
2064	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)
2065	{	1914	{
2066	object *
2067	tmp;
2068
2069	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)	1915	for (object *tmp = op->inv; tmp; tmp = tmp->below)
2070	{
2071	if ((type == -1 || tmp->type == type) && (!strcmp (str, tmp->name)))	1916	if ((type == -1 || tmp->type == type) && (!strcmp (str, tmp->name)))
2072	return tmp;	1917	return tmp;
2073	}	1918
2074	return NULL;	1919	return 0;
2075	}	1920	}
2076		1921
2077	/*	1922	/*
2078	* present_arch_in_ob(archetype, object) searches for any objects with	1923	* present_arch_in_ob(archetype, object) searches for any objects with
2079	* a matching archetype in the inventory of the given object.	1924	* a matching archetype in the inventory of the given object.
2080	* The first matching object is returned, or NULL if none.	1925	* The first matching object is returned, or NULL if none.
2081	*/	1926	*/
2082
2083	object *	1927	object *
2084	present_arch_in_ob (const archetype *at, const object *op)	1928	present_arch_in_ob (const archetype *at, const object *op)
2085	{	1929	{
2086	object *
2087	tmp;
2088
2089	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)	1930	for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below)
2090	if (tmp->arch == at)	1931	if (tmp->arch == at)
2091	return tmp;	1932	return tmp;
		1933
2092	return NULL;	1934	return NULL;
2093	}	1935	}
2094		1936
2095	/*	1937	/*
2096	* activate recursively a flag on an object inventory	1938	* activate recursively a flag on an object inventory
2097	*/	1939	*/
2098	void	1940	void
2099	flag_inv (object *op, int flag)	1941	flag_inv (object *op, int flag)
2100	{	1942	{
2101	object *
2102	tmp;
2103
2104	if (op->inv)	1943	if (op->inv)
2105	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)	1944	for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below)
2106	{	1945	{
2107	SET_FLAG (tmp, flag);	1946	SET_FLAG (tmp, flag);
2108	flag_inv (tmp, flag);	1947	flag_inv (tmp, flag);
2109	}	1948	}
2110	} /*	1949	}
		1950
		1951	/*
2111	* desactivate recursively a flag on an object inventory	1952	* deactivate recursively a flag on an object inventory
2112	*/	1953	*/
2113	void	1954	void
2114	unflag_inv (object *op, int flag)	1955	unflag_inv (object *op, int flag)
2115	{	1956	{
2116	object *
2117	tmp;
2118
2119	if (op->inv)	1957	if (op->inv)
2120	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)	1958	for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below)
2121	{	1959	{
2122	CLEAR_FLAG (tmp, flag);	1960	CLEAR_FLAG (tmp, flag);
2123	unflag_inv (tmp, flag);	1961	unflag_inv (tmp, flag);
2124	}	1962	}
2125	}	1963	}
…		…
2128	* 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
2129	* all it's inventory (recursively).	1967	* all it's inventory (recursively).
2130	* 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
2131	* 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.
2132	*/	1970	*/
2133
2134	void	1971	void
2135	set_cheat (object *op)	1972	set_cheat (object *op)
2136	{	1973	{
2137	SET_FLAG (op, FLAG_WAS_WIZ);	1974	SET_FLAG (op, FLAG_WAS_WIZ);
2138	flag_inv (op, FLAG_WAS_WIZ);	1975	flag_inv (op, FLAG_WAS_WIZ);
…		…
2157	* 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
2158	* 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
2159	* 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
2160	* customized, changed states, etc.	1997	* customized, changed states, etc.
2161	*/	1998	*/
2162
2163	int	1999	int
2164	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)
2165	{	2001	{
2166	int
2167	i,
2168	index = 0, flag;	2002	int index = 0, flag;
2169	static int
2170	altern[SIZEOFFREE];	2003	int altern[SIZEOFFREE];
2171		2004
2172	for (i = start; i < stop; i++)	2005	for (int i = start; i < stop; i++)
2173	{	2006	{
2174	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]);
2175	if (!flag)	2008	if (!flag)
2176	altern[index++] = i;	2009	altern [index++] = i;
2177		2010
2178	/* 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.
2179	* 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.
2180	* 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 -
2181	* 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
2182	* 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
2183	* to only the spaces immediately surrounding the target area, and	2016	* to only the spaces immediately surrounding the target area, and
2184	* won't look 2 spaces south of the target space.	2017	* won't look 2 spaces south of the target space.
2185	*/	2018	*/
2186	else if ((flag & AB_NO_PASS) && maxfree[i] < stop)	2019	else if ((flag & P_NO_PASS) && maxfree[i] < stop)
2187	stop = maxfree[i];	2020	stop = maxfree[i];
2188	}	2021	}
		2022
2189	if (!index)	2023	if (!index)
2190	return -1;	2024	return -1;
		2025
2191	return altern[RANDOM () % index];	2026	return altern[RANDOM () % index];
2192	}	2027	}
2193		2028
2194	/*	2029	/*
2195	* find_first_free_spot(archetype, maptile, x, y) works like	2030	* find_first_free_spot(archetype, maptile, x, y) works like
2196	* find_free_spot(), but it will search max number of squares.	2031	* find_free_spot(), but it will search max number of squares.
2197	* 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.
2198	* 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.
2199	*/	2034	*/
2200
2201	int	2035	int
2202	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)
2203	{	2037	{
2204	int
2205	i;
2206
2207	for (i = 0; i < SIZEOFFREE; i++)	2038	for (int i = 0; i < SIZEOFFREE; i++)
2208	{
2209	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]))
2210	return i;	2040	return i;
2211	}	2041
2212	return -1;	2042	return -1;
2213	}	2043	}
2214		2044
2215	/*	2045	/*
2216	* The function permute(arr, begin, end) randomly reorders the array	2046	* The function permute(arr, begin, end) randomly reorders the array
2217	* arr[begin..end-1].	2047	* arr[begin..end-1].
		2048	* now uses a fisher-yates shuffle, old permute was broken
2218	*/	2049	*/
2219	static void	2050	static void
2220	permute (int *arr, int begin, int end)	2051	permute (int *arr, int begin, int end)
2221	{	2052	{
2222	int	2053	arr += begin;
2223	i,
2224	j,
2225	tmp,
2226	len;
2227
2228	len = end - begin;	2054	end -= begin;
2229	for (i = begin; i < end; i++)
2230	{
2231	j = begin + RANDOM () % len;
2232		2055
2233	tmp = arr[i];	2056	while (--end)
2234	arr[i] = arr[j];	2057	swap (arr [end], arr [RANDOM () % (end + 1)]);
2235	arr[j] = tmp;
2236	}
2237	}	2058	}
2238		2059
2239	/* new function to make monster searching more efficient, and effective!	2060	/* new function to make monster searching more efficient, and effective!
2240	* This basically returns a randomized array (in the passed pointer) of	2061	* This basically returns a randomized array (in the passed pointer) of
2241	* 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
…		…
2244	* 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.
2245	*/	2066	*/
2246	void	2067	void
2247	get_search_arr (int *search_arr)	2068	get_search_arr (int *search_arr)
2248	{	2069	{
2249	int	2070	int i;
2250	i;
2251		2071
2252	for (i = 0; i < SIZEOFFREE; i++)	2072	for (i = 0; i < SIZEOFFREE; i++)
2253	{
2254	search_arr[i] = i;	2073	search_arr[i] = i;
2255	}
2256		2074
2257	permute (search_arr, 1, SIZEOFFREE1 + 1);	2075	permute (search_arr, 1, SIZEOFFREE1 + 1);
2258	permute (search_arr, SIZEOFFREE1 + 1, SIZEOFFREE2 + 1);	2076	permute (search_arr, SIZEOFFREE1 + 1, SIZEOFFREE2 + 1);
2259	permute (search_arr, SIZEOFFREE2 + 1, SIZEOFFREE);	2077	permute (search_arr, SIZEOFFREE2 + 1, SIZEOFFREE);
2260	}	2078	}
…		…
2269	* Perhaps incorrectly, but I'm making the assumption that exclude	2087	* Perhaps incorrectly, but I'm making the assumption that exclude
2270	* 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
2271	* 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
2272	* there is capable of.	2090	* there is capable of.
2273	*/	2091	*/
2274
2275	int	2092	int
2276	find_dir (maptile *m, int x, int y, object *exclude)	2093	find_dir (maptile *m, int x, int y, object *exclude)
2277	{	2094	{
2278	int
2279	i,
2280	max = SIZEOFFREE, mflags;	2095	int i, max = SIZEOFFREE, mflags;
2281		2096
2282	sint16 nx, ny;	2097	sint16 nx, ny;
2283	object *	2098	object *tmp;
2284	tmp;
2285	maptile *	2099	maptile *mp;
2286	mp;
2287		2100
2288	MoveType blocked, move_type;	2101	MoveType blocked, move_type;
2289		2102
2290	if (exclude && exclude->head)	2103	if (exclude && exclude->head)
2291	{	2104	{
…		…
2303	mp = m;	2116	mp = m;
2304	nx = x + freearr_x[i];	2117	nx = x + freearr_x[i];
2305	ny = y + freearr_y[i];	2118	ny = y + freearr_y[i];
2306		2119
2307	mflags = get_map_flags (m, &mp, nx, ny, &nx, &ny);	2120	mflags = get_map_flags (m, &mp, nx, ny, &nx, &ny);
		2121
2308	if (mflags & P_OUT_OF_MAP)	2122	if (mflags & P_OUT_OF_MAP)
2309	{
2310	max = maxfree[i];	2123	max = maxfree[i];
2311	}
2312	else	2124	else
2313	{	2125	{
2314	blocked = GET_MAP_MOVE_BLOCK (mp, nx, ny);	2126	mapspace &ms = mp->at (nx, ny);
		2127
		2128	blocked = ms.move_block;
2315		2129
2316	if ((move_type & blocked) == move_type)	2130	if ((move_type & blocked) == move_type)
2317	{
2318	max = maxfree[i];	2131	max = maxfree[i];
2319	}
2320	else if (mflags & P_IS_ALIVE)	2132	else if (mflags & P_IS_ALIVE)
2321	{	2133	{
2322	for (tmp = GET_MAP_OB (mp, nx, ny); tmp != NULL; tmp = tmp->above)	2134	for (tmp = ms.bot; tmp; tmp = tmp->above)
2323	{	2135	if ((tmp->flag [FLAG_MONSTER] || tmp->type == PLAYER)
2324	if ((QUERY_FLAG (tmp, FLAG_MONSTER) || tmp->type == PLAYER) && (tmp != exclude || (tmp->head && tmp->head != exclude)))	2136	&& (tmp != exclude || (tmp->head && tmp->head != exclude)))
2325	{
2326	break;	2137	break;
2327	}	2138
2328	}
2329	if (tmp)	2139	if (tmp)
2330	{
2331	return freedir[i];	2140	return freedir[i];
2332	}
2333	}	2141	}
2334	}	2142	}
2335	}	2143	}
		2144
2336	return 0;	2145	return 0;
2337	}	2146	}
2338		2147
2339	/*	2148	/*
2340	* distance(object 1, object 2) will return the square of the	2149	* distance(object 1, object 2) will return the square of the
2341	* distance between the two given objects.	2150	* distance between the two given objects.
2342	*/	2151	*/
2343
2344	int	2152	int
2345	distance (const object *ob1, const object *ob2)	2153	distance (const object *ob1, const object *ob2)
2346	{	2154	{
2347	int
2348	i;
2349
2350	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);
2351	return i;
2352	}	2156	}
2353		2157
2354	/*	2158	/*
2355	* 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
2356	* 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
2357	* object, needs to travel toward it.	2161	* object, needs to travel toward it.
2358	*/	2162	*/
2359
2360	int	2163	int
2361	find_dir_2 (int x, int y)	2164	find_dir_2 (int x, int y)
2362	{	2165	{
2363	int	2166	int q;
2364	q;
2365		2167
2366	if (y)	2168	if (y)
2367	q = x * 100 / y;	2169	q = x * 100 / y;
2368	else if (x)	2170	else if (x)
2369	q = -300 * x;	2171	q = -300 * x;
…		…
2404	int	2206	int
2405	absdir (int d)	2207	absdir (int d)
2406	{	2208	{
2407	while (d < 1)	2209	while (d < 1)
2408	d += 8;	2210	d += 8;
		2211
2409	while (d > 8)	2212	while (d > 8)
2410	d -= 8;	2213	d -= 8;
		2214
2411	return d;	2215	return d;
2412	}	2216	}
2413		2217
2414	/*	2218	/*
2415	* dirdiff(dir1, dir2) returns how many 45-degrees differences there is	2219	* dirdiff(dir1, dir2) returns how many 45-degrees differences there is
…		…
2417	*/	2221	*/
2418		2222
2419	int	2223	int
2420	dirdiff (int dir1, int dir2)	2224	dirdiff (int dir1, int dir2)
2421	{	2225	{
2422	int	2226	int d;
2423	d;
2424		2227
2425	d = abs (dir1 - dir2);	2228	d = abs (dir1 - dir2);
2426	if (d > 4)	2229	if (d > 4)
2427	d = 8 - d;	2230	d = 8 - d;
		2231
2428	return d;	2232	return d;
2429	}	2233	}
2430		2234
2431	/* peterm:	2235	/* peterm:
2432	* 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.
…		…
2435	* 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
2436	* 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.
2437	* 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
2438	* functions.	2242	* functions.
2439	*/	2243	*/
2440
2441	int
2442	reduction_dir[SIZEOFFREE][3] = {	2244	int reduction_dir[SIZEOFFREE][3] = {
2443	{0, 0, 0}, /* 0 */	2245	{0, 0, 0}, /* 0 */
2444	{0, 0, 0}, /* 1 */	2246	{0, 0, 0}, /* 1 */
2445	{0, 0, 0}, /* 2 */	2247	{0, 0, 0}, /* 2 */
2446	{0, 0, 0}, /* 3 */	2248	{0, 0, 0}, /* 3 */
2447	{0, 0, 0}, /* 4 */	2249	{0, 0, 0}, /* 4 */
…		…
2495	* find a path to that monster that we found. If not,	2297	* find a path to that monster that we found. If not,
2496	* we don't bother going toward it. Returns 1 if we	2298	* we don't bother going toward it. Returns 1 if we
2497	* can see a direct way to get it	2299	* can see a direct way to get it
2498	* Modified to be map tile aware -.MSW	2300	* Modified to be map tile aware -.MSW
2499	*/	2301	*/
2500
2501
2502	int	2302	int
2503	can_see_monsterP (maptile *m, int x, int y, int dir)	2303	can_see_monsterP (maptile *m, int x, int y, int dir)
2504	{	2304	{
2505	sint16 dx, dy;	2305	sint16 dx, dy;
2506	int
2507	mflags;	2306	int mflags;
2508		2307
2509	if (dir < 0)	2308	if (dir < 0)
2510	return 0; /* exit condition: invalid direction */	2309	return 0; /* exit condition: invalid direction */
2511		2310
2512	dx = x + freearr_x[dir];	2311	dx = x + freearr_x[dir];
…		…
2525	return 0;	2324	return 0;
2526		2325
2527	/* yes, can see. */	2326	/* yes, can see. */
2528	if (dir < 9)	2327	if (dir < 9)
2529	return 1;	2328	return 1;
		2329
2530	return can_see_monsterP (m, x, y, reduction_dir[dir][0]) |	2330	return can_see_monsterP (m, x, y, reduction_dir[dir][0])
2531	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]);
2532	}	2333	}
2533
2534
2535		2334
2536	/*	2335	/*
2537	* 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
2538	* picked up by the picker. Returnes 1 if it can be	2337	* picked up by the picker. Returnes 1 if it can be
2539	* picked up, otherwise 0.	2338	* picked up, otherwise 0.
…		…
2550	return /*QUERY_FLAG(who,FLAG_WIZ)|| */	2349	return /*QUERY_FLAG(who,FLAG_WIZ)|| */
2551	(item->weight > 0 && !QUERY_FLAG (item, FLAG_NO_PICK) &&	2350	(item->weight > 0 && !QUERY_FLAG (item, FLAG_NO_PICK) &&
2552	!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));
2553	}	2352	}
2554		2353
2555
2556	/*	2354	/*
2557	* create clone from object to another	2355	* create clone from object to another
2558	*/	2356	*/
2559	object *	2357	object *
2560	object_create_clone (object *asrc)	2358	object_create_clone (object *asrc)
2561	{	2359	{
2562	object *
2563	dst = NULL, *tmp, *src, *part, *prev, *item;	2360	object *dst = 0, *tmp, *src, *part, *prev, *item;
2564		2361
2565	if (!asrc)	2362	if (!asrc)
2566	return NULL;	2363	return 0;
		2364
2567	src = asrc;	2365	src = asrc;
2568	if (src->head)	2366	if (src->head)
2569	src = src->head;	2367	src = src->head;
2570		2368
2571	prev = NULL;	2369	prev = 0;
2572	for (part = src; part; part = part->more)	2370	for (part = src; part; part = part->more)
2573	{	2371	{
2574	tmp = get_object ();	2372	tmp = part->clone ();
2575	copy_object (part, tmp);
2576	tmp->x -= src->x;	2373	tmp->x -= src->x;
2577	tmp->y -= src->y;	2374	tmp->y -= src->y;
		2375
2578	if (!part->head)	2376	if (!part->head)
2579	{	2377	{
2580	dst = tmp;	2378	dst = tmp;
2581	tmp->head = NULL;	2379	tmp->head = 0;
2582	}	2380	}
2583	else	2381	else
2584	{
2585	tmp->head = dst;	2382	tmp->head = dst;
2586	}	2383
2587	tmp->more = NULL;	2384	tmp->more = 0;
		2385
2588	if (prev)	2386	if (prev)
2589	prev->more = tmp;	2387	prev->more = tmp;
		2388
2590	prev = tmp;	2389	prev = tmp;
2591	}	2390	}
2592		2391
2593	for (item = src->inv; item; item = item->below)	2392	for (item = src->inv; item; item = item->below)
2594	insert_ob_in_ob (object_create_clone (item), dst);	2393	insert_ob_in_ob (object_create_clone (item), dst);
…		…
2600	/* 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 */
2601	/* 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, */
2602	/* 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. */
2603	/* 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. */
2604	/* Also remember that multiparts objects are not supported for now. */	2403	/* Also remember that multiparts objects are not supported for now. */
2605
2606	object *	2404	object *
2607	load_object_str (const char *obstr)	2405	load_object_str (const char *obstr)
2608	{	2406	{
2609	object *op;	2407	object *op;
2610	char filename[MAX_BUF];	2408	char filename[MAX_BUF];
…		…
2620	}	2418	}
2621		2419
2622	fprintf (tempfile, obstr);	2420	fprintf (tempfile, obstr);
2623	fclose (tempfile);	2421	fclose (tempfile);
2624		2422
2625	op = get_object ();	2423	op = object::create ();
2626		2424
2627	object_thawer thawer (filename);	2425	object_thawer thawer (filename);
2628		2426
2629	if (thawer)	2427	if (thawer)
2630	load_object (thawer, op, 0);	2428	load_object (thawer, op, 0);
…		…
2640	* returns NULL if no match.	2438	* returns NULL if no match.
2641	*/	2439	*/
2642	object *	2440	object *
2643	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)
2644	{	2442	{
2645	object *tmp;
2646
2647	for (tmp = who->inv; tmp; tmp = tmp->below)	2443	for (object *tmp = who->inv; tmp; tmp = tmp->below)
2648	if (tmp->type == type && tmp->subtype == subtype)	2444	if (tmp->type == type && tmp->subtype == subtype)
2649	return tmp;	2445	return tmp;
2650		2446
2651	return NULL;	2447	return 0;
2652	}	2448	}
2653		2449
2654	/* 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,
2655	* otherwise return NULL.	2451	* otherwise return NULL.
2656	*	2452	*
…		…
2658	* do the desired thing.	2454	* do the desired thing.
2659	*/	2455	*/
2660	key_value *	2456	key_value *
2661	get_ob_key_link (const object *ob, const char *key)	2457	get_ob_key_link (const object *ob, const char *key)
2662	{	2458	{
2663	key_value *link;
2664
2665	for (link = ob->key_values; link != NULL; link = link->next)	2459	for (key_value *link = ob->key_values; link; link = link->next)
2666	if (link->key == key)	2460	if (link->key == key)
2667	return link;	2461	return link;
2668		2462
2669	return NULL;	2463	return 0;
2670	}	2464	}
2671		2465
2672	/*	2466	/*
2673	* 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.
2674	*	2468	*
…		…
2714	* Returns TRUE on success.	2508	* Returns TRUE on success.
2715	*/	2509	*/
2716	int	2510	int
2717	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)
2718	{	2512	{
2719	key_value *
2720	field = NULL, *last = NULL;	2513	key_value *field = NULL, *last = NULL;
2721		2514
2722	for (field = op->key_values; field != NULL; field = field->next)	2515	for (field = op->key_values; field != NULL; field = field->next)
2723	{	2516	{
2724	if (field->key != canonical_key)	2517	if (field->key != canonical_key)
2725	{	2518	{
…		…
2753	/* IF we get here, key doesn't exist */	2546	/* IF we get here, key doesn't exist */
2754		2547
2755	/* No field, we'll have to add it. */	2548	/* No field, we'll have to add it. */
2756		2549
2757	if (!add_key)	2550	if (!add_key)
2758	{
2759	return FALSE;	2551	return FALSE;
2760	}	2552
2761	/* There isn't any good reason to store a null	2553	/* There isn't any good reason to store a null
2762	* value in the key/value list. If the archetype has	2554	* value in the key/value list. If the archetype has
2763	* this key, then we should also have it, so shouldn't	2555	* this key, then we should also have it, so shouldn't
2764	* be here. If user wants to store empty strings,	2556	* be here. If user wants to store empty strings,
2765	* should pass in ""	2557	* should pass in ""

Diff Legend

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