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.15 by root, Mon Sep 4 13:55:54 2006 UTC vs.
Revision 1.24 by root, Sun Sep 10 16:00:23 2006 UTC

		1
1	/*	2	/*
2	* static char *rcsid_object_c =	3	* static char *rcsid_object_c =
3	* "$Id: object.C,v 1.15 2006/09/04 13:55:54 root Exp $";	4	* "$Id: object.C,v 1.24 2006/09/10 16:00:23 root Exp $";
4	*/	5	*/
5		6
6	/*	7	/*
7	CrossFire, A Multiplayer game for X-windows	8	CrossFire, A Multiplayer game for X-windows
8		9
…		…
28		29
29	/* Eneq(@csd.uu.se): Added weight-modifiers in environment of objects.	30	/* Eneq(@csd.uu.se): Added weight-modifiers in environment of objects.
30	sub/add_weight will transcend the environment updating the carrying	31	sub/add_weight will transcend the environment updating the carrying
31	variable. */	32	variable. */
32	#include <global.h>	33	#include <global.h>
33	#ifndef WIN32 /* ---win32 exclude headers */	34	#ifndef WIN32 /* ---win32 exclude headers */
34	#include <stdio.h>	35	# include <stdio.h>
35	#include <sys/types.h>	36	# include <sys/types.h>
36	#include <sys/uio.h>	37	# include <sys/uio.h>
37	#endif /* win32 */	38	#endif /* win32 */
38	#include <object.h>	39	#include <object.h>
39	#include <funcpoint.h>	40	#include <funcpoint.h>
40	#include <skills.h>	41	#include <skills.h>
41	#include <loader.h>	42	#include <loader.h>
42	int nrofallocobjects = 0;	43	int nrofallocobjects = 0;
43		44
44	object *objects; /* Pointer to the list of used objects */	45	object *objects; /* Pointer to the list of used objects */
45	object *active_objects; /* List of active objects that need to be processed */	46	object *active_objects; /* List of active objects that need to be processed */
46		47
47	short freearr_x[SIZEOFFREE]=	48	short freearr_x[SIZEOFFREE] = { 0, 0, 1, 1, 1, 0, -1, -1, -1, 0, 1, 2, 2, 2, 2, 2, 1, 0, -1, -2, -2, -2, -2, -2, -1,
48	{0,0,1,1,1,0,-1,-1,-1,0,1,2,2,2,2,2,1,0,-1,-2,-2,-2,-2,-2,-1,
49	0,1,2,3,3,3,3,3,3,3,2,1,0,-1,-2,-3,-3,-3,-3,-3,-3,-3,-2,-1};	49	0, 1, 2, 3, 3, 3, 3, 3, 3, 3, 2, 1, 0, -1, -2, -3, -3, -3, -3, -3, -3, -3, -2, -1
50	short freearr_y[SIZEOFFREE]=	50	};
51	{0,-1,-1,0,1,1,1,0,-1,-2,-2,-2,-1,0,1,2,2,2,2,2,1,0,-1,-2,-2,	51	short freearr_y[SIZEOFFREE] = { 0, -1, -1, 0, 1, 1, 1, 0, -1, -2, -2, -2, -1, 0, 1, 2, 2, 2, 2, 2, 1, 0, -1, -2, -2,
52	-3,-3,-3,-3,-2,-1,0,1,2,3,3,3,3,3,3,3,2,1,0,-1,-2,-3,-3,-3};	52	-3, -3, -3, -3, -2, -1, 0, 1, 2, 3, 3, 3, 3, 3, 3, 3, 2, 1, 0, -1, -2, -3, -3, -3
53	int maxfree[SIZEOFFREE]=	53	};
54	{0,9,10,13,14,17,18,21,22,25,26,27,30,31,32,33,36,37,39,39,42,43,44,45,	54	int maxfree[SIZEOFFREE] = { 0, 9, 10, 13, 14, 17, 18, 21, 22, 25, 26, 27, 30, 31, 32, 33, 36, 37, 39, 39, 42, 43, 44, 45,
55	48,49,49,49,49,49,49,49,49,49,49,49,49,49,49,49,49,49,49,49,49,49,49,49,49};	55	48, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49
		56	};
56	int freedir[SIZEOFFREE]= {	57	int freedir[SIZEOFFREE] = {
57	0,1,2,3,4,5,6,7,8,1,2,2,2,3,4,4,4,5,6,6,6,7,8,8,8,	58	0, 1, 2, 3, 4, 5, 6, 7, 8, 1, 2, 2, 2, 3, 4, 4, 4, 5, 6, 6, 6, 7, 8, 8, 8,
58	1,2,2,2,2,2,3,4,4,4,4,4,5,6,6,6,6,6,7,8,8,8,8,8};	59	1, 2, 2, 2, 2, 2, 3, 4, 4, 4, 4, 4, 5, 6, 6, 6, 6, 6, 7, 8, 8, 8, 8, 8
59		60	};
60		61
61	/* Returns TRUE if every key_values in wants has a partner with the same value in has. */	62	/* Returns TRUE if every key_values in wants has a partner with the same value in has. */
		63	static int
62	static int compare_ob_value_lists_one(const object * wants, const object * has) {	64	compare_ob_value_lists_one (const object *wants, const object *has)
		65	{
63	key_value * wants_field;	66	key_value *wants_field;
64		67
65	/* n-squared behaviour (see get_ob_key_link()), but I'm hoping both	68	/* n-squared behaviour (see get_ob_key_link()), but I'm hoping both
66	* objects with lists are rare, and lists stay short. If not, use a	69	* objects with lists are rare, and lists stay short. If not, use a
67	* different structure or at least keep the lists sorted...	70	* different structure or at least keep the lists sorted...
68	*/	71	*/
69		72
70	/* For each field in wants, */	73	/* For each field in wants, */
71	for (wants_field = wants->key_values; wants_field != NULL; wants_field = wants_field->next) {	74	for (wants_field = wants->key_values; wants_field != NULL; wants_field = wants_field->next)
		75	{
72	key_value * has_field;	76	key_value *has_field;
73		77
74	/* Look for a field in has with the same key. */	78	/* Look for a field in has with the same key. */
75	has_field = get_ob_key_link(has, wants_field->key);	79	has_field = get_ob_key_link (has, wants_field->key);
		80
		81	if (has_field == NULL)
76		82	{
77	if (has_field == NULL) {
78	/* No field with that name. */	83	/* No field with that name. */
79	return FALSE;	84	return FALSE;
80	}
81		85	}
		86
82	/* Found the matching field. */	87	/* Found the matching field. */
83	if (has_field->value != wants_field->value) {	88	if (has_field->value != wants_field->value)
		89	{
84	/* Values don't match, so this half of the comparison is false. */	90	/* Values don't match, so this half of the comparison is false. */
85	return FALSE;	91	return FALSE;
86	}	92	}
87		93
88	/* If we get here, we found a match. Now for the next field in wants. */	94	/* If we get here, we found a match. Now for the next field in wants. */
89	}
90		95	}
		96
91	/* If we get here, every field in wants has a matching field in has. */	97	/* If we get here, every field in wants has a matching field in has. */
92	return TRUE;	98	return TRUE;
93	}	99	}
94		100
95	/* Returns TRUE if ob1 has the same key_values as ob2. */	101	/* Returns TRUE if ob1 has the same key_values as ob2. */
		102	static int
96	static int compare_ob_value_lists(const object * ob1, const object * ob2) {	103	compare_ob_value_lists (const object *ob1, const object *ob2)
		104	{
97	/* However, there may be fields in has which aren't partnered in wants,	105	/* However, there may be fields in has which aren't partnered in wants,
98	* so we need to run the comparison *twice*. :(	106	* so we need to run the comparison *twice*. :(
99	*/	107	*/
100	return compare_ob_value_lists_one(ob1, ob2) && compare_ob_value_lists_one(ob2, ob1);	108	return compare_ob_value_lists_one (ob1, ob2) && compare_ob_value_lists_one (ob2, ob1);
101	}	109	}
102		110
103	/* Function examines the 2 objects given to it, and returns true if	111	/* Function examines the 2 objects given to it, and returns true if
104	* they can be merged together.	112	* they can be merged together.
105	*	113	*
…		…
111	*	119	*
112	* Improvements made with merge: Better checking on potion, and also	120	* Improvements made with merge: Better checking on potion, and also
113	* check weight	121	* check weight
114	*/	122	*/
115		123
116	int CAN_MERGE(object *ob1, object *ob2) {	124	bool object::can_merge (object *ob1, object *ob2)
117		125	{
118	/* A couple quicksanity checks */	126	/* A couple quicksanity checks */
119	if ((ob1 == ob2) || (ob1->type != ob2->type)) return 0;	127	if ((ob1 == ob2) || (ob1->type != ob2->type))
		128	return 0;
120		129
121	if (ob1->speed != ob2->speed) return 0;	130	if (ob1->speed != ob2->speed)
122	/* Note sure why the following is the case - either the object has to	131	return 0;
123	* be animated or have a very low speed. Is this an attempted monster	132
124	* check?	133	/* Do not merge objects if nrof would overflow. We use 1UL<<31 since that
		134	* value could not be stored in a sint32 (which unfortunately sometimes is
		135	* used to store nrof).
125	*/	136	*/
126	if (!QUERY_FLAG(ob1,FLAG_ANIMATE) && FABS((ob1)->speed) > MIN_ACTIVE_SPEED)	137	if (ob1->nrof + ob2->nrof >= 1UL << 31)
		138	return 0;
		139
		140	/* If the objects have been identified, set the BEEN_APPLIED flag.
		141	* This is to the comparison of the flags below will be OK. We
		142	* just can't ignore the been applied or identified flags, as they
		143	* are not equal - just if it has been identified, the been_applied
		144	* flags lose any meaning.
		145	*/
		146	if (QUERY_FLAG (ob1, FLAG_IDENTIFIED))
		147	SET_FLAG (ob1, FLAG_BEEN_APPLIED);
		148
		149	if (QUERY_FLAG (ob2, FLAG_IDENTIFIED))
		150	SET_FLAG (ob2, FLAG_BEEN_APPLIED);
		151
		152
		153	/* the 0x400000 on flags2 is FLAG_INV_LOCK. I don't think something
		154	* being locked in inventory should prevent merging.
		155	* 0x4 in flags3 is CLIENT_SENT
		156	*/
		157	if ((ob1->arch != ob2->arch) ||
		158	(ob1->flags[0] != ob2->flags[0]) ||
		159	(ob1->flags[1] != ob2->flags[1]) ||
		160	((ob1->flags[2] & ~0x400000) != (ob2->flags[2] & ~0x400000)) ||
		161	((ob1->flags[3] & ~0x4) != (ob2->flags[3] & ~0x4)) ||
		162	(ob1->name != ob2->name) ||
		163	(ob1->title != ob2->title) ||
		164	(ob1->msg != ob2->msg) ||
		165	(ob1->weight != ob2->weight) ||
		166	(memcmp (&ob1->resist, &ob2->resist, sizeof (ob1->resist)) != 0) ||
		167	(memcmp (&ob1->stats, &ob2->stats, sizeof (ob1->stats)) != 0) ||
		168	(ob1->attacktype != ob2->attacktype) ||
		169	(ob1->magic != ob2->magic) ||
		170	(ob1->slaying != ob2->slaying) ||
		171	(ob1->skill != ob2->skill) ||
		172	(ob1->value != ob2->value) ||
		173	(ob1->animation_id != ob2->animation_id) ||
		174	(ob1->client_type != ob2->client_type) ||
		175	(ob1->materialname != ob2->materialname) ||
		176	(ob1->lore != ob2->lore) ||
		177	(ob1->subtype != ob2->subtype) ||
		178	(ob1->move_type != ob2->move_type) ||
		179	(ob1->move_block != ob2->move_block) ||
		180	(ob1->move_allow != ob2->move_allow) ||
		181	(ob1->move_on != ob2->move_on) ||
		182	(ob1->move_off != ob2->move_off) || (ob1->move_slow != ob2->move_slow) || (ob1->move_slow_penalty != ob2->move_slow_penalty))
		183	return 0;
		184
		185	/* This is really a spellbook check - really, we should
		186	* check all objects in the inventory.
		187	*/
		188	if (ob1->inv || ob2->inv)
		189	{
		190	/* if one object has inventory but the other doesn't, not equiv */
		191	if ((ob1->inv && !ob2->inv) || (ob2->inv && !ob1->inv))
127	return 0;	192	return 0;
128		193
129	/* Do not merge objects if nrof would overflow. We use 1UL<<31 since that	194	/* Now check to see if the two inventory objects could merge */
130	* value could not be stored in a sint32 (which unfortunately sometimes is	195	if (!CAN_MERGE (ob1->inv, ob2->inv))
131	* used to store nrof).
132	*/
133	if (ob1->nrof+ob2->nrof >= 1UL<<31)
134	return 0;	196	return 0;
135		197
136	/* This is really a spellbook check - really, we should
137	* check all objects in the inventory.
138	*/
139	if (ob1->inv || ob2->inv) {
140	/* if one object has inventory but the other doesn't, not equiv */
141	if ((ob1->inv && !ob2->inv) || (ob2->inv && !ob1->inv)) return 0;
142
143	/* Now check to see if the two inventory objects could merge */
144	if (!CAN_MERGE(ob1->inv, ob2->inv)) return 0;
145
146	/* inventory ok - still need to check rest of this object to see	198	/* inventory ok - still need to check rest of this object to see
147	* if it is valid.	199	* if it is valid.
148	*/	200	*/
149	}	201	}
150		202
151	/* If the objects have been identified, set the BEEN_APPLIED flag.	203	/* Don't merge objects that are applied. With the new 'body' code,
152	* This is to the comparison of the flags below will be OK. We	204	* it is possible for most any character to have more than one of
153	* just can't ignore the been applied or identified flags, as they	205	* some items equipped, and we don't want those to merge.
154	* are not equal - just if it has been identified, the been_applied
155	* flags lose any meaning.
156	*/	206	*/
157	if (QUERY_FLAG(ob1, FLAG_IDENTIFIED))	207	if (QUERY_FLAG (ob1, FLAG_APPLIED) || QUERY_FLAG (ob2, FLAG_APPLIED))
158	SET_FLAG(ob1, FLAG_BEEN_APPLIED);	208	return 0;
159		209
160	if (QUERY_FLAG(ob2, FLAG_IDENTIFIED))	210	/* Note sure why the following is the case - either the object has to
161	SET_FLAG(ob2, FLAG_BEEN_APPLIED);	211	* be animated or have a very low speed. Is this an attempted monster
162		212	* check?
163
164	/* the 0x400000 on flags2 is FLAG_INV_LOCK. I don't think something
165	* being locked in inventory should prevent merging.
166	* 0x4 in flags3 is CLIENT_SENT
167	*/	213	*/
168	if ((ob1->arch != ob2->arch) ||	214	if (!QUERY_FLAG (ob1, FLAG_ANIMATE) && FABS ((ob1)->speed) > MIN_ACTIVE_SPEED)
169	(ob1->flags[0] != ob2->flags[0]) ||	215	return 0;
170	(ob1->flags[1] != ob2->flags[1]) ||	216
171	((ob1->flags[2] & ~0x400000) != (ob2->flags[2] & ~ 0x400000)) ||	217	switch (ob1->type)
172	((ob1->flags[3] & ~0x4) != (ob2->flags[3] & ~0x4)) ||	218	{
173	(ob1->name != ob2->name) ||	219	case SCROLL:
174	(ob1->title != ob2->title) ||
175	(ob1->msg != ob2->msg) ||
176	(ob1->weight != ob2->weight) ||
177	(memcmp(&ob1->resist, &ob2->resist, sizeof(ob1->resist))!=0) ||
178	(memcmp(&ob1->stats, &ob2->stats, sizeof(ob1->stats))!=0) ||
179	(ob1->attacktype != ob2->attacktype) ||
180	(ob1->magic != ob2->magic) ||
181	(ob1->slaying != ob2->slaying) ||
182	(ob1->skill != ob2->skill) ||
183	(ob1->value != ob2->value) ||
184	(ob1->animation_id != ob2->animation_id) ||
185	(ob1->client_type != ob2->client_type) ||
186	(ob1->materialname != ob2->materialname) ||
187	(ob1->lore != ob2->lore) ||	220	if (ob1->level != ob2->level)
188	(ob1->subtype != ob2->subtype) ||
189	(ob1->move_type != ob2->move_type) ||
190	(ob1->move_block != ob2->move_block) ||
191	(ob1->move_allow != ob2->move_allow) ||
192	(ob1->move_on != ob2->move_on) ||
193	(ob1->move_off != ob2->move_off) ||
194	(ob1->move_slow != ob2->move_slow) ||
195	(ob1->move_slow_penalty != ob2->move_slow_penalty)
196	)
197	return 0;	221	return 0;
		222	break;
		223	}
198		224
199	/* Don't merge objects that are applied. With the new 'body' code,	225	if (ob1->key_values != NULL || ob2->key_values != NULL)
200	* it is possible for most any character to have more than one of	226	{
201	* some items equipped, and we don't want those to merge.	227	/* At least one of these has key_values. */
202	*/	228	if ((ob1->key_values == NULL) != (ob2->key_values == NULL))
203	if (QUERY_FLAG(ob1, FLAG_APPLIED) || QUERY_FLAG(ob2, FLAG_APPLIED))	229	/* One has fields, but the other one doesn't. */
204	return 0;	230	return 0;
205
206	switch (ob1->type) {
207	case SCROLL:
208	if (ob1->level != ob2->level) return 0;
209	break;
210
211	}
212	if (ob1->key_values != NULL || ob2->key_values != NULL) {
213	/* At least one of these has key_values. */
214	if ((ob1->key_values == NULL) != (ob2->key_values == NULL)) {
215	/* One has fields, but the other one doesn't. */
216	return 0;
217	} else if (!compare_ob_value_lists(ob1, ob2)) {	231	else if (!compare_ob_value_lists (ob1, ob2))
218	return 0;	232	return 0;
219	}	233	}
		234
		235	//TODO: generate an event or call into perl for additional checks
		236	if (ob1->self || ob2->self)
220	}	237	{
		238	ob1->optimise ();
		239	ob2->optimise ();
221		240
		241	if (ob1->self || ob2->self)
		242	return 0;
		243	}
		244
222	/* Everything passes, must be OK. */	245	/* Everything passes, must be OK. */
223	return 1;	246	return 1;
224	}	247	}
225		248
226	/*	249	/*
227	* sum_weight() is a recursive function which calculates the weight	250	* sum_weight() is a recursive function which calculates the weight
228	* an object is carrying. It goes through in figures out how much	251	* an object is carrying. It goes through in figures out how much
229	* containers are carrying, and sums it up.	252	* containers are carrying, and sums it up.
230	*/	253	*/
231	signed long sum_weight(object *op) {	254	signed long
		255	sum_weight (object *op)
		256	{
232	signed long sum;	257	signed long sum;
233	object *inv;	258	object *inv;
		259
234	for(sum = 0, inv = op->inv; inv != NULL; inv = inv->below) {	260	for (sum = 0, inv = op->inv; inv != NULL; inv = inv->below)
		261	{
235	if (inv->inv)	262	if (inv->inv)
236	sum_weight(inv);	263	sum_weight (inv);
237	sum += inv->carrying + inv->weight * (inv->nrof ? inv->nrof : 1);	264	sum += inv->carrying + inv->weight * (inv->nrof ? inv->nrof : 1);
238	}	265	}
239	if (op->type == CONTAINER && op->stats.Str)	266	if (op->type == CONTAINER && op->stats.Str)
240	sum = (sum * (100 - op->stats.Str))/100;	267	sum = (sum * (100 - op->stats.Str)) / 100;
241	if(op->carrying != sum)	268	if (op->carrying != sum)
242	op->carrying = sum;	269	op->carrying = sum;
243	return sum;	270	return sum;
244	}	271	}
245		272
246	/**	273	/**
247	* Return the outermost environment object for a given object.	274	* Return the outermost environment object for a given object.
248	*/	275	*/
249		276
		277	object *
250	object *object_get_env_recursive (object *op) {	278	object_get_env_recursive (object *op)
		279	{
251	while (op->env != NULL)	280	while (op->env != NULL)
252	op = op->env;	281	op = op->env;
253	return op;	282	return op;
254	}	283	}
255		284
256	/*	285	/*
257	* Eneq(@csd.uu.se): Since we can have items buried in a character we need	286	* Eneq(@csd.uu.se): Since we can have items buried in a character we need
258	* a better check. We basically keeping traversing up until we can't	287	* a better check. We basically keeping traversing up until we can't
259	* or find a player.	288	* or find a player.
260	*/	289	*/
261		290
		291	object *
262	object *is_player_inv (object *op) {	292	is_player_inv (object *op)
		293	{
263	for (;op!=NULL&&op->type!=PLAYER; op=op->env)	294	for (; op != NULL && op->type != PLAYER; op = op->env)
264	if (op->env==op)	295	if (op->env == op)
265	op->env = NULL;	296	op->env = NULL;
266	return op;	297	return op;
267	}	298	}
268		299
269	/*	300	/*
270	* Used by: Crossedit: dump. Server DM commands: dumpbelow, dump.	301	* Used by: Crossedit: dump. Server DM commands: dumpbelow, dump.
271	* Some error messages.	302	* Some error messages.
272	* The result of the dump is stored in the static global errmsg array.	303	* The result of the dump is stored in the static global errmsg array.
273	*/	304	*/
274		305
		306	void
275	void dump_object2(object *op) {	307	dump_object2 (object *op)
		308	{
276	errmsg[0] = 0;	309	errmsg[0] = 0;
277	return;	310	return;
278	//TODO//D#d#	311	//TODO//D#d#
279	#if 0	312	#if 0
280	char *cp;	313	char *cp;
		314
281	/* object *tmp;*/	315	/* object *tmp;*/
282		316
283	if(op->arch!=NULL) {	317	if (op->arch != NULL)
		318	{
284	strcat(errmsg,"arch ");	319	strcat (errmsg, "arch ");
285	strcat(errmsg,op->arch->name?op->arch->name:"(null)");	320	strcat (errmsg, op->arch->name ? op->arch->name : "(null)");
286	strcat(errmsg,"\n");	321	strcat (errmsg, "\n");
287	if((cp=get_ob_diff(op,&empty_archetype->clone))!=NULL)	322	if ((cp = get_ob_diff (op, &empty_archetype->clone)) != NULL)
288	strcat(errmsg,cp);	323	strcat (errmsg, cp);
289	#if 0	324	# if 0
290	/* Don't dump player diffs - they are too long, mostly meaningless, and	325	/* Don't dump player diffs - they are too long, mostly meaningless, and
291	* will overflow the buffer.	326	* will overflow the buffer.
292	* Changed so that we don't dump inventory either. This may	327	* Changed so that we don't dump inventory either. This may
293	* also overflow the buffer.	328	* also overflow the buffer.
294	*/	329	*/
295	if(op->type!=PLAYER && (cp=get_ob_diff(op,&empty_archetype->clone))!=NULL)	330	if (op->type != PLAYER && (cp = get_ob_diff (op, &empty_archetype->clone)) != NULL)
296	strcat(errmsg,cp);	331	strcat (errmsg, cp);
297	for (tmp=op->inv; tmp; tmp=tmp->below)	332	for (tmp = op->inv; tmp; tmp = tmp->below)
298	dump_object2(tmp);	333	dump_object2 (tmp);
		334	# endif
		335	strcat (errmsg, "end\n");
		336	}
		337	else
		338	{
		339	strcat (errmsg, "Object ");
		340	if (op->name == NULL)
		341	strcat (errmsg, "(null)");
		342	else
		343	strcat (errmsg, op->name);
		344	strcat (errmsg, "\n");
		345	# if 0
		346	if ((cp = get_ob_diff (op, &empty_archetype->clone)) != NULL)
		347	strcat (errmsg, cp);
		348	for (tmp = op->inv; tmp; tmp = tmp->below)
		349	dump_object2 (tmp);
		350	# endif
		351	strcat (errmsg, "end\n");
		352	}
299	#endif	353	#endif
300	strcat(errmsg,"end\n");
301	} else {
302	strcat(errmsg,"Object ");
303	if (op->name==NULL) strcat(errmsg, "(null)");
304	else strcat(errmsg,op->name);
305	strcat(errmsg,"\n");
306	#if 0
307	if((cp=get_ob_diff(op,&empty_archetype->clone))!=NULL)
308	strcat(errmsg,cp);
309	for (tmp=op->inv; tmp; tmp=tmp->below)
310	dump_object2(tmp);
311	#endif
312	strcat(errmsg,"end\n");
313	}
314	#endif
315	}	354	}
316		355
317	/*	356	/*
318	* Dumps an object. Returns output in the static global errmsg array.	357	* Dumps an object. Returns output in the static global errmsg array.
319	*/	358	*/
320		359
		360	void
321	void dump_object(object *op) {	361	dump_object (object *op)
		362	{
322	if(op==NULL) {	363	if (op == NULL)
		364	{
323	strcpy(errmsg,"[NULL pointer]");	365	strcpy (errmsg, "[NULL pointer]");
324	return;	366	return;
325	}	367	}
326	errmsg[0]='\0';	368	errmsg[0] = '\0';
327	dump_object2(op);	369	dump_object2 (op);
328	}	370	}
329		371
		372	void
330	void dump_all_objects(void) {	373	dump_all_objects (void)
		374	{
331	object *op;	375	object *op;
		376
332	for(op=objects;op!=NULL;op=op->next) {	377	for (op = objects; op != NULL; op = op->next)
		378	{
333	dump_object(op);	379	dump_object (op);
334	fprintf(logfile, "Object %d\n:%s\n", op->count, errmsg);	380	fprintf (logfile, "Object %d\n:%s\n", op->count, errmsg);
335	}	381	}
336	}	382	}
337		383
338	/*	384	/*
339	* get_nearest_part(multi-object, object 2) returns the part of the	385	* get_nearest_part(multi-object, object 2) returns the part of the
340	* multi-object 1 which is closest to the second object.	386	* multi-object 1 which is closest to the second object.
341	* If it's not a multi-object, it is returned.	387	* If it's not a multi-object, it is returned.
342	*/	388	*/
343		389
		390	object *
344	object *get_nearest_part(object *op, const object *pl) {	391	get_nearest_part (object *op, const object *pl)
		392	{
345	object *tmp,*closest;	393	object *tmp, *closest;
346	int last_dist,i;	394	int last_dist, i;
		395
347	if(op->more==NULL)	396	if (op->more == NULL)
348	return op;	397	return op;
349	for(last_dist=distance(op,pl),closest=op,tmp=op->more;tmp!=NULL;tmp=tmp->more)	398	for (last_dist = distance (op, pl), closest = op, tmp = op->more; tmp != NULL; tmp = tmp->more)
350	if((i=distance(tmp,pl))<last_dist)	399	if ((i = distance (tmp, pl)) < last_dist)
351	closest=tmp,last_dist=i;	400	closest = tmp, last_dist = i;
352	return closest;	401	return closest;
353	}	402	}
354		403
355	/*	404	/*
356	* Returns the object which has the count-variable equal to the argument.	405	* Returns the object which has the count-variable equal to the argument.
357	*/	406	*/
358		407
		408	object *
359	object *find_object(tag_t i) {	409	find_object (tag_t i)
		410	{
360	object *op;	411	object *op;
		412
361	for(op=objects;op!=NULL;op=op->next)	413	for (op = objects; op != NULL; op = op->next)
362	if(op->count==i)	414	if (op->count == i)
363	break;	415	break;
364	return op;	416	return op;
365	}	417	}
366		418
367	/*	419	/*
368	* Returns the first object which has a name equal to the argument.	420	* Returns the first object which has a name equal to the argument.
369	* Used only by the patch command, but not all that useful.	421	* Used only by the patch command, but not all that useful.
370	* Enables features like "patch <name-of-other-player> food 999"	422	* Enables features like "patch <name-of-other-player> food 999"
371	*/	423	*/
372		424
		425	object *
373	object *find_object_name(const char *str) {	426	find_object_name (const char *str)
		427	{
374	const char *name = shstr::find (str);	428	const char *name = shstr::find (str);
375	object *op;	429	object *op;
		430
376	for(op=objects;op!=NULL;op=op->next)	431	for (op = objects; op != NULL; op = op->next)
377	if(&op->name == name)	432	if (&op->name == name)
378	break;	433	break;
379		434
380	return op;	435	return op;
381	}	436	}
382		437
		438	void
383	void free_all_object_data ()	439	free_all_object_data ()
384	{	440	{
385	LOG (llevDebug, "%d allocated objects\n", nrofallocobjects);	441	LOG (llevDebug, "%d allocated objects\n", nrofallocobjects);
386	}	442	}
387		443
388	/*	444	/*
…		…
394	* prompt, he is removed, and we don't want to treat him as an owner of	450	* prompt, he is removed, and we don't want to treat him as an owner of
395	* anything, so check removed flag. I don't expect that this should break	451	* anything, so check removed flag. I don't expect that this should break
396	* anything - once an object is removed, it is basically dead anyways.	452	* anything - once an object is removed, it is basically dead anyways.
397	*/	453	*/
398		454
		455	object *
399	object *get_owner(object *op) {	456	get_owner (object *op)
		457	{
400	if(op->owner==NULL)	458	if (op->owner == NULL)
401	return NULL;
402
403	if (!QUERY_FLAG(op->owner,FLAG_FREED) && !QUERY_FLAG(op->owner, FLAG_REMOVED) &&
404	op->owner->count==op->ownercount)
405	return op->owner;
406
407	op->owner=NULL;
408	op->ownercount=0;
409	return NULL;	459	return NULL;
410	}
411		460
		461	if (!QUERY_FLAG (op->owner, FLAG_FREED) && !QUERY_FLAG (op->owner, FLAG_REMOVED) && op->owner->count == op->ownercount)
		462	return op->owner;
		463
		464	op->owner = NULL;
		465	op->ownercount = 0;
		466	return NULL;
		467	}
		468
		469	void
412	void clear_owner(object *op)	470	clear_owner (object *op)
413	{	471	{
414	if (!op) return;	472	if (!op)
		473	return;
415		474
416	if (op->owner && op->ownercount == op->owner->count)	475	if (op->owner && op->ownercount == op->owner->count)
417	op->owner->refcount--;	476	op->owner->refcount--;
418		477
419	op->owner = NULL;	478	op->owner = NULL;
420	op->ownercount = 0;	479	op->ownercount = 0;
421	}	480	}
422		481
423	/*	482	/*
424	* Sets the owner and sets the skill and exp pointers to owner's current	483	* Sets the owner and sets the skill and exp pointers to owner's current
425	* skill and experience objects.	484	* skill and experience objects.
426	*/	485	*/
		486	void
427	void set_owner (object *op, object *owner)	487	set_owner (object *op, object *owner)
428	{	488	{
429	if(owner==NULL||op==NULL)	489	if (owner == NULL || op == NULL)
430	return;	490	return;
431		491
432	/* next line added to allow objects which own objects */	492	/* next line added to allow objects which own objects */
433	/* Add a check for ownercounts in here, as I got into an endless loop	493	/* Add a check for ownercounts in here, as I got into an endless loop
434	* with the fireball owning a poison cloud which then owned the	494	* with the fireball owning a poison cloud which then owned the
435	* fireball. I believe that was caused by one of the objects getting	495	* fireball. I believe that was caused by one of the objects getting
436	* freed and then another object replacing it. Since the ownercounts	496	* freed and then another object replacing it. Since the ownercounts
437	* didn't match, this check is valid and I believe that cause is valid.	497	* didn't match, this check is valid and I believe that cause is valid.
438	*/	498	*/
439	while (owner->owner && owner!=owner->owner &&	499	while (owner->owner && owner != owner->owner && owner->ownercount == owner->owner->count)
440	owner->ownercount==owner->owner->count) owner=owner->owner;	500	owner = owner->owner;
441		501
442	/* IF the owner still has an owner, we did not resolve to a final owner.	502	/* IF the owner still has an owner, we did not resolve to a final owner.
443	* so lets not add to that.	503	* so lets not add to that.
444	*/	504	*/
445	if (owner->owner) return;	505	if (owner->owner)
		506	return;
446		507
447	op->owner=owner;	508	op->owner = owner;
448		509
449	op->ownercount=owner->count;	510	op->ownercount = owner->count;
450	owner->refcount++;	511	owner->refcount++;
451		512
452	}	513	}
453		514
454	/* Set the owner to clone's current owner and set the skill and experience	515	/* Set the owner to clone's current owner and set the skill and experience
455	* objects to clone's objects (typically those objects that where the owner's	516	* objects to clone's objects (typically those objects that where the owner's
…		…
459	* Use this function if player created an object (e.g. fire bullet, swarm	520	* Use this function if player created an object (e.g. fire bullet, swarm
460	* spell), and this object creates further objects whose kills should be	521	* spell), and this object creates further objects whose kills should be
461	* accounted for the player's original skill, even if player has changed	522	* accounted for the player's original skill, even if player has changed
462	* skills meanwhile.	523	* skills meanwhile.
463	*/	524	*/
		525	void
464	void copy_owner (object *op, object *clone)	526	copy_owner (object *op, object *clone)
465	{	527	{
466	object *owner = get_owner (clone);	528	object *owner = get_owner (clone);
		529
467	if (owner == NULL) {	530	if (owner == NULL)
		531	{
468	/* players don't have owners - they own themselves. Update	532	/* players don't have owners - they own themselves. Update
469	* as appropriate.	533	* as appropriate.
470	*/	534	*/
471	if (clone->type == PLAYER) owner=clone;	535	if (clone->type == PLAYER)
		536	owner = clone;
		537	else
472	else return;	538	return;
473	}	539	}
474	set_owner(op, owner);	540	set_owner (op, owner);
475		541
476	}	542	}
477		543
478	/* Zero the key_values on op, decrementing the shared-string	544	/* Zero the key_values on op, decrementing the shared-string
479	* refcounts and freeing the links.	545	* refcounts and freeing the links.
480	*/	546	*/
		547	static void
481	static void free_key_values(object * op)	548	free_key_values (object *op)
482	{	549	{
483	for (key_value *i = op->key_values; i != 0; )	550	for (key_value *i = op->key_values; i != 0;)
484	{	551	{
485	key_value *next = i->next;	552	key_value *next = i->next;
486	delete i;	553	delete i;
		554
487	i = next;	555	i = next;
488	}	556	}
489		557
490	op->key_values = 0;	558	op->key_values = 0;
491	}	559	}
492		560
493	void object::clear ()	561	void object::clear ()
494	{	562	{
495	attachable_base::clear ();	563	attachable_base::clear ();
496		564
497	free_key_values (this);	565	free_key_values (this);
498		566
499	name = 0;	567	name = 0;
500	name_pl = 0;	568	name_pl = 0;
501	title = 0;	569	title = 0;
502	race = 0;	570	race = 0;
503	slaying = 0;	571	slaying = 0;
504	skill = 0;	572	skill = 0;
505	msg = 0;	573	msg = 0;
506	lore = 0;	574	lore = 0;
507	custom_name = 0;	575	custom_name = 0;
508	materialname = 0;	576	materialname = 0;
509		577
510	memset (static_cast<object_pod *>(this), 0, sizeof (object_pod));	578	memset (static_cast < object_pod * >(this), 0, sizeof (object_pod));
511		579
512	SET_FLAG (this, FLAG_REMOVED);	580	SET_FLAG (this, FLAG_REMOVED);
513	}	581	}
514		582
515	void object::clone (object *destination)	583	void object::clone (object *destination)
516	{	584	{
517	*(object_copy *)destination = *(object_copy *)this;	585	*(object_copy *) destination = *this;
518	*(object_pod *)destination = *(object_pod *)this;	586	*(object_pod *) destination = *this;
519		587
520	if (self || cb)	588	if (self || cb)
521	INVOKE_OBJECT (CLONE, this, ARG_OBJECT (destination));	589	INVOKE_OBJECT (CLONE, this, ARG_OBJECT (destination));
522	}	590	}
523		591
524	/*	592	/*
525	* clear_object() frees everything allocated by an object, and also	593	* clear_object() frees everything allocated by an object, and also
526	* clears all variables and flags to default settings.	594	* clears all variables and flags to default settings.
527	*/	595	*/
528		596
		597	void
529	void clear_object (object *op)	598	clear_object (object *op)
530	{	599	{
531	op->clear ();	600	op->clear ();
532		601
533	op->contr = NULL;	602	op->contr = NULL;
534	op->below = NULL;	603	op->below = NULL;
535	op->above = NULL;	604	op->above = NULL;
536	op->inv = NULL;	605	op->inv = NULL;
537	op->container=NULL;	606	op->container = NULL;
538	op->env=NULL;	607	op->env = NULL;
539	op->more=NULL;	608	op->more = NULL;
540	op->head=NULL;	609	op->head = NULL;
541	op->map=NULL;	610	op->map = NULL;
542	op->refcount=0;	611	op->refcount = 0;
543	op->active_next = NULL;	612	op->active_next = NULL;
544	op->active_prev = NULL;	613	op->active_prev = NULL;
545	/* What is not cleared is next, prev, and count */	614	/* What is not cleared is next, prev, and count */
546		615
547	op->expmul = 1.0;	616	op->expmul = 1.0;
…		…
559	* data that is malloc'd needs to be re-malloc/copied. Otherwise,	628	* data that is malloc'd needs to be re-malloc/copied. Otherwise,
560	* if the first object is freed, the pointers in the new object	629	* if the first object is freed, the pointers in the new object
561	* will point at garbage.	630	* will point at garbage.
562	*/	631	*/
563		632
		633	void
564	void copy_object (object *op2, object *op)	634	copy_object (object *op2, object *op)
565	{	635	{
566	bool is_freed = QUERY_FLAG (op, FLAG_FREED);	636	bool is_freed = QUERY_FLAG (op, FLAG_FREED);
567	bool is_removed = QUERY_FLAG (op, FLAG_REMOVED);	637	bool is_removed = QUERY_FLAG (op, FLAG_REMOVED);
568		638
569	op2->clone (op);	639	op2->clone (op);
570		640
		641	if (is_freed)
571	if (is_freed) SET_FLAG (op, FLAG_FREED);	642	SET_FLAG (op, FLAG_FREED);
		643	if (is_removed)
572	if (is_removed) SET_FLAG (op, FLAG_REMOVED);	644	SET_FLAG (op, FLAG_REMOVED);
573		645
574	if (op2->speed < 0)	646	if (op2->speed < 0)
575	op->speed_left = op2->speed_left - RANDOM() % 200 / 100.0;	647	op->speed_left = op2->speed_left - RANDOM () % 200 / 100.0;
576		648
577	/* Copy over key_values, if any. */	649	/* Copy over key_values, if any. */
578	if (op2->key_values != NULL)	650	if (op2->key_values)
579	{	651	{
580	key_value *tail = NULL;	652	key_value *tail = 0;
581	key_value *i;	653	key_value *i;
582		654
583	op->key_values = NULL;	655	op->key_values = 0;
584		656
585	for (i = op2->key_values; i != NULL; i = i->next)	657	for (i = op2->key_values; i; i = i->next)
586	{	658	{
587	key_value *new_link = new key_value;	659	key_value *new_link = new key_value;
588		660
589	new_link->next = NULL;	661	new_link->next = 0;
590	new_link->key = i->key;	662	new_link->key = i->key;
591	new_link->value = i->value;	663	new_link->value = i->value;
592		664
593	/* Try and be clever here, too. */	665	/* Try and be clever here, too. */
594	if (op->key_values == NULL)	666	if (!op->key_values)
595	{	667	{
596	op->key_values = new_link;	668	op->key_values = new_link;
597	tail = new_link;	669	tail = new_link;
598	}	670	}
599	else	671	else
…		…
606		678
607	update_ob_speed (op);	679	update_ob_speed (op);
608	}	680	}
609		681
610	/*	682	/*
611	* get_object() grabs an object from the list of unused objects, makes
612	* sure it is initialised, and returns it.
613	* If there are no free objects, expand_objects() is called to get more.
614	*/
615
616	object *get_object ()
617	{
618	object *op = new object;
619
620	op->count = ++ob_count;
621
622	op->active_next = 0;
623	op->active_prev = 0;
624
625	op->next = objects;
626	op->prev = 0;
627
628	if (objects)
629	objects->prev = op;
630
631	objects = op;
632
633	SET_FLAG (op, FLAG_REMOVED);
634
635	op->expmul = 1.0;
636	op->face = blank_face;
637	op->attacked_by_count = -1;
638
639	return op;
640	}
641
642	/*
643	* If an object with the IS_TURNABLE() flag needs to be turned due	683	* If an object with the IS_TURNABLE() flag needs to be turned due
644	* to the closest player being on the other side, this function can	684	* to the closest player being on the other side, this function can
645	* be called to update the face variable, _and_ how it looks on the map.	685	* be called to update the face variable, _and_ how it looks on the map.
646	*/	686	*/
647		687
		688	void
648	void update_turn_face(object *op) {	689	update_turn_face (object *op)
		690	{
649	if(!QUERY_FLAG(op,FLAG_IS_TURNABLE)||op->arch==NULL)	691	if (!QUERY_FLAG (op, FLAG_IS_TURNABLE) || op->arch == NULL)
650	return;	692	return;
651	SET_ANIMATION(op, op->direction);	693	SET_ANIMATION (op, op->direction);
652	update_object(op,UP_OBJ_FACE);	694	update_object (op, UP_OBJ_FACE);
653	}	695	}
654		696
655	/*	697	/*
656	* Updates the speed of an object. If the speed changes from 0 to another	698	* Updates the speed of an object. If the speed changes from 0 to another
657	* value, or vice versa, then add/remove the object from the active list.	699	* value, or vice versa, then add/remove the object from the active list.
658	* This function needs to be called whenever the speed of an object changes.	700	* This function needs to be called whenever the speed of an object changes.
659	*/	701	*/
660		702
		703	void
661	void update_ob_speed(object *op) {	704	update_ob_speed (object *op)
		705	{
662	extern int arch_init;	706	extern int arch_init;
663		707
664	/* No reason putting the archetypes objects on the speed list,	708	/* No reason putting the archetypes objects on the speed list,
665	* since they never really need to be updated.	709	* since they never really need to be updated.
666	*/	710	*/
667		711
668	if (QUERY_FLAG(op, FLAG_FREED) && op->speed) {	712	if (QUERY_FLAG (op, FLAG_FREED) && op->speed)
		713	{
669	LOG(llevError,"Object %s is freed but has speed.\n", &op->name);	714	LOG (llevError, "Object %s is freed but has speed.\n", &op->name);
670	#ifdef MANY_CORES	715	#ifdef MANY_CORES
671	abort();	716	abort ();
672	#else	717	#else
673	op->speed = 0;	718	op->speed = 0;
674	#endif	719	#endif
675	}	720	}
676	if (arch_init) {	721	if (arch_init)
		722	{
		723	return;
		724	}
		725	if (FABS (op->speed) > MIN_ACTIVE_SPEED)
		726	{
		727	/* If already on active list, don't do anything */
		728	if (op->active_next || op->active_prev || op == active_objects)
677	return;	729	return;
678	}
679	if (FABS(op->speed)>MIN_ACTIVE_SPEED) {
680	/* If already on active list, don't do anything */
681	if (op->active_next || op->active_prev || op==active_objects)
682	return;
683		730
684	/* process_events() expects us to insert the object at the beginning	731	/* process_events() expects us to insert the object at the beginning
685	* of the list. */	732	* of the list. */
686	op->active_next = active_objects;	733	op->active_next = active_objects;
687	if (op->active_next!=NULL)	734	if (op->active_next != NULL)
688	op->active_next->active_prev = op;	735	op->active_next->active_prev = op;
689	active_objects = op;	736	active_objects = op;
		737	}
		738	else
690	}	739	{
691	else {
692	/* If not on the active list, nothing needs to be done */	740	/* If not on the active list, nothing needs to be done */
693	if (!op->active_next && !op->active_prev && op!=active_objects)	741	if (!op->active_next && !op->active_prev && op != active_objects)
694	return;	742	return;
695		743
696	if (op->active_prev==NULL) {	744	if (op->active_prev == NULL)
		745	{
697	active_objects = op->active_next;	746	active_objects = op->active_next;
698	if (op->active_next!=NULL)	747	if (op->active_next != NULL)
699	op->active_next->active_prev = NULL;	748	op->active_next->active_prev = NULL;
		749	}
		750	else
700	}	751	{
701	else {
702	op->active_prev->active_next = op->active_next;	752	op->active_prev->active_next = op->active_next;
703	if (op->active_next)	753	if (op->active_next)
704	op->active_next->active_prev = op->active_prev;	754	op->active_next->active_prev = op->active_prev;
705	}	755	}
706	op->active_next = NULL;	756	op->active_next = NULL;
707	op->active_prev = NULL;	757	op->active_prev = NULL;
708	}	758	}
709	}	759	}
710		760
711	/* This function removes object 'op' from the list of active	761	/* This function removes object 'op' from the list of active
712	* objects.	762	* objects.
…		…
714	* reference maps where you don't want an object that isn't	764	* reference maps where you don't want an object that isn't
715	* in play chewing up cpu time getting processed.	765	* in play chewing up cpu time getting processed.
716	* The reverse of this is to call update_ob_speed, which	766	* The reverse of this is to call update_ob_speed, which
717	* will do the right thing based on the speed of the object.	767	* will do the right thing based on the speed of the object.
718	*/	768	*/
		769	void
719	void remove_from_active_list(object *op)	770	remove_from_active_list (object *op)
720	{	771	{
721	/* If not on the active list, nothing needs to be done */	772	/* If not on the active list, nothing needs to be done */
722	if (!op->active_next && !op->active_prev && op!=active_objects)	773	if (!op->active_next && !op->active_prev && op != active_objects)
723	return;	774	return;
724		775
725	if (op->active_prev==NULL) {	776	if (op->active_prev == NULL)
		777	{
726	active_objects = op->active_next;	778	active_objects = op->active_next;
727	if (op->active_next!=NULL)	779	if (op->active_next != NULL)
728	op->active_next->active_prev = NULL;	780	op->active_next->active_prev = NULL;
		781	}
		782	else
729	}	783	{
730	else {
731	op->active_prev->active_next = op->active_next;	784	op->active_prev->active_next = op->active_next;
732	if (op->active_next)	785	if (op->active_next)
733	op->active_next->active_prev = op->active_prev;	786	op->active_next->active_prev = op->active_prev;
734	}	787	}
735	op->active_next = NULL;	788	op->active_next = NULL;
736	op->active_prev = NULL;	789	op->active_prev = NULL;
737	}	790	}
738		791
739	/*	792	/*
740	* update_object() updates the array which represents the map.	793	* update_object() updates the array which represents the map.
741	* It takes into account invisible objects (and represent squares covered	794	* It takes into account invisible objects (and represent squares covered
…		…
756	* UP_OBJ_CHANGE: object has somehow changed. In this case, we always update	809	* UP_OBJ_CHANGE: object has somehow changed. In this case, we always update
757	* as that is easier than trying to look at what may have changed.	810	* as that is easier than trying to look at what may have changed.
758	* UP_OBJ_FACE: only the objects face has changed.	811	* UP_OBJ_FACE: only the objects face has changed.
759	*/	812	*/
760		813
		814	void
761	void update_object(object *op, int action) {	815	update_object (object *op, int action)
		816	{
762	int update_now=0, flags;	817	int update_now = 0, flags;
763	MoveType move_on, move_off, move_block, move_slow;	818	MoveType move_on, move_off, move_block, move_slow;
764		819
765	if (op == NULL) {	820	if (op == NULL)
		821	{
766	/* this should never happen */	822	/* this should never happen */
767	LOG(llevDebug,"update_object() called for NULL object.\n");	823	LOG (llevDebug, "update_object() called for NULL object.\n");
768	return;	824	return;
769	}
770		825	}
		826
771	if(op->env!=NULL) {	827	if (op->env != NULL)
		828	{
772	/* Animation is currently handled by client, so nothing	829	/* Animation is currently handled by client, so nothing
773	* to do in this case.	830	* to do in this case.
774	*/	831	*/
775	return;	832	return;
776	}	833	}
777		834
778	/* If the map is saving, don't do anything as everything is	835	/* If the map is saving, don't do anything as everything is
779	* going to get freed anyways.	836	* going to get freed anyways.
780	*/	837	*/
781	if (!op->map || op->map->in_memory == MAP_SAVING) return;	838	if (!op->map || op->map->in_memory == MAP_SAVING)
782		839	return;
		840
783	/* make sure the object is within map boundaries */	841	/* make sure the object is within map boundaries */
784	if (op->x < 0 || op->x >= MAP_WIDTH(op->map) ||	842	if (op->x < 0 || op->x >= MAP_WIDTH (op->map) || op->y < 0 || op->y >= MAP_HEIGHT (op->map))
785	op->y < 0 || op->y >= MAP_HEIGHT(op->map)) {	843	{
786	LOG(llevError,"update_object() called for object out of map!\n");	844	LOG (llevError, "update_object() called for object out of map!\n");
787	#ifdef MANY_CORES	845	#ifdef MANY_CORES
788	abort();	846	abort ();
789	#endif	847	#endif
790	return;	848	return;
791	}
792		849	}
		850
793	flags = GET_MAP_FLAGS(op->map, op->x, op->y);	851	flags = GET_MAP_FLAGS (op->map, op->x, op->y);
794	SET_MAP_FLAGS(op->map, op->x, op->y, flags | P_NEED_UPDATE);	852	SET_MAP_FLAGS (op->map, op->x, op->y, flags | P_NEED_UPDATE);
795	move_slow = GET_MAP_MOVE_SLOW(op->map, op->x, op->y);	853	move_slow = GET_MAP_MOVE_SLOW (op->map, op->x, op->y);
796	move_on = GET_MAP_MOVE_ON(op->map, op->x, op->y);	854	move_on = GET_MAP_MOVE_ON (op->map, op->x, op->y);
797	move_block = GET_MAP_MOVE_BLOCK(op->map, op->x, op->y);	855	move_block = GET_MAP_MOVE_BLOCK (op->map, op->x, op->y);
798	move_off = GET_MAP_MOVE_OFF(op->map, op->x, op->y);	856	move_off = GET_MAP_MOVE_OFF (op->map, op->x, op->y);
799		857
800	if (action == UP_OBJ_INSERT) {	858	if (action == UP_OBJ_INSERT)
		859	{
801	if (QUERY_FLAG(op, FLAG_BLOCKSVIEW) && !(flags & P_BLOCKSVIEW))	860	if (QUERY_FLAG (op, FLAG_BLOCKSVIEW) && !(flags & P_BLOCKSVIEW))
802	update_now=1;	861	update_now = 1;
803		862
804	if (QUERY_FLAG(op, FLAG_NO_MAGIC) && !(flags & P_NO_MAGIC))	863	if (QUERY_FLAG (op, FLAG_NO_MAGIC) && !(flags & P_NO_MAGIC))
805	update_now=1;	864	update_now = 1;
806		865
807	if (QUERY_FLAG(op, FLAG_DAMNED) && !(flags & P_NO_CLERIC))	866	if (QUERY_FLAG (op, FLAG_DAMNED) && !(flags & P_NO_CLERIC))
808	update_now=1;	867	update_now = 1;
809		868
810	if (QUERY_FLAG(op, FLAG_ALIVE) && !(flags & P_IS_ALIVE))	869	if (QUERY_FLAG (op, FLAG_ALIVE) && !(flags & P_IS_ALIVE))
811	update_now=1;	870	update_now = 1;
812		871
		872	if (op->type == SAFE_GROUND && !(flags & P_SAFE))
		873	update_now = 1;
		874
813	if ((move_on | op->move_on) != move_on) update_now=1;	875	if ((move_on | op->move_on) != move_on)
		876	update_now = 1;
		877
814	if ((move_off | op->move_off) != move_off) update_now=1;	878	if ((move_off | op->move_off) != move_off)
		879	update_now = 1;
		880
815	/* This isn't perfect, but I don't expect a lot of objects to	881	/* This isn't perfect, but I don't expect a lot of objects to
816	* to have move_allow right now.	882	* to have move_allow right now.
817	*/	883	*/
818	if (((move_block | op->move_block) & ~op->move_allow) != move_block)	884	if (((move_block | op->move_block) & ~op->move_allow) != move_block)
819	update_now=1;	885	update_now = 1;
		886
820	if ((move_slow | op->move_slow) != move_slow) update_now=1;	887	if ((move_slow | op->move_slow) != move_slow)
		888	update_now = 1;
821	}	889	}
822	/* if the object is being removed, we can't make intelligent	890	/* if the object is being removed, we can't make intelligent
823	* decisions, because remove_ob can't really pass the object	891	* decisions, because remove_ob can't really pass the object
824	* that is being removed.	892	* that is being removed.
825	*/	893	*/
826	else if (action == UP_OBJ_CHANGE || action == UP_OBJ_REMOVE) {	894	else if (action == UP_OBJ_CHANGE || action == UP_OBJ_REMOVE)
		895	{
827	update_now=1;	896	update_now = 1;
		897	}
828	} else if (action == UP_OBJ_FACE) {	898	else if (action == UP_OBJ_FACE)
		899	{
829	/* Nothing to do for that case */	900	/* Nothing to do for that case */
		901	}
		902	else
830	}	903	{
831	else {
832	LOG(llevError,"update_object called with invalid action: %d\n", action);	904	LOG (llevError, "update_object called with invalid action: %d\n", action);
833	}	905	}
834		906
835	if (update_now) {	907	if (update_now)
		908	{
836	SET_MAP_FLAGS(op->map, op->x, op->y, flags | P_NO_ERROR | P_NEED_UPDATE);	909	SET_MAP_FLAGS (op->map, op->x, op->y, flags | P_NO_ERROR | P_NEED_UPDATE);
837	update_position(op->map, op->x, op->y);	910	update_position (op->map, op->x, op->y);
838	}	911	}
839		912
840	if(op->more!=NULL)	913	if (op->more != NULL)
841	update_object(op->more, action);	914	update_object (op->more, action);
842	}	915	}
843		916
		917	static
		918	std::vector < object *>
		919	mortals;
		920
		921	void object::free_mortals ()
		922	{
		923	for (std::vector < object *>::iterator i = mortals.begin (); i != mortals.end (); ++i)
		924	delete *
		925	i;
		926
		927	mortals.clear ();
		928	}
		929
		930	object::object ()
		931	{
		932	SET_FLAG (this, FLAG_REMOVED);
		933
		934	expmul = 1.0;
		935	face = blank_face;
		936	attacked_by_count = -1;
		937	}
		938
		939	object::~object ()
		940	{
		941	free_key_values (this);
		942	}
		943
		944	void object::link ()
		945	{
		946	count = ++ob_count;
		947
		948	prev = 0;
		949	next = objects;
		950
		951	if (objects)
		952	objects->prev = this;
		953
		954	objects = this;
		955	}
		956
		957	void object::unlink ()
		958	{
		959	count = 0;
		960
		961	/* Remove this object from the list of used objects */
		962	if (prev)
		963	prev->next = next;
		964	if (next)
		965	next->prev = prev;
		966	if (this == objects)
		967	objects = next;
		968	}
		969
		970	object *object::create ()
		971	{
		972	object *
		973	op = new object;
		974
		975	op->link ();
		976	return op;
		977	}
844		978
845	/*	979	/*
846	* free_object() frees everything allocated by an object, removes	980	* free_object() frees everything allocated by an object, removes
847	* it from the list of used objects, and puts it on the list of	981	* it from the list of used objects, and puts it on the list of
848	* free objects. The IS_FREED() flag is set in the object.	982	* free objects. The IS_FREED() flag is set in the object.
…		…
850	* this function to succeed.	984	* this function to succeed.
851	*	985	*
852	* If free_inventory is set, free inventory as well. Else drop items in	986	* If free_inventory is set, free inventory as well. Else drop items in
853	* inventory to the ground.	987	* inventory to the ground.
854	*/	988	*/
855		989	void object::free (bool free_inventory)
856	void
857	free_object (object * ob)
858	{	990	{
859	free_object2 (ob, 0);
860	}
861
862	void
863	free_object2 (object * ob, int free_inventory)
864	{
865	object *tmp, *op;
866
867	if (!QUERY_FLAG (ob, FLAG_REMOVED))	991	if (!QUERY_FLAG (this, FLAG_REMOVED))
868	{	992	{
869	LOG (llevDebug, "Free object called with non removed object\n");	993	LOG (llevDebug, "Free object called with non removed object\n");
870	dump_object (ob);	994	dump_object (this);
871	#ifdef MANY_CORES	995	#ifdef MANY_CORES
872	abort ();	996	abort ();
873	#endif	997	#endif
874	}	998	}
875		999
876	if (QUERY_FLAG (ob, FLAG_FRIENDLY))	1000	if (QUERY_FLAG (this, FLAG_FRIENDLY))
877	{	1001	{
878	LOG (llevMonster, "Warning: tried to free friendly object.\n");	1002	LOG (llevMonster, "Warning: tried to free friendly object.\n");
879	remove_friendly_object (ob);	1003	remove_friendly_object (this);
880	}	1004	}
881		1005
882	if (QUERY_FLAG (ob, FLAG_FREED))	1006	if (QUERY_FLAG (this, FLAG_FREED))
883	{	1007	{
884	dump_object (ob);	1008	dump_object (this);
885	LOG (llevError, "Trying to free freed object.\n%s\n", errmsg);	1009	LOG (llevError, "Trying to free freed object.\n%s\n", errmsg);
886	return;	1010	return;
887	}	1011	}
888		1012
889	if (ob->more != NULL)	1013	if (more)
890	{
891	free_object2 (ob->more, free_inventory);
892	ob->more = NULL;
893	}	1014	{
		1015	more->free (free_inventory);
		1016	more = 0;
		1017	}
894		1018
895	if (ob->inv)	1019	if (inv)
896	{	1020	{
897	/* Only if the space blocks everything do we not process -	1021	/* Only if the space blocks everything do we not process -
898	* if some form of movemnt is allowed, let objects	1022	* if some form of movement is allowed, let objects
899	* drop on that space.	1023	* drop on that space.
900	*/	1024	*/
901	if (free_inventory || ob->map == NULL	1025	if (free_inventory || !map || map->in_memory != MAP_IN_MEMORY || (GET_MAP_MOVE_BLOCK (map, x, y) == MOVE_ALL))
902	|| ob->map->in_memory != MAP_IN_MEMORY	1026	{
903	|| (GET_MAP_MOVE_BLOCK (ob->map, ob->x, ob->y) == MOVE_ALL))	1027	object *
904	{	1028	op = inv;
905	op = ob->inv;
906		1029
907	while (op != NULL)	1030	while (op)
908	{	1031	{
		1032	object *
909	tmp = op->below;	1033	tmp = op->below;
		1034
910	remove_ob (op);	1035	remove_ob (op);
911	free_object2 (op, free_inventory);	1036	op->free (free_inventory);
912	op = tmp;	1037	op = tmp;
913	}	1038	}
914	}	1039	}
915	else	1040	else
916	{ /* Put objects in inventory onto this space */	1041	{ /* Put objects in inventory onto this space */
917	op = ob->inv;	1042	object *
		1043	op = inv;
918		1044
919	while (op != NULL)	1045	while (op)
920	{	1046	{
		1047	object *
921	tmp = op->below;	1048	tmp = op->below;
		1049
922	remove_ob (op);	1050	remove_ob (op);
923		1051
924	if (QUERY_FLAG (op, FLAG_STARTEQUIP)	1052	if (QUERY_FLAG (op, FLAG_STARTEQUIP)
925	|| QUERY_FLAG (op, FLAG_NO_DROP) || op->type == RUNE	1053	|| QUERY_FLAG (op, FLAG_NO_DROP) || op->type == RUNE || op->type == TRAP || QUERY_FLAG (op, FLAG_IS_A_TEMPLATE))
926	|| op->type == TRAP || QUERY_FLAG (op, FLAG_IS_A_TEMPLATE))	1054	free_object (op);
927	free_object (op);	1055	else
928	else	1056	{
929	{	1057	op->x = x;
930	op->x = ob->x;	1058	op->y = y;
931	op->y = ob->y;
932	insert_ob_in_map (op, ob->map, NULL, 0); /* Insert in same map as the envir */	1059	insert_ob_in_map (op, map, 0, 0); /* Insert in same map as the envir */
933	}	1060	}
934		1061
935	op = tmp;	1062	op = tmp;
936	}	1063	}
937	}	1064	}
938	}	1065	}
939		1066
940	/* Remove object from the active list */	1067	/* Remove object from the active list */
941	ob->speed = 0;	1068	speed = 0;
942	update_ob_speed (ob);	1069	update_ob_speed (this);
943		1070
		1071	unlink ();
		1072
944	SET_FLAG (ob, FLAG_FREED);	1073	SET_FLAG (this, FLAG_FREED);
945	ob->count = 0;
946		1074
947	/* Remove this object from the list of used objects */	1075	mortals.push_back (this);
948	if (ob->prev == NULL)
949	{
950	objects = ob->next;
951
952	if (objects != NULL)
953	objects->prev = NULL;
954	}
955	else
956	{
957	ob->prev->next = ob->next;
958
959	if (ob->next != NULL)
960	ob->next->prev = ob->prev;
961	}
962
963	free_key_values (ob);
964
965	/* Now link it with the free_objects list: */
966	ob->prev = 0;
967	ob->next = 0;
968
969	delete ob;
970	}	1076	}
971		1077
972	/*	1078	/*
973	* sub_weight() recursively (outwards) subtracts a number from the	1079	* sub_weight() recursively (outwards) subtracts a number from the
974	* weight of an object (and what is carried by it's environment(s)).	1080	* weight of an object (and what is carried by it's environment(s)).
975	*/	1081	*/
976		1082
		1083	void
977	void sub_weight (object *op, signed long weight) {	1084	sub_weight (object *op, signed long weight)
		1085	{
978	while (op != NULL) {	1086	while (op != NULL)
		1087	{
979	if (op->type == CONTAINER) {	1088	if (op->type == CONTAINER)
		1089	{
980	weight=(signed long)(weight*(100-op->stats.Str)/100);	1090	weight = (signed long) (weight * (100 - op->stats.Str) / 100);
981	}	1091	}
982	op->carrying-=weight;	1092	op->carrying -= weight;
983	op = op->env;	1093	op = op->env;
984	}	1094	}
985	}	1095	}
986		1096
987	/* remove_ob(op):	1097	/* remove_ob(op):
988	* This function removes the object op from the linked list of objects	1098	* This function removes the object op from the linked list of objects
989	* which it is currently tied to. When this function is done, the	1099	* which it is currently tied to. When this function is done, the
…		…
991	* environment, the x and y coordinates will be updated to	1101	* environment, the x and y coordinates will be updated to
992	* the previous environment.	1102	* the previous environment.
993	* Beware: This function is called from the editor as well!	1103	* Beware: This function is called from the editor as well!
994	*/	1104	*/
995		1105
		1106	void
996	void remove_ob(object *op) {	1107	remove_ob (object *op)
997	object *tmp,*last=NULL;	1108	{
998	object *otmp;	1109	object *
999	tag_t tag;	1110	tmp, *
		1111	last = NULL;
		1112	object *
		1113	otmp;
		1114	tag_t
		1115	tag;
		1116	int
1000	int check_walk_off;	1117	check_walk_off;
1001	mapstruct *m;	1118	mapstruct *
1002	sint16 x,y;
1003		1119	m;
		1120	sint16
		1121	x,
		1122	y;
1004		1123
		1124
1005	if(QUERY_FLAG(op,FLAG_REMOVED)) {	1125	if (QUERY_FLAG (op, FLAG_REMOVED))
		1126	{
1006	dump_object(op);	1127	dump_object (op);
1007	LOG(llevError,"Trying to remove removed object.\n%s\n",errmsg);	1128	LOG (llevError, "Trying to remove removed object.\n%s\n", errmsg);
1008		1129
1009	/* Changed it to always dump core in this case. As has been learned	1130	/* Changed it to always dump core in this case. As has been learned
1010	* in the past, trying to recover from errors almost always	1131	* in the past, trying to recover from errors almost always
1011	* make things worse, and this is a real error here - something	1132	* make things worse, and this is a real error here - something
1012	* that should not happen.	1133	* that should not happen.
1013	* Yes, if this was a mission critical app, trying to do something	1134	* Yes, if this was a mission critical app, trying to do something
1014	* to recover may make sense, but that is because failure of the app	1135	* to recover may make sense, but that is because failure of the app
1015	* may have other disastrous problems. Cf runs out of a script	1136	* may have other disastrous problems. Cf runs out of a script
1016	* so is easily enough restarted without any real problems.	1137	* so is easily enough restarted without any real problems.
1017	* MSW 2001-07-01	1138	* MSW 2001-07-01
1018	*/	1139	*/
1019	abort();	1140	abort ();
1020	}	1141	}
1021	if(op->more!=NULL)	1142	if (op->more != NULL)
1022	remove_ob(op->more);	1143	remove_ob (op->more);
1023		1144
1024	SET_FLAG(op, FLAG_REMOVED);	1145	SET_FLAG (op, FLAG_REMOVED);
1025		1146
1026	/*	1147	/*
1027	* In this case, the object to be removed is in someones	1148	* In this case, the object to be removed is in someones
1028	* inventory.	1149	* inventory.
1029	*/	1150	*/
1030	if(op->env!=NULL) {	1151	if (op->env != NULL)
		1152	{
1031	if(op->nrof)	1153	if (op->nrof)
1032	sub_weight(op->env, op->weight*op->nrof);	1154	sub_weight (op->env, op->weight * op->nrof);
1033	else	1155	else
1034	sub_weight(op->env, op->weight+op->carrying);	1156	sub_weight (op->env, op->weight + op->carrying);
1035		1157
1036	/* NO_FIX_PLAYER is set when a great many changes are being	1158	/* NO_FIX_PLAYER is set when a great many changes are being
1037	* made to players inventory. If set, avoiding the call	1159	* made to players inventory. If set, avoiding the call
1038	* to save cpu time.	1160	* to save cpu time.
1039	*/	1161	*/
1040	if ((otmp=is_player_inv(op->env))!=NULL && otmp->contr &&	1162	if ((otmp = is_player_inv (op->env)) != NULL && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
1041	!QUERY_FLAG(otmp,FLAG_NO_FIX_PLAYER))
1042	fix_player(otmp);	1163	fix_player (otmp);
1043		1164
1044	if(op->above!=NULL)	1165	if (op->above != NULL)
1045	op->above->below=op->below;	1166	op->above->below = op->below;
1046	else	1167	else
1047	op->env->inv=op->below;	1168	op->env->inv = op->below;
1048		1169
1049	if(op->below!=NULL)	1170	if (op->below != NULL)
1050	op->below->above=op->above;	1171	op->below->above = op->above;
1051		1172
1052	/* we set up values so that it could be inserted into	1173	/* we set up values so that it could be inserted into
1053	* the map, but we don't actually do that - it is up	1174	* the map, but we don't actually do that - it is up
1054	* to the caller to decide what we want to do.	1175	* to the caller to decide what we want to do.
1055	*/	1176	*/
1056	op->x=op->env->x,op->y=op->env->y;	1177	op->x = op->env->x, op->y = op->env->y;
1057	op->map=op->env->map;	1178	op->map = op->env->map;
1058	op->above=NULL,op->below=NULL;	1179	op->above = NULL, op->below = NULL;
1059	op->env=NULL;	1180	op->env = NULL;
1060	return;	1181	return;
1061	}	1182	}
1062		1183
1063	/* If we get here, we are removing it from a map */	1184	/* If we get here, we are removing it from a map */
1064	if (op->map == NULL) return;	1185	if (op->map == NULL)
		1186	return;
1065		1187
1066	x = op->x;	1188	x = op->x;
1067	y = op->y;	1189	y = op->y;
1068	m = get_map_from_coord(op->map, &x, &y);	1190	m = get_map_from_coord (op->map, &x, &y);
1069		1191
1070	if (!m) {	1192	if (!m)
		1193	{
1071	LOG(llevError,"remove_ob called when object was on map but appears to not be within valid coordinates? %s (%d,%d)\n",	1194	LOG (llevError, "remove_ob called when object was on map but appears to not be within valid coordinates? %s (%d,%d)\n",
1072	op->map->path, op->x, op->y);	1195	op->map->path, op->x, op->y);
1073	/* in old days, we used to set x and y to 0 and continue.	1196	/* in old days, we used to set x and y to 0 and continue.
1074	* it seems if we get into this case, something is probablye	1197	* it seems if we get into this case, something is probablye
1075	* screwed up and should be fixed.	1198	* screwed up and should be fixed.
1076	*/	1199	*/
1077	abort();	1200	abort ();
1078	}	1201	}
1079	if (op->map != m) {	1202	if (op->map != m)
		1203	{
1080	LOG(llevDebug,"remove_ob: Object not really on map it claimed to be on? %s != %s, %d,%d != %d,%d\n",	1204	LOG (llevDebug, "remove_ob: Object not really on map it claimed to be on? %s != %s, %d,%d != %d,%d\n",
1081	op->map->path, m->path, op->x, op->y, x, y);	1205	op->map->path, m->path, op->x, op->y, x, y);
1082	}	1206	}
1083		1207
1084	/* Re did the following section of code - it looks like it had	1208	/* Re did the following section of code - it looks like it had
1085	* lots of logic for things we no longer care about	1209	* lots of logic for things we no longer care about
1086	*/	1210	*/
1087		1211
1088	/* link the object above us */	1212	/* link the object above us */
1089	if (op->above)	1213	if (op->above)
1090	op->above->below=op->below;	1214	op->above->below = op->below;
1091	else	1215	else
1092	SET_MAP_TOP(m,x,y,op->below); /* we were top, set new top */	1216	SET_MAP_TOP (m, x, y, op->below); /* we were top, set new top */
1093		1217
1094	/* Relink the object below us, if there is one */	1218	/* Relink the object below us, if there is one */
1095	if(op->below) {	1219	if (op->below)
		1220	{
1096	op->below->above=op->above;	1221	op->below->above = op->above;
1097	} else {	1222	}
		1223	else
		1224	{
1098	/* Nothing below, which means we need to relink map object for this space	1225	/* Nothing below, which means we need to relink map object for this space
1099	* use translated coordinates in case some oddness with map tiling is	1226	* use translated coordinates in case some oddness with map tiling is
1100	* evident	1227	* evident
1101	*/	1228	*/
1102	if(GET_MAP_OB(m,x,y)!=op) {	1229	if (GET_MAP_OB (m, x, y) != op)
		1230	{
1103	dump_object(op);	1231	dump_object (op);
1104	LOG(llevError,"remove_ob: GET_MAP_OB does not return object to be removed even though it appears to be on the bottom?\n%s\n", errmsg);	1232	LOG (llevError, "remove_ob: GET_MAP_OB does not return object to be removed even though it appears to be on the bottom?\n%s\n",
		1233	errmsg);
1105	dump_object(GET_MAP_OB(m,x,y));	1234	dump_object (GET_MAP_OB (m, x, y));
1106	LOG(llevError,"%s\n",errmsg);	1235	LOG (llevError, "%s\n", errmsg);
1107	}	1236	}
1108	SET_MAP_OB(m,x,y,op->above); /* goes on above it. */	1237	SET_MAP_OB (m, x, y, op->above); /* goes on above it. */
1109	}	1238	}
1110	op->above=NULL;	1239	op->above = NULL;
1111	op->below=NULL;	1240	op->below = NULL;
1112		1241
1113	if (op->map->in_memory == MAP_SAVING)	1242	if (op->map->in_memory == MAP_SAVING)
1114	return;	1243	return;
1115		1244
1116	tag = op->count;	1245	tag = op->count;
1117	check_walk_off = ! QUERY_FLAG (op, FLAG_NO_APPLY);	1246	check_walk_off = !QUERY_FLAG (op, FLAG_NO_APPLY);
1118	for(tmp=GET_MAP_OB(m,x,y);tmp!=NULL;tmp=tmp->above) {	1247	for (tmp = GET_MAP_OB (m, x, y); tmp != NULL; tmp = tmp->above)
		1248	{
1119	/* No point updating the players look faces if he is the object	1249	/* No point updating the players look faces if he is the object
1120	* being removed.	1250	* being removed.
1121	*/	1251	*/
1122		1252
1123	if(tmp->type==PLAYER && tmp!=op) {	1253	if (tmp->type == PLAYER && tmp != op)
		1254	{
1124	/* If a container that the player is currently using somehow gets	1255	/* If a container that the player is currently using somehow gets
1125	* removed (most likely destroyed), update the player view	1256	* removed (most likely destroyed), update the player view
1126	* appropriately.	1257	* appropriately.
1127	*/	1258	*/
1128	if (tmp->container==op) {	1259	if (tmp->container == op)
		1260	{
1129	CLEAR_FLAG(op, FLAG_APPLIED);	1261	CLEAR_FLAG (op, FLAG_APPLIED);
1130	tmp->container=NULL;	1262	tmp->container = NULL;
1131	}	1263	}
1132	tmp->contr->socket.update_look=1;	1264	tmp->contr->socket.update_look = 1;
1133	}	1265	}
1134	/* See if player moving off should effect something */	1266	/* See if player moving off should effect something */
1135	if (check_walk_off && ((op->move_type & tmp->move_off) &&	1267	if (check_walk_off && ((op->move_type & tmp->move_off) && (op->move_type & ~tmp->move_off & ~tmp->move_block) == 0))
1136	(op->move_type & ~tmp->move_off & ~tmp->move_block)==0)) {	1268	{
1137		1269
1138	move_apply(tmp, op, NULL);	1270	move_apply (tmp, op, NULL);
1139	if (was_destroyed (op, tag)) {	1271	if (was_destroyed (op, tag))
		1272	{
1140	LOG (llevError, "BUG: remove_ob(): name %s, archname %s destroyed "	1273	LOG (llevError, "BUG: remove_ob(): name %s, archname %s destroyed " "leaving object\n", &tmp->name, &tmp->arch->name);
1141	"leaving object\n", &tmp->name, &tmp->arch->name);
1142	}	1274	}
1143	}	1275	}
1144		1276
1145	/* Eneq(@csd.uu.se): Fixed this to skip tmp->above=tmp */	1277	/* Eneq(@csd.uu.se): Fixed this to skip tmp->above=tmp */
1146		1278
1147	if(tmp->above == tmp)	1279	if (tmp->above == tmp)
1148	tmp->above = NULL;	1280	tmp->above = NULL;
1149	last=tmp;	1281	last = tmp;
1150	}	1282	}
1151	/* last == NULL of there are no objects on this space */	1283	/* last == NULL of there are no objects on this space */
1152	if (last==NULL) {	1284	if (last == NULL)
		1285	{
1153	/* set P_NEED_UPDATE, otherwise update_position will complain. In theory,	1286	/* set P_NEED_UPDATE, otherwise update_position will complain. In theory,
1154	* we could preserve the flags (GET_MAP_FLAGS), but update_position figures	1287	* we could preserve the flags (GET_MAP_FLAGS), but update_position figures
1155	* those out anyways, and if there are any flags set right now, they won't	1288	* those out anyways, and if there are any flags set right now, they won't
1156	* be correct anyways.	1289	* be correct anyways.
1157	*/	1290	*/
1158	SET_MAP_FLAGS(op->map, op->x, op->y, P_NEED_UPDATE);	1291	SET_MAP_FLAGS (op->map, op->x, op->y, P_NEED_UPDATE);
1159	update_position(op->map, op->x, op->y);	1292	update_position (op->map, op->x, op->y);
1160	}	1293	}
1161	else	1294	else
1162	update_object(last, UP_OBJ_REMOVE);	1295	update_object (last, UP_OBJ_REMOVE);
1163		1296
1164	if(QUERY_FLAG(op,FLAG_BLOCKSVIEW)|| (op->glow_radius != 0))	1297	if (QUERY_FLAG (op, FLAG_BLOCKSVIEW) || (op->glow_radius != 0))
1165	update_all_los(op->map, op->x, op->y);	1298	update_all_los (op->map, op->x, op->y);
1166		1299
1167	}	1300	}
1168		1301
1169	/*	1302	/*
1170	* merge_ob(op,top):	1303	* merge_ob(op,top):
…		…
1173	* merges op to the first matching object.	1306	* merges op to the first matching object.
1174	* If top is NULL, it is calculated.	1307	* If top is NULL, it is calculated.
1175	* Returns pointer to object if it succeded in the merge, otherwise NULL	1308	* Returns pointer to object if it succeded in the merge, otherwise NULL
1176	*/	1309	*/
1177		1310
		1311	object *
1178	object *merge_ob(object *op, object *top) {	1312	merge_ob (object *op, object *top)
		1313	{
1179	if(!op->nrof)	1314	if (!op->nrof)
1180	return 0;	1315	return 0;
1181	if(top==NULL)	1316	if (top == NULL)
1182	for(top=op;top!=NULL&&top->above!=NULL;top=top->above);	1317	for (top = op; top != NULL && top->above != NULL; top = top->above);
1183	for(;top!=NULL;top=top->below) {	1318	for (; top != NULL; top = top->below)
		1319	{
1184	if(top==op)	1320	if (top == op)
1185	continue;	1321	continue;
1186	if (CAN_MERGE(op,top))	1322	if (CAN_MERGE (op, top))
1187	{	1323	{
1188	top->nrof+=op->nrof;	1324	top->nrof += op->nrof;
		1325
1189	/* CLEAR_FLAG(top,FLAG_STARTEQUIP);*/	1326	/* CLEAR_FLAG(top,FLAG_STARTEQUIP);*/
1190	op->weight = 0; /* Don't want any adjustements now */	1327	op->weight = 0; /* Don't want any adjustements now */
1191	remove_ob(op);	1328	remove_ob (op);
1192	free_object(op);	1329	free_object (op);
1193	return top;	1330	return top;
1194	}	1331	}
1195	}	1332	}
1196	return NULL;	1333	return NULL;
1197	}	1334	}
1198		1335
1199	/*	1336	/*
1200	* same as insert_ob_in_map except it handle separate coordinates and do a clean	1337	* same as insert_ob_in_map except it handle separate coordinates and do a clean
1201	* job preparing multi-part monsters	1338	* job preparing multi-part monsters
1202	*/	1339	*/
		1340	object *
1203	object *insert_ob_in_map_at(object *op, mapstruct *m, object *originator, int flag, int x, int y){	1341	insert_ob_in_map_at (object *op, mapstruct *m, object *originator, int flag, int x, int y)
1204	object* tmp;	1342	{
		1343	object *
		1344	tmp;
		1345
1205	if (op->head)	1346	if (op->head)
1206	op=op->head;	1347	op = op->head;
1207	for (tmp=op;tmp;tmp=tmp->more){	1348	for (tmp = op; tmp; tmp = tmp->more)
		1349	{
1208	tmp->x=x+tmp->arch->clone.x;	1350	tmp->x = x + tmp->arch->clone.x;
1209	tmp->y=y+tmp->arch->clone.y;	1351	tmp->y = y + tmp->arch->clone.y;
1210	}	1352	}
1211	return insert_ob_in_map (op, m, originator, flag);	1353	return insert_ob_in_map (op, m, originator, flag);
1212	}	1354	}
1213		1355
1214	/*	1356	/*
1215	* insert_ob_in_map (op, map, originator, flag):	1357	* insert_ob_in_map (op, map, originator, flag):
1216	* This function inserts the object in the two-way linked list	1358	* This function inserts the object in the two-way linked list
…		…
1230	* new object if 'op' was merged with other object	1372	* new object if 'op' was merged with other object
1231	* NULL if 'op' was destroyed	1373	* NULL if 'op' was destroyed
1232	* just 'op' otherwise	1374	* just 'op' otherwise
1233	*/	1375	*/
1234		1376
		1377	object *
1235	object *insert_ob_in_map (object *op, mapstruct *m, object *originator, int flag)	1378	insert_ob_in_map (object *op, mapstruct *m, object *originator, int flag)
1236	{	1379	{
1237	object *tmp, *top, *floor=NULL;	1380	object *
1238	sint16 x,y;	1381	tmp, *
		1382	top, *
		1383	floor = NULL;
		1384	sint16
		1385	x,
		1386	y;
1239		1387
1240	if (QUERY_FLAG (op, FLAG_FREED)) {	1388	if (QUERY_FLAG (op, FLAG_FREED))
		1389	{
1241	LOG (llevError, "Trying to insert freed object!\n");	1390	LOG (llevError, "Trying to insert freed object!\n");
		1391	return NULL;
		1392	}
		1393	if (m == NULL)
		1394	{
		1395	dump_object (op);
		1396	LOG (llevError, "Trying to insert in null-map!\n%s\n", errmsg);
		1397	return op;
		1398	}
		1399	if (out_of_map (m, op->x, op->y))
		1400	{
		1401	dump_object (op);
		1402	LOG (llevError, "Trying to insert object outside the map.\n%s\n", errmsg);
		1403	#ifdef MANY_CORES
		1404	/* Better to catch this here, as otherwise the next use of this object
		1405	* is likely to cause a crash. Better to find out where it is getting
		1406	* improperly inserted.
		1407	*/
		1408	abort ();
		1409	#endif
		1410	return op;
		1411	}
		1412	if (!QUERY_FLAG (op, FLAG_REMOVED))
		1413	{
		1414	dump_object (op);
		1415	LOG (llevError, "Trying to insert (map) inserted object.\n%s\n", errmsg);
		1416	return op;
		1417	}
		1418	if (op->more != NULL)
		1419	{
		1420	/* The part may be on a different map. */
		1421
		1422	object *
		1423	more = op->more;
		1424
		1425	/* We really need the caller to normalize coordinates - if
		1426	* we set the map, that doesn't work if the location is within
		1427	* a map and this is straddling an edge. So only if coordinate
		1428	* is clear wrong do we normalize it.
		1429	*/
		1430	if (OUT_OF_REAL_MAP (more->map, more->x, more->y))
		1431	{
		1432	more->map = get_map_from_coord (m, &more->x, &more->y);
		1433	}
		1434	else if (!more->map)
		1435	{
		1436	/* For backwards compatibility - when not dealing with tiled maps,
		1437	* more->map should always point to the parent.
		1438	*/
		1439	more->map = m;
		1440	}
		1441
		1442	if (insert_ob_in_map (more, more->map, originator, flag) == NULL)
		1443	{
		1444	if (!op->head)
		1445	LOG (llevError, "BUG: insert_ob_in_map(): inserting op->more killed op\n");
		1446	return NULL;
		1447	}
		1448	}
		1449	CLEAR_FLAG (op, FLAG_REMOVED);
		1450
		1451	/* Ideally, the caller figures this out. However, it complicates a lot
		1452	* of areas of callers (eg, anything that uses find_free_spot would now
		1453	* need extra work
		1454	*/
		1455	op->map = get_map_from_coord (m, &op->x, &op->y);
		1456	x = op->x;
		1457	y = op->y;
		1458
		1459	/* this has to be done after we translate the coordinates.
		1460	*/
		1461	if (op->nrof && !(flag & INS_NO_MERGE))
		1462	{
		1463	for (tmp = GET_MAP_OB (op->map, x, y); tmp != NULL; tmp = tmp->above)
		1464	if (CAN_MERGE (op, tmp))
		1465	{
		1466	op->nrof += tmp->nrof;
		1467	remove_ob (tmp);
		1468	free_object (tmp);
		1469	}
		1470	}
		1471
		1472	CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */
		1473	CLEAR_FLAG (op, FLAG_INV_LOCKED);
		1474	if (!QUERY_FLAG (op, FLAG_ALIVE))
		1475	CLEAR_FLAG (op, FLAG_NO_STEAL);
		1476
		1477	if (flag & INS_BELOW_ORIGINATOR)
		1478	{
		1479	if (originator->map != op->map || originator->x != op->x || originator->y != op->y)
		1480	{
		1481	LOG (llevError, "insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n");
		1482	abort ();
		1483	}
		1484	op->above = originator;
		1485	op->below = originator->below;
		1486	if (op->below)
		1487	op->below->above = op;
		1488	else
		1489	SET_MAP_OB (op->map, op->x, op->y, op);
		1490	/* since *below* originator, no need to update top */
		1491	originator->below = op;
		1492	}
		1493	else
		1494	{
		1495	/* If there are other objects, then */
		1496	if ((!(flag & INS_MAP_LOAD)) && ((top = GET_MAP_OB (op->map, op->x, op->y)) != NULL))
		1497	{
		1498	object *
		1499	last = NULL;
		1500
		1501	/*
		1502	* If there are multiple objects on this space, we do some trickier handling.
		1503	* We've already dealt with merging if appropriate.
		1504	* Generally, we want to put the new object on top. But if
		1505	* flag contains INS_ABOVE_FLOOR_ONLY, once we find the last
		1506	* floor, we want to insert above that and no further.
		1507	* Also, if there are spell objects on this space, we stop processing
		1508	* once we get to them. This reduces the need to traverse over all of
		1509	* them when adding another one - this saves quite a bit of cpu time
		1510	* when lots of spells are cast in one area. Currently, it is presumed
		1511	* that flying non pickable objects are spell objects.
		1512	*/
		1513
		1514	while (top != NULL)
		1515	{
		1516	if (QUERY_FLAG (top, FLAG_IS_FLOOR) || QUERY_FLAG (top, FLAG_OVERLAY_FLOOR))
		1517	floor = top;
		1518	if (QUERY_FLAG (top, FLAG_NO_PICK) && (top->move_type & (MOVE_FLY_LOW | MOVE_FLY_HIGH)) && !QUERY_FLAG (top, FLAG_IS_FLOOR))
		1519	{
		1520	/* We insert above top, so we want this object below this */
		1521	top = top->below;
		1522	break;
		1523	}
		1524	last = top;
		1525	top = top->above;
		1526	}
		1527	/* Don't want top to be NULL, so set it to the last valid object */
		1528	top = last;
		1529
		1530	/* We let update_position deal with figuring out what the space
		1531	* looks like instead of lots of conditions here.
		1532	* makes things faster, and effectively the same result.
		1533	*/
		1534
		1535	/* Have object 'fall below' other objects that block view.
		1536	* Unless those objects are exits, type 66
		1537	* If INS_ON_TOP is used, don't do this processing
		1538	* Need to find the object that in fact blocks view, otherwise
		1539	* stacking is a bit odd.
		1540	*/
		1541	if (!(flag & INS_ON_TOP) &&
		1542	(get_map_flags (op->map, NULL, op->x, op->y, NULL, NULL) & P_BLOCKSVIEW) && (op->face && !op->face->visibility))
		1543	{
		1544	for (last = top; last != floor; last = last->below)
		1545	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))
		1546	break;
		1547	/* Check to see if we found the object that blocks view,
		1548	* and make sure we have a below pointer for it so that
		1549	* we can get inserted below this one, which requires we
		1550	* set top to the object below us.
		1551	*/
		1552	if (last && last->below && last != floor)
		1553	top = last->below;
		1554	}
		1555	} /* If objects on this space */
		1556	if (flag & INS_MAP_LOAD)
		1557	top = GET_MAP_TOP (op->map, op->x, op->y);
		1558	if (flag & INS_ABOVE_FLOOR_ONLY)
		1559	top = floor;
		1560
		1561	/* Top is the object that our object (op) is going to get inserted above.
		1562	*/
		1563
		1564	/* First object on this space */
		1565	if (!top)
		1566	{
		1567	op->above = GET_MAP_OB (op->map, op->x, op->y);
		1568	if (op->above)
		1569	op->above->below = op;
		1570	op->below = NULL;
		1571	SET_MAP_OB (op->map, op->x, op->y, op);
		1572	}
		1573	else
		1574	{ /* get inserted into the stack above top */
		1575	op->above = top->above;
		1576	if (op->above)
		1577	op->above->below = op;
		1578	op->below = top;
		1579	top->above = op;
		1580	}
		1581	if (op->above == NULL)
		1582	SET_MAP_TOP (op->map, op->x, op->y, op);
		1583	} /* else not INS_BELOW_ORIGINATOR */
		1584
		1585	if (op->type == PLAYER)
		1586	op->contr->do_los = 1;
		1587
		1588	/* If we have a floor, we know the player, if any, will be above
		1589	* it, so save a few ticks and start from there.
		1590	*/
		1591	if (!(flag & INS_MAP_LOAD))
		1592	for (tmp = floor ? floor : GET_MAP_OB (op->map, op->x, op->y); tmp != NULL; tmp = tmp->above)
		1593	{
		1594	if (tmp->type == PLAYER)
		1595	tmp->contr->socket.update_look = 1;
		1596	}
		1597
		1598	/* If this object glows, it may affect lighting conditions that are
		1599	* visible to others on this map. But update_all_los is really
		1600	* an inefficient way to do this, as it means los for all players
		1601	* on the map will get recalculated. The players could very well
		1602	* be far away from this change and not affected in any way -
		1603	* this should get redone to only look for players within range,
		1604	* or just updating the P_NEED_UPDATE for spaces within this area
		1605	* of effect may be sufficient.
		1606	*/
		1607	if (MAP_DARKNESS (op->map) && (op->glow_radius != 0))
		1608	update_all_los (op->map, op->x, op->y);
		1609
		1610
		1611	/* updates flags (blocked, alive, no magic, etc) for this map space */
		1612	update_object (op, UP_OBJ_INSERT);
		1613
		1614
		1615	/* Don't know if moving this to the end will break anything. However,
		1616	* we want to have update_look set above before calling this.
		1617	*
		1618	* check_move_on() must be after this because code called from
		1619	* check_move_on() depends on correct map flags (so functions like
		1620	* blocked() and wall() work properly), and these flags are updated by
		1621	* update_object().
		1622	*/
		1623
		1624	/* if this is not the head or flag has been passed, don't check walk on status */
		1625
		1626	if (!(flag & INS_NO_WALK_ON) && !op->head)
		1627	{
		1628	if (check_move_on (op, originator))
1242	return NULL;	1629	return NULL;
1243	}
1244	if(m==NULL) {
1245	dump_object(op);
1246	LOG(llevError,"Trying to insert in null-map!\n%s\n",errmsg);
1247	return op;
1248	}
1249	if(out_of_map(m,op->x,op->y)) {
1250	dump_object(op);
1251	LOG(llevError,"Trying to insert object outside the map.\n%s\n", errmsg);
1252	#ifdef MANY_CORES
1253	/* Better to catch this here, as otherwise the next use of this object
1254	* is likely to cause a crash. Better to find out where it is getting
1255	* improperly inserted.
1256	*/
1257	abort();
1258	#endif
1259	return op;
1260	}
1261	if(!QUERY_FLAG(op,FLAG_REMOVED)) {
1262	dump_object(op);
1263	LOG(llevError,"Trying to insert (map) inserted object.\n%s\n", errmsg);
1264	return op;
1265	}
1266	if(op->more!=NULL) {
1267	/* The part may be on a different map. */
1268		1630
1269	object *more = op->more;
1270
1271	/* We really need the caller to normalize coordinates - if
1272	* we set the map, that doesn't work if the location is within
1273	* a map and this is straddling an edge. So only if coordinate
1274	* is clear wrong do we normalize it.
1275	*/
1276	if (OUT_OF_REAL_MAP(more->map, more->x, more->y)) {
1277	more->map = get_map_from_coord(m, &more->x, &more->y);
1278	} else if (!more->map) {
1279	/* For backwards compatibility - when not dealing with tiled maps,
1280	* more->map should always point to the parent.
1281	*/
1282	more->map = m;
1283	}
1284
1285	if (insert_ob_in_map(more, more->map, originator, flag) == NULL) {
1286	if ( ! op->head)
1287	LOG (llevError, "BUG: insert_ob_in_map(): inserting op->more killed op\n");
1288	return NULL;
1289	}
1290	}
1291	CLEAR_FLAG(op,FLAG_REMOVED);
1292
1293	/* Ideally, the caller figures this out. However, it complicates a lot
1294	* of areas of callers (eg, anything that uses find_free_spot would now
1295	* need extra work
1296	*/
1297	op->map=get_map_from_coord(m, &op->x, &op->y);
1298	x = op->x;
1299	y = op->y;
1300
1301	/* this has to be done after we translate the coordinates.
1302	*/
1303	if(op->nrof && !(flag & INS_NO_MERGE)) {
1304	for(tmp=GET_MAP_OB(op->map,x,y);tmp!=NULL;tmp=tmp->above)
1305	if (CAN_MERGE(op,tmp)) {
1306	op->nrof+=tmp->nrof;
1307	remove_ob(tmp);
1308	free_object(tmp);
1309	}
1310	}
1311
1312	CLEAR_FLAG(op,FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */
1313	CLEAR_FLAG(op, FLAG_INV_LOCKED);
1314	if (!QUERY_FLAG(op, FLAG_ALIVE))
1315	CLEAR_FLAG(op, FLAG_NO_STEAL);
1316
1317	if (flag & INS_BELOW_ORIGINATOR) {
1318	if (originator->map != op->map || originator->x != op->x ||
1319	originator->y != op->y) {
1320	LOG(llevError,"insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n");
1321	abort();
1322	}
1323	op->above = originator;
1324	op->below = originator->below;
1325	if (op->below) op->below->above = op;
1326	else SET_MAP_OB(op->map, op->x, op->y, op);
1327	/* since *below* originator, no need to update top */
1328	originator->below = op;
1329	} else {
1330	/* If there are other objects, then */
1331	if((! (flag & INS_MAP_LOAD)) && ((top=GET_MAP_OB(op->map,op->x,op->y))!=NULL)) {
1332	object *last=NULL;
1333	/*
1334	* If there are multiple objects on this space, we do some trickier handling.
1335	* We've already dealt with merging if appropriate.
1336	* Generally, we want to put the new object on top. But if
1337	* flag contains INS_ABOVE_FLOOR_ONLY, once we find the last
1338	* floor, we want to insert above that and no further.
1339	* Also, if there are spell objects on this space, we stop processing
1340	* once we get to them. This reduces the need to traverse over all of
1341	* them when adding another one - this saves quite a bit of cpu time
1342	* when lots of spells are cast in one area. Currently, it is presumed
1343	* that flying non pickable objects are spell objects.
1344	*/
1345
1346	while (top != NULL) {
1347	if (QUERY_FLAG(top, FLAG_IS_FLOOR) ||
1348	QUERY_FLAG(top, FLAG_OVERLAY_FLOOR)) floor = top;
1349	if (QUERY_FLAG(top, FLAG_NO_PICK)
1350	&& (top->move_type & (MOVE_FLY_LOW |MOVE_FLY_HIGH))
1351	&& !QUERY_FLAG(top, FLAG_IS_FLOOR))
1352	{
1353	/* We insert above top, so we want this object below this */
1354	top=top->below;
1355	break;
1356	}
1357	last = top;
1358	top = top->above;
1359	}
1360	/* Don't want top to be NULL, so set it to the last valid object */
1361	top = last;
1362
1363	/* We let update_position deal with figuring out what the space
1364	* looks like instead of lots of conditions here.
1365	* makes things faster, and effectively the same result.
1366	*/
1367
1368	/* Have object 'fall below' other objects that block view.
1369	* Unless those objects are exits, type 66
1370	* If INS_ON_TOP is used, don't do this processing
1371	* Need to find the object that in fact blocks view, otherwise
1372	* stacking is a bit odd.
1373	*/
1374	if (!(flag & INS_ON_TOP) &&
1375	(get_map_flags(op->map, NULL, op->x, op->y, NULL, NULL) & P_BLOCKSVIEW) &&
1376	(op->face && !op->face->visibility)) {
1377	for (last=top; last != floor; last=last->below)
1378	if (QUERY_FLAG(last, FLAG_BLOCKSVIEW)&&(last->type != EXIT)) break;
1379	/* Check to see if we found the object that blocks view,
1380	* and make sure we have a below pointer for it so that
1381	* we can get inserted below this one, which requires we
1382	* set top to the object below us.
1383	*/
1384	if (last && last->below && last != floor) top=last->below;
1385	}
1386	} /* If objects on this space */
1387	if (flag & INS_MAP_LOAD)
1388	top = GET_MAP_TOP(op->map,op->x,op->y);
1389	if (flag & INS_ABOVE_FLOOR_ONLY) top = floor;
1390
1391	/* Top is the object that our object (op) is going to get inserted above.
1392	*/
1393
1394	/* First object on this space */
1395	if (!top) {
1396	op->above = GET_MAP_OB(op->map, op->x, op->y);
1397	if (op->above) op->above->below = op;
1398	op->below = NULL;
1399	SET_MAP_OB(op->map, op->x, op->y, op);
1400	} else { /* get inserted into the stack above top */
1401	op->above = top->above;
1402	if (op->above) op->above->below = op;
1403	op->below = top;
1404	top->above = op;
1405	}
1406	if (op->above==NULL)
1407	SET_MAP_TOP(op->map,op->x, op->y, op);
1408	} /* else not INS_BELOW_ORIGINATOR */
1409
1410	if(op->type==PLAYER)
1411	op->contr->do_los=1;
1412
1413	/* If we have a floor, we know the player, if any, will be above
1414	* it, so save a few ticks and start from there.
1415	*/
1416	if (!(flag & INS_MAP_LOAD))
1417	for(tmp=floor?floor:GET_MAP_OB(op->map,op->x,op->y);tmp!=NULL;tmp=tmp->above) {
1418	if (tmp->type == PLAYER)
1419	tmp->contr->socket.update_look=1;
1420	}
1421
1422	/* If this object glows, it may affect lighting conditions that are
1423	* visible to others on this map. But update_all_los is really
1424	* an inefficient way to do this, as it means los for all players
1425	* on the map will get recalculated. The players could very well
1426	* be far away from this change and not affected in any way -
1427	* this should get redone to only look for players within range,
1428	* or just updating the P_NEED_UPDATE for spaces within this area
1429	* of effect may be sufficient.
1430	*/
1431	if(MAP_DARKNESS(op->map) && (op->glow_radius != 0))
1432	update_all_los(op->map, op->x, op->y);
1433
1434
1435	/* updates flags (blocked, alive, no magic, etc) for this map space */
1436	update_object(op,UP_OBJ_INSERT);
1437
1438
1439	/* Don't know if moving this to the end will break anything. However,
1440	* we want to have update_look set above before calling this.
1441	*
1442	* check_move_on() must be after this because code called from
1443	* check_move_on() depends on correct map flags (so functions like
1444	* blocked() and wall() work properly), and these flags are updated by
1445	* update_object().
1446	*/
1447
1448	/* if this is not the head or flag has been passed, don't check walk on status */
1449
1450	if (!(flag & INS_NO_WALK_ON) && !op->head) {
1451	if (check_move_on(op, originator))
1452	return NULL;
1453
1454	/* If we are a multi part object, lets work our way through the check	1631	/* If we are a multi part object, lets work our way through the check
1455	* walk on's.	1632	* walk on's.
1456	*/	1633	*/
1457	for (tmp=op->more; tmp!=NULL; tmp=tmp->more)	1634	for (tmp = op->more; tmp != NULL; tmp = tmp->more)
1458	if (check_move_on (tmp, originator))	1635	if (check_move_on (tmp, originator))
1459	return NULL;	1636	return NULL;
1460	}	1637	}
1461	return op;	1638	return op;
1462	}	1639	}
1463		1640
1464	/* this function inserts an object in the map, but if it	1641	/* this function inserts an object in the map, but if it
1465	* finds an object of its own type, it'll remove that one first.	1642	* finds an object of its own type, it'll remove that one first.
1466	* op is the object to insert it under: supplies x and the map.	1643	* op is the object to insert it under: supplies x and the map.
1467	*/	1644	*/
		1645	void
1468	void replace_insert_ob_in_map(const char *arch_string, object *op) {	1646	replace_insert_ob_in_map (const char *arch_string, object *op)
		1647	{
1469	object *tmp;	1648	object *
1470	object *tmp1;	1649	tmp;
		1650	object *
		1651	tmp1;
1471		1652
1472	/* first search for itself and remove any old instances */	1653	/* first search for itself and remove any old instances */
1473		1654
1474	for(tmp=GET_MAP_OB(op->map,op->x,op->y); tmp!=NULL; tmp=tmp->above) {	1655	for (tmp = GET_MAP_OB (op->map, op->x, op->y); tmp != NULL; tmp = tmp->above)
		1656	{
1475	if(!strcmp(tmp->arch->name,arch_string)) /* same archetype */ {	1657	if (!strcmp (tmp->arch->name, arch_string)) /* same archetype */
		1658	{
1476	remove_ob(tmp);	1659	remove_ob (tmp);
1477	free_object(tmp);	1660	free_object (tmp);
1478	}	1661	}
1479	}	1662	}
1480		1663
1481	tmp1=arch_to_object(find_archetype(arch_string));	1664	tmp1 = arch_to_object (find_archetype (arch_string));
1482		1665
1483		1666
1484	tmp1->x = op->x; tmp1->y = op->y;	1667	tmp1->x = op->x;
		1668	tmp1->y = op->y;
1485	insert_ob_in_map(tmp1,op->map,op,0);	1669	insert_ob_in_map (tmp1, op->map, op, 0);
1486	}	1670	}
1487		1671
1488	/*	1672	/*
1489	* get_split_ob(ob,nr) splits up ob into two parts. The part which	1673	* get_split_ob(ob,nr) splits up ob into two parts. The part which
1490	* is returned contains nr objects, and the remaining parts contains	1674	* is returned contains nr objects, and the remaining parts contains
1491	* the rest (or is removed and freed if that number is 0).	1675	* the rest (or is removed and freed if that number is 0).
1492	* On failure, NULL is returned, and the reason put into the	1676	* On failure, NULL is returned, and the reason put into the
1493	* global static errmsg array.	1677	* global static errmsg array.
1494	*/	1678	*/
1495		1679
		1680	object *
1496	object *get_split_ob(object *orig_ob, uint32 nr) {	1681	get_split_ob (object *orig_ob, uint32 nr)
1497	object *newob;	1682	{
		1683	object *
		1684	newob;
		1685	int
1498	int is_removed = (QUERY_FLAG (orig_ob, FLAG_REMOVED) != 0);	1686	is_removed = (QUERY_FLAG (orig_ob, FLAG_REMOVED) != 0);
1499		1687
1500	if(orig_ob->nrof<nr) {	1688	if (orig_ob->nrof < nr)
1501	sprintf(errmsg,"There are only %d %ss.",	1689	{
1502	orig_ob->nrof?orig_ob->nrof:1, &orig_ob->name);	1690	sprintf (errmsg, "There are only %d %ss.", orig_ob->nrof ? orig_ob->nrof : 1, &orig_ob->name);
1503	return NULL;	1691	return NULL;
1504	}	1692	}
1505	newob = object_create_clone(orig_ob);	1693	newob = object_create_clone (orig_ob);
1506	if((orig_ob->nrof-=nr)<1) {	1694	if ((orig_ob->nrof -= nr) < 1)
		1695	{
1507	if ( ! is_removed)	1696	if (!is_removed)
1508	remove_ob(orig_ob);	1697	remove_ob (orig_ob);
1509	free_object2(orig_ob, 1);	1698	free_object2 (orig_ob, 1);
1510	}	1699	}
1511	else if ( ! is_removed) {	1700	else if (!is_removed)
		1701	{
1512	if(orig_ob->env!=NULL)	1702	if (orig_ob->env != NULL)
1513	sub_weight (orig_ob->env,orig_ob->weight*nr);	1703	sub_weight (orig_ob->env, orig_ob->weight * nr);
1514	if (orig_ob->env == NULL && orig_ob->map->in_memory!=MAP_IN_MEMORY) {	1704	if (orig_ob->env == NULL && orig_ob->map->in_memory != MAP_IN_MEMORY)
		1705	{
1515	strcpy(errmsg, "Tried to split object whose map is not in memory.");	1706	strcpy (errmsg, "Tried to split object whose map is not in memory.");
1516	LOG(llevDebug,
1517	"Error, Tried to split object whose map is not in memory.\n");	1707	LOG (llevDebug, "Error, Tried to split object whose map is not in memory.\n");
1518	return NULL;	1708	return NULL;
1519	}	1709	}
1520	}	1710	}
1521	newob->nrof=nr;	1711	newob->nrof = nr;
1522		1712
1523	return newob;	1713	return newob;
1524	}	1714	}
1525		1715
1526	/*	1716	/*
1527	* decrease_ob_nr(object, number) decreases a specified number from	1717	* decrease_ob_nr(object, number) decreases a specified number from
1528	* the amount of an object. If the amount reaches 0, the object	1718	* the amount of an object. If the amount reaches 0, the object
1529	* is subsequently removed and freed.	1719	* is subsequently removed and freed.
1530	*	1720	*
1531	* Return value: 'op' if something is left, NULL if the amount reached 0	1721	* Return value: 'op' if something is left, NULL if the amount reached 0
1532	*/	1722	*/
1533		1723
		1724	object *
1534	object *decrease_ob_nr (object *op, uint32 i)	1725	decrease_ob_nr (object *op, uint32 i)
1535	{	1726	{
1536	object *tmp;	1727	object *
		1728	tmp;
1537	player *pl;	1729	player *
		1730	pl;
1538		1731
1539	if (i == 0) /* objects with op->nrof require this check */	1732	if (i == 0) /* objects with op->nrof require this check */
1540	return op;	1733	return op;
1541		1734
1542	if (i > op->nrof)	1735	if (i > op->nrof)
1543	i = op->nrof;	1736	i = op->nrof;
1544		1737
1545	if (QUERY_FLAG (op, FLAG_REMOVED))	1738	if (QUERY_FLAG (op, FLAG_REMOVED))
1546	{	1739	{
1547	op->nrof -= i;	1740	op->nrof -= i;
1548	}	1741	}
1549	else if (op->env != NULL)	1742	else if (op->env != NULL)
1550	{	1743	{
1551	/* is this object in the players inventory, or sub container	1744	/* is this object in the players inventory, or sub container
1552	* therein?	1745	* therein?
1553	*/	1746	*/
1554	tmp = is_player_inv (op->env);	1747	tmp = is_player_inv (op->env);
1555	/* nope. Is this a container the player has opened?	1748	/* nope. Is this a container the player has opened?
1556	* If so, set tmp to that player.	1749	* If so, set tmp to that player.
1557	* IMO, searching through all the players will mostly	1750	* IMO, searching through all the players will mostly
1558	* likely be quicker than following op->env to the map,	1751	* likely be quicker than following op->env to the map,
1559	* and then searching the map for a player.	1752	* and then searching the map for a player.
1560	*/	1753	*/
1561	if (!tmp) {	1754	if (!tmp)
		1755	{
1562	for (pl=first_player; pl; pl=pl->next)	1756	for (pl = first_player; pl; pl = pl->next)
1563	if (pl->ob->container == op->env) break;	1757	if (pl->ob->container == op->env)
		1758	break;
		1759	if (pl)
1564	if (pl) tmp=pl->ob;	1760	tmp = pl->ob;
		1761	else
1565	else tmp=NULL;	1762	tmp = NULL;
1566	}	1763	}
1567		1764
1568	if (i < op->nrof) {	1765	if (i < op->nrof)
		1766	{
1569	sub_weight (op->env, op->weight * i);	1767	sub_weight (op->env, op->weight * i);
1570	op->nrof -= i;	1768	op->nrof -= i;
1571	if (tmp) {	1769	if (tmp)
		1770	{
1572	esrv_send_item(tmp, op);	1771	esrv_send_item (tmp, op);
1573	}	1772	}
		1773	}
1574	} else {	1774	else
		1775	{
1575	remove_ob (op);	1776	remove_ob (op);
1576	op->nrof = 0;	1777	op->nrof = 0;
1577	if (tmp) {	1778	if (tmp)
		1779	{
1578	esrv_del_item(tmp->contr, op->count);	1780	esrv_del_item (tmp->contr, op->count);
1579	}	1781	}
1580	}	1782	}
1581	}	1783	}
1582	else	1784	else
1583	{	1785	{
		1786	object *
1584	object *above = op->above;	1787	above = op->above;
1585		1788
1586	if (i < op->nrof) {	1789	if (i < op->nrof)
		1790	{
1587	op->nrof -= i;	1791	op->nrof -= i;
		1792	}
1588	} else {	1793	else
		1794	{
1589	remove_ob (op);	1795	remove_ob (op);
1590	op->nrof = 0;	1796	op->nrof = 0;
1591	}	1797	}
1592	/* Since we just removed op, op->above is null */	1798	/* Since we just removed op, op->above is null */
1593	for (tmp = above; tmp != NULL; tmp = tmp->above)	1799	for (tmp = above; tmp != NULL; tmp = tmp->above)
1594	if (tmp->type == PLAYER) {	1800	if (tmp->type == PLAYER)
		1801	{
1595	if (op->nrof)	1802	if (op->nrof)
1596	esrv_send_item(tmp, op);	1803	esrv_send_item (tmp, op);
1597	else	1804	else
1598	esrv_del_item(tmp->contr, op->count);	1805	esrv_del_item (tmp->contr, op->count);
1599	}	1806	}
1600	}	1807	}
1601		1808
1602	if (op->nrof) {	1809	if (op->nrof)
		1810	{
1603	return op;	1811	return op;
1604	} else {	1812	}
		1813	else
		1814	{
1605	free_object (op);	1815	free_object (op);
1606	return NULL;	1816	return NULL;
1607	}	1817	}
1608	}	1818	}
1609		1819
1610	/*	1820	/*
1611	* add_weight(object, weight) adds the specified weight to an object,	1821	* add_weight(object, weight) adds the specified weight to an object,
1612	* and also updates how much the environment(s) is/are carrying.	1822	* and also updates how much the environment(s) is/are carrying.
1613	*/	1823	*/
1614		1824
		1825	void
1615	void add_weight (object *op, signed long weight) {	1826	add_weight (object *op, signed long weight)
		1827	{
1616	while (op!=NULL) {	1828	while (op != NULL)
		1829	{
1617	if (op->type == CONTAINER) {	1830	if (op->type == CONTAINER)
		1831	{
1618	weight=(signed long)(weight*(100-op->stats.Str)/100);	1832	weight = (signed long) (weight * (100 - op->stats.Str) / 100);
1619	}	1833	}
1620	op->carrying+=weight;	1834	op->carrying += weight;
1621	op=op->env;	1835	op = op->env;
1622	}	1836	}
1623	}	1837	}
1624		1838
1625	/*	1839	/*
1626	* insert_ob_in_ob(op,environment):	1840	* insert_ob_in_ob(op,environment):
1627	* This function inserts the object op in the linked list	1841	* This function inserts the object op in the linked list
…		…
1634	*	1848	*
1635	* The function returns now pointer to inserted item, and return value can	1849	* The function returns now pointer to inserted item, and return value can
1636	* be != op, if items are merged. -Tero	1850	* be != op, if items are merged. -Tero
1637	*/	1851	*/
1638		1852
		1853	object *
1639	object *insert_ob_in_ob(object *op,object *where) {	1854	insert_ob_in_ob (object *op, object *where)
1640	object *tmp, *otmp;	1855	{
		1856	object *
		1857	tmp, *
		1858	otmp;
1641		1859
1642	if(!QUERY_FLAG(op,FLAG_REMOVED)) {	1860	if (!QUERY_FLAG (op, FLAG_REMOVED))
		1861	{
1643	dump_object(op);	1862	dump_object (op);
1644	LOG(llevError,"Trying to insert (ob) inserted object.\n%s\n", errmsg);	1863	LOG (llevError, "Trying to insert (ob) inserted object.\n%s\n", errmsg);
1645	return op;	1864	return op;
1646	}	1865	}
1647	if(where==NULL) {	1866	if (where == NULL)
		1867	{
1648	dump_object(op);	1868	dump_object (op);
1649	LOG(llevError,"Trying to put object in NULL.\n%s\n", errmsg);	1869	LOG (llevError, "Trying to put object in NULL.\n%s\n", errmsg);
1650	return op;	1870	return op;
1651	}	1871	}
1652	if (where->head) {	1872	if (where->head)
1653	LOG(llevDebug,	1873	{
1654	"Warning: Tried to insert object wrong part of multipart object.\n");	1874	LOG (llevDebug, "Warning: Tried to insert object wrong part of multipart object.\n");
1655	where = where->head;	1875	where = where->head;
1656	}	1876	}
1657	if (op->more) {	1877	if (op->more)
		1878	{
1658	LOG(llevError, "Tried to insert multipart object %s (%d)\n",	1879	LOG (llevError, "Tried to insert multipart object %s (%d)\n", &op->name, op->count);
1659	&op->name, op->count);
1660	return op;	1880	return op;
1661	}	1881	}
1662	CLEAR_FLAG(op, FLAG_OBJ_ORIGINAL);	1882	CLEAR_FLAG (op, FLAG_OBJ_ORIGINAL);
1663	CLEAR_FLAG(op, FLAG_REMOVED);	1883	CLEAR_FLAG (op, FLAG_REMOVED);
1664	if(op->nrof) {	1884	if (op->nrof)
		1885	{
1665	for(tmp=where->inv;tmp!=NULL;tmp=tmp->below)	1886	for (tmp = where->inv; tmp != NULL; tmp = tmp->below)
1666	if ( CAN_MERGE(tmp,op) ) {	1887	if (CAN_MERGE (tmp, op))
		1888	{
1667	/* return the original object and remove inserted object	1889	/* return the original object and remove inserted object
1668	(client needs the original object) */	1890	(client needs the original object) */
1669	tmp->nrof += op->nrof;	1891	tmp->nrof += op->nrof;
1670	/* Weight handling gets pretty funky. Since we are adding to	1892	/* Weight handling gets pretty funky. Since we are adding to
1671	* tmp->nrof, we need to increase the weight.	1893	* tmp->nrof, we need to increase the weight.
1672	*/	1894	*/
1673	add_weight (where, op->weight*op->nrof);	1895	add_weight (where, op->weight * op->nrof);
1674	SET_FLAG(op, FLAG_REMOVED);	1896	SET_FLAG (op, FLAG_REMOVED);
1675	free_object(op); /* free the inserted object */	1897	free_object (op); /* free the inserted object */
1676	op = tmp;	1898	op = tmp;
1677	remove_ob (op); /* and fix old object's links */	1899	remove_ob (op); /* and fix old object's links */
1678	CLEAR_FLAG(op, FLAG_REMOVED);	1900	CLEAR_FLAG (op, FLAG_REMOVED);
1679	break;	1901	break;
1680	}	1902	}
1681		1903
1682	/* I assume combined objects have no inventory	1904	/* I assume combined objects have no inventory
1683	* We add the weight - this object could have just been removed	1905	* We add the weight - this object could have just been removed
1684	* (if it was possible to merge). calling remove_ob will subtract	1906	* (if it was possible to merge). calling remove_ob will subtract
1685	* the weight, so we need to add it in again, since we actually do	1907	* the weight, so we need to add it in again, since we actually do
1686	* the linking below	1908	* the linking below
1687	*/	1909	*/
1688	add_weight (where, op->weight*op->nrof);	1910	add_weight (where, op->weight * op->nrof);
		1911	}
1689	} else	1912	else
1690	add_weight (where, (op->weight+op->carrying));	1913	add_weight (where, (op->weight + op->carrying));
1691		1914
1692	otmp=is_player_inv(where);	1915	otmp = is_player_inv (where);
1693	if (otmp&&otmp->contr!=NULL) {	1916	if (otmp && otmp->contr != NULL)
		1917	{
1694	if (!QUERY_FLAG(otmp,FLAG_NO_FIX_PLAYER))	1918	if (!QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
1695	fix_player(otmp);	1919	fix_player (otmp);
1696	}	1920	}
1697		1921
1698	op->map=NULL;	1922	op->map = NULL;
1699	op->env=where;	1923	op->env = where;
1700	op->above=NULL;	1924	op->above = NULL;
1701	op->below=NULL;	1925	op->below = NULL;
1702	op->x=0,op->y=0;	1926	op->x = 0, op->y = 0;
1703		1927
1704	/* reset the light list and los of the players on the map */	1928	/* reset the light list and los of the players on the map */
1705	if((op->glow_radius!=0)&&where->map)	1929	if ((op->glow_radius != 0) && where->map)
1706	{	1930	{
1707	#ifdef DEBUG_LIGHTS	1931	#ifdef DEBUG_LIGHTS
1708	LOG(llevDebug, " insert_ob_in_ob(): got %s to insert in map/op\n",	1932	LOG (llevDebug, " insert_ob_in_ob(): got %s to insert in map/op\n", op->name);
1709	op->name);
1710	#endif /* DEBUG_LIGHTS */	1933	#endif /* DEBUG_LIGHTS */
		1934	if (MAP_DARKNESS (where->map))
1711	if (MAP_DARKNESS(where->map)) update_all_los(where->map, where->x, where->y);	1935	update_all_los (where->map, where->x, where->y);
1712	}	1936	}
1713		1937
1714	/* Client has no idea of ordering so lets not bother ordering it here.	1938	/* Client has no idea of ordering so lets not bother ordering it here.
1715	* It sure simplifies this function...	1939	* It sure simplifies this function...
1716	*/	1940	*/
1717	if (where->inv==NULL)	1941	if (where->inv == NULL)
1718	where->inv=op;	1942	where->inv = op;
1719	else {	1943	else
		1944	{
1720	op->below = where->inv;	1945	op->below = where->inv;
1721	op->below->above = op;	1946	op->below->above = op;
1722	where->inv = op;	1947	where->inv = op;
1723	}	1948	}
1724	return op;	1949	return op;
1725	}	1950	}
1726		1951
1727	/*	1952	/*
1728	* Checks if any objects has a move_type that matches objects	1953	* Checks if any objects has a move_type that matches objects
…		…
1743	* MSW 2001-07-08: Check all objects on space, not just those below	1968	* MSW 2001-07-08: Check all objects on space, not just those below
1744	* object being inserted. insert_ob_in_map may not put new objects	1969	* object being inserted. insert_ob_in_map may not put new objects
1745	* on top.	1970	* on top.
1746	*/	1971	*/
1747		1972
		1973	int
1748	int check_move_on (object *op, object *originator)	1974	check_move_on (object *op, object *originator)
1749	{	1975	{
1750	object *tmp;	1976	object *
1751	tag_t tag;	1977	tmp;
1752	mapstruct *m=op->map;	1978	tag_t
		1979	tag;
		1980	mapstruct *
		1981	m = op->map;
		1982	int
1753	int x=op->x, y=op->y;	1983	x = op->x, y = op->y;
1754	MoveType move_on, move_slow, move_block;	1984	MoveType
		1985	move_on,
		1986	move_slow,
		1987	move_block;
1755		1988
1756	if(QUERY_FLAG(op,FLAG_NO_APPLY))	1989	if (QUERY_FLAG (op, FLAG_NO_APPLY))
1757	return 0;	1990	return 0;
1758		1991
1759	tag = op->count;	1992	tag = op->count;
1760		1993
1761	move_on = GET_MAP_MOVE_ON(op->map, op->x, op->y);	1994	move_on = GET_MAP_MOVE_ON (op->map, op->x, op->y);
1762	move_slow = GET_MAP_MOVE_SLOW(op->map, op->x, op->y);	1995	move_slow = GET_MAP_MOVE_SLOW (op->map, op->x, op->y);
1763	move_block = GET_MAP_MOVE_BLOCK(op->map, op->x, op->y);	1996	move_block = GET_MAP_MOVE_BLOCK (op->map, op->x, op->y);
1764		1997
1765	/* if nothing on this space will slow op down or be applied,	1998	/* if nothing on this space will slow op down or be applied,
1766	* no need to do checking below. have to make sure move_type	1999	* no need to do checking below. have to make sure move_type
1767	* is set, as lots of objects don't have it set - we treat that	2000	* is set, as lots of objects don't have it set - we treat that
1768	* as walking.	2001	* as walking.
1769	*/	2002	*/
1770	if (op->move_type && !(op->move_type & move_on) && !(op->move_type & move_slow))	2003	if (op->move_type && !(op->move_type & move_on) && !(op->move_type & move_slow))
1771	return 0;	2004	return 0;
1772		2005
1773	/* This is basically inverse logic of that below - basically,	2006	/* This is basically inverse logic of that below - basically,
1774	* if the object can avoid the move on or slow move, they do so,	2007	* if the object can avoid the move on or slow move, they do so,
1775	* but can't do it if the alternate movement they are using is	2008	* but can't do it if the alternate movement they are using is
1776	* blocked. Logic on this seems confusing, but does seem correct.	2009	* blocked. Logic on this seems confusing, but does seem correct.
1777	*/	2010	*/
1778	if ((op->move_type & ~move_on & ~move_block) != 0 &&	2011	if ((op->move_type & ~move_on & ~move_block) != 0 && (op->move_type & ~move_slow & ~move_block) != 0)
1779	(op->move_type & ~move_slow & ~move_block) != 0) return 0;	2012	return 0;
1780		2013
1781	/* The objects have to be checked from top to bottom.	2014	/* The objects have to be checked from top to bottom.
1782	* Hence, we first go to the top:	2015	* Hence, we first go to the top:
1783	*/	2016	*/
1784		2017
1785	for (tmp=GET_MAP_OB(op->map, op->x, op->y); tmp!=NULL &&	2018	for (tmp = GET_MAP_OB (op->map, op->x, op->y); tmp != NULL && tmp->above != NULL; tmp = tmp->above)
1786	tmp->above!=NULL; tmp=tmp->above) {	2019	{
1787	/* Trim the search when we find the first other spell effect	2020	/* Trim the search when we find the first other spell effect
1788	* this helps performance so that if a space has 50 spell objects,	2021	* this helps performance so that if a space has 50 spell objects,
1789	* we don't need to check all of them.	2022	* we don't need to check all of them.
1790	*/	2023	*/
1791	if ((tmp->move_type & MOVE_FLY_LOW) && QUERY_FLAG(tmp, FLAG_NO_PICK)) break;	2024	if ((tmp->move_type & MOVE_FLY_LOW) && QUERY_FLAG (tmp, FLAG_NO_PICK))
		2025	break;
1792	}	2026	}
1793	for(;tmp!=NULL; tmp=tmp->below) {	2027	for (; tmp != NULL; tmp = tmp->below)
		2028	{
		2029	if (tmp == op)
1794	if (tmp == op) continue; /* Can't apply yourself */	2030	continue; /* Can't apply yourself */
1795		2031
1796	/* Check to see if one of the movement types should be slowed down.	2032	/* Check to see if one of the movement types should be slowed down.
1797	* Second check makes sure that the movement types not being slowed	2033	* Second check makes sure that the movement types not being slowed
1798	* (~slow_move) is not blocked on this space - just because the	2034	* (~slow_move) is not blocked on this space - just because the
1799	* space doesn't slow down swimming (for example), if you can't actually	2035	* space doesn't slow down swimming (for example), if you can't actually
1800	* swim on that space, can't use it to avoid the penalty.	2036	* swim on that space, can't use it to avoid the penalty.
1801	*/	2037	*/
1802	if (!QUERY_FLAG(op, FLAG_WIZPASS)) {	2038	if (!QUERY_FLAG (op, FLAG_WIZPASS))
		2039	{
1803	if ((!op->move_type && tmp->move_slow & MOVE_WALK) ||	2040	if ((!op->move_type && tmp->move_slow & MOVE_WALK) ||
1804	((op->move_type & tmp->move_slow) &&
1805	(op->move_type & ~tmp->move_slow & ~tmp->move_block) == 0)) {	2041	((op->move_type & tmp->move_slow) && (op->move_type & ~tmp->move_slow & ~tmp->move_block) == 0))
		2042	{
1806		2043
		2044	float
1807	float diff;	2045	diff;
1808		2046
1809	diff = tmp->move_slow_penalty*FABS(op->speed);	2047	diff = tmp->move_slow_penalty * FABS (op->speed);
1810	if (op->type == PLAYER) {	2048	if (op->type == PLAYER)
		2049	{
1811	if ((QUERY_FLAG(tmp, FLAG_IS_HILLY) && find_skill_by_number(op, SK_CLIMBING)) ||	2050	if ((QUERY_FLAG (tmp, FLAG_IS_HILLY) && find_skill_by_number (op, SK_CLIMBING)) ||
1812	(QUERY_FLAG(tmp, FLAG_IS_WOODED) && find_skill_by_number(op, SK_WOODSMAN))) {	2051	(QUERY_FLAG (tmp, FLAG_IS_WOODED) && find_skill_by_number (op, SK_WOODSMAN)))
		2052	{
1813	diff /= 4.0;	2053	diff /= 4.0;
1814	}	2054	}
1815	}	2055	}
1816	op->speed_left -= diff;	2056	op->speed_left -= diff;
1817	}	2057	}
1818	}	2058	}
1819		2059
1820	/* Basically same logic as above, except now for actual apply. */	2060	/* Basically same logic as above, except now for actual apply. */
1821	if ((!op->move_type && tmp->move_on & MOVE_WALK) ||	2061	if ((!op->move_type && tmp->move_on & MOVE_WALK) ||
1822	((op->move_type & tmp->move_on) &&
1823	(op->move_type & ~tmp->move_on & ~tmp->move_block)==0)) {	2062	((op->move_type & tmp->move_on) && (op->move_type & ~tmp->move_on & ~tmp->move_block) == 0))
		2063	{
1824		2064
1825	move_apply(tmp, op, originator);	2065	move_apply (tmp, op, originator);
1826	if (was_destroyed (op, tag))	2066	if (was_destroyed (op, tag))
1827	return 1;	2067	return 1;
1828		2068
1829	/* what the person/creature stepped onto has moved the object	2069	/* what the person/creature stepped onto has moved the object
1830	* someplace new. Don't process any further - if we did,	2070	* someplace new. Don't process any further - if we did,
1831	* have a feeling strange problems would result.	2071	* have a feeling strange problems would result.
1832	*/	2072	*/
1833	if (op->map != m || op->x != x || op->y != y) return 0;	2073	if (op->map != m || op->x != x || op->y != y)
		2074	return 0;
1834	}	2075	}
1835	}	2076	}
1836	return 0;	2077	return 0;
1837	}	2078	}
1838		2079
1839	/*	2080	/*
1840	* present_arch(arch, map, x, y) searches for any objects with	2081	* present_arch(arch, map, x, y) searches for any objects with
1841	* a matching archetype at the given map and coordinates.	2082	* a matching archetype at the given map and coordinates.
1842	* The first matching object is returned, or NULL if none.	2083	* The first matching object is returned, or NULL if none.
1843	*/	2084	*/
1844		2085
		2086	object *
1845	object *present_arch(const archetype *at, mapstruct *m, int x, int y) {	2087	present_arch (const archetype *at, mapstruct *m, int x, int y)
		2088	{
1846	object *tmp;	2089	object *
		2090	tmp;
		2091
1847	if(m==NULL || out_of_map(m,x,y)) {	2092	if (m == NULL || out_of_map (m, x, y))
		2093	{
1848	LOG(llevError,"Present_arch called outside map.\n");	2094	LOG (llevError, "Present_arch called outside map.\n");
1849	return NULL;	2095	return NULL;
1850	}	2096	}
1851	for(tmp=GET_MAP_OB(m,x,y); tmp != NULL; tmp = tmp->above)	2097	for (tmp = GET_MAP_OB (m, x, y); tmp != NULL; tmp = tmp->above)
1852	if(tmp->arch == at)	2098	if (tmp->arch == at)
1853	return tmp;	2099	return tmp;
1854	return NULL;	2100	return NULL;
1855	}	2101	}
1856		2102
1857	/*	2103	/*
1858	* present(type, map, x, y) searches for any objects with	2104	* present(type, map, x, y) searches for any objects with
1859	* a matching type variable at the given map and coordinates.	2105	* a matching type variable at the given map and coordinates.
1860	* The first matching object is returned, or NULL if none.	2106	* The first matching object is returned, or NULL if none.
1861	*/	2107	*/
1862		2108
		2109	object *
1863	object *present(unsigned char type,mapstruct *m, int x,int y) {	2110	present (unsigned char type, mapstruct *m, int x, int y)
		2111	{
1864	object *tmp;	2112	object *
		2113	tmp;
		2114
1865	if(out_of_map(m,x,y)) {	2115	if (out_of_map (m, x, y))
		2116	{
1866	LOG(llevError,"Present called outside map.\n");	2117	LOG (llevError, "Present called outside map.\n");
1867	return NULL;	2118	return NULL;
1868	}	2119	}
1869	for(tmp=GET_MAP_OB(m,x,y);tmp!=NULL;tmp=tmp->above)	2120	for (tmp = GET_MAP_OB (m, x, y); tmp != NULL; tmp = tmp->above)
1870	if(tmp->type==type)	2121	if (tmp->type == type)
1871	return tmp;	2122	return tmp;
1872	return NULL;	2123	return NULL;
1873	}	2124	}
1874		2125
1875	/*	2126	/*
1876	* present_in_ob(type, object) searches for any objects with	2127	* present_in_ob(type, object) searches for any objects with
1877	* a matching type variable in the inventory of the given object.	2128	* a matching type variable in the inventory of the given object.
1878	* The first matching object is returned, or NULL if none.	2129	* The first matching object is returned, or NULL if none.
1879	*/	2130	*/
1880		2131
		2132	object *
1881	object *present_in_ob(unsigned char type, const object *op) {	2133	present_in_ob (unsigned char type, const object *op)
		2134	{
1882	object *tmp;	2135	object *
		2136	tmp;
		2137
1883	for(tmp=op->inv;tmp!=NULL;tmp=tmp->below)	2138	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)
1884	if(tmp->type==type)	2139	if (tmp->type == type)
1885	return tmp;	2140	return tmp;
1886	return NULL;	2141	return NULL;
1887	}	2142	}
1888		2143
1889	/*	2144	/*
…		…
1899	* the object name, not the archetype name. this is so that the	2154	* the object name, not the archetype name. this is so that the
1900	* spell code can use one object type (force), but change it's name	2155	* spell code can use one object type (force), but change it's name
1901	* to be unique.	2156	* to be unique.
1902	*/	2157	*/
1903		2158
		2159	object *
1904	object *present_in_ob_by_name(int type, const char *str, const object *op) {	2160	present_in_ob_by_name (int type, const char *str, const object *op)
		2161	{
1905	object *tmp;	2162	object *
		2163	tmp;
1906		2164
1907	for(tmp=op->inv; tmp!=NULL; tmp=tmp->below) {	2165	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)
		2166	{
1908	if ((type==-1 || tmp->type==type) && (!strcmp(str, tmp->name)))	2167	if ((type == -1 || tmp->type == type) && (!strcmp (str, tmp->name)))
1909	return tmp;	2168	return tmp;
1910	}	2169	}
1911	return NULL;	2170	return NULL;
1912	}	2171	}
1913		2172
1914	/*	2173	/*
1915	* present_arch_in_ob(archetype, object) searches for any objects with	2174	* present_arch_in_ob(archetype, object) searches for any objects with
1916	* a matching archetype in the inventory of the given object.	2175	* a matching archetype in the inventory of the given object.
1917	* The first matching object is returned, or NULL if none.	2176	* The first matching object is returned, or NULL if none.
1918	*/	2177	*/
1919		2178
		2179	object *
1920	object *present_arch_in_ob(const archetype *at, const object *op) {	2180	present_arch_in_ob (const archetype *at, const object *op)
		2181	{
1921	object *tmp;	2182	object *
		2183	tmp;
		2184
1922	for(tmp=op->inv;tmp!=NULL;tmp=tmp->below)	2185	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)
1923	if( tmp->arch == at)	2186	if (tmp->arch == at)
1924	return tmp;	2187	return tmp;
1925	return NULL;	2188	return NULL;
1926	}	2189	}
1927		2190
1928	/*	2191	/*
1929	* activate recursively a flag on an object inventory	2192	* activate recursively a flag on an object inventory
1930	*/	2193	*/
		2194	void
1931	void flag_inv(object*op, int flag){	2195	flag_inv (object *op, int flag)
		2196	{
1932	object *tmp;	2197	object *
		2198	tmp;
		2199
1933	if(op->inv)	2200	if (op->inv)
1934	for(tmp = op->inv; tmp != NULL; tmp = tmp->below){	2201	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)
		2202	{
1935	SET_FLAG(tmp, flag);	2203	SET_FLAG (tmp, flag);
1936	flag_inv(tmp,flag);	2204	flag_inv (tmp, flag);
1937	}	2205	}
1938	}/*	2206	} /*
1939	* desactivate recursively a flag on an object inventory	2207	* desactivate recursively a flag on an object inventory
1940	*/	2208	*/
		2209	void
1941	void unflag_inv(object*op, int flag){	2210	unflag_inv (object *op, int flag)
		2211	{
1942	object *tmp;	2212	object *
		2213	tmp;
		2214
1943	if(op->inv)	2215	if (op->inv)
1944	for(tmp = op->inv; tmp != NULL; tmp = tmp->below){	2216	for (tmp = op->inv; tmp != NULL; tmp = tmp->below)
		2217	{
1945	CLEAR_FLAG(tmp, flag);	2218	CLEAR_FLAG (tmp, flag);
1946	unflag_inv(tmp,flag);	2219	unflag_inv (tmp, flag);
1947	}	2220	}
1948	}	2221	}
1949		2222
1950	/*	2223	/*
1951	* set_cheat(object) sets the cheat flag (WAS_WIZ) in the object and in	2224	* set_cheat(object) sets the cheat flag (WAS_WIZ) in the object and in
1952	* all it's inventory (recursively).	2225	* all it's inventory (recursively).
1953	* If checksums are used, a player will get set_cheat called for	2226	* If checksums are used, a player will get set_cheat called for
1954	* him/her-self and all object carried by a call to this function.	2227	* him/her-self and all object carried by a call to this function.
1955	*/	2228	*/
1956		2229
		2230	void
1957	void set_cheat(object *op) {	2231	set_cheat (object *op)
		2232	{
1958	SET_FLAG(op, FLAG_WAS_WIZ);	2233	SET_FLAG (op, FLAG_WAS_WIZ);
1959	flag_inv(op, FLAG_WAS_WIZ);	2234	flag_inv (op, FLAG_WAS_WIZ);
1960	}	2235	}
1961		2236
1962	/*	2237	/*
1963	* find_free_spot(object, map, x, y, start, stop) will search for	2238	* find_free_spot(object, map, x, y, start, stop) will search for
1964	* a spot at the given map and coordinates which will be able to contain	2239	* a spot at the given map and coordinates which will be able to contain
…		…
1979	* to know if the space in question will block the object. We can't use	2254	* to know if the space in question will block the object. We can't use
1980	* the archetype because that isn't correct if the monster has been	2255	* the archetype because that isn't correct if the monster has been
1981	* customized, changed states, etc.	2256	* customized, changed states, etc.
1982	*/	2257	*/
1983		2258
		2259	int
1984	int find_free_spot(const object *ob, mapstruct *m,int x,int y,int start,int stop) {	2260	find_free_spot (const object *ob, mapstruct *m, int x, int y, int start, int stop)
		2261	{
		2262	int
		2263	i,
1985	int i,index=0, flag;	2264	index = 0, flag;
		2265	static int
1986	static int altern[SIZEOFFREE];	2266	altern[SIZEOFFREE];
1987		2267
1988	for(i=start;i<stop;i++) {	2268	for (i = start; i < stop; i++)
		2269	{
1989	flag = ob_blocked(ob,m,x+freearr_x[i],y+freearr_y[i]);	2270	flag = ob_blocked (ob, m, x + freearr_x[i], y + freearr_y[i]);
1990	if(!flag)	2271	if (!flag)
1991	altern[index++]=i;	2272	altern[index++] = i;
1992		2273
1993	/* Basically, if we find a wall on a space, we cut down the search size.	2274	/* Basically, if we find a wall on a space, we cut down the search size.
1994	* In this way, we won't return spaces that are on another side of a wall.	2275	* In this way, we won't return spaces that are on another side of a wall.
1995	* This mostly work, but it cuts down the search size in all directions -	2276	* This mostly work, but it cuts down the search size in all directions -
1996	* if the space being examined only has a wall to the north and empty	2277	* if the space being examined only has a wall to the north and empty
1997	* spaces in all the other directions, this will reduce the search space	2278	* spaces in all the other directions, this will reduce the search space
1998	* to only the spaces immediately surrounding the target area, and	2279	* to only the spaces immediately surrounding the target area, and
1999	* won't look 2 spaces south of the target space.	2280	* won't look 2 spaces south of the target space.
2000	*/	2281	*/
2001	else if ((flag & AB_NO_PASS) && maxfree[i]<stop)	2282	else if ((flag & AB_NO_PASS) && maxfree[i] < stop)
2002	stop=maxfree[i];	2283	stop = maxfree[i];
2003	}	2284	}
2004	if(!index) return -1;	2285	if (!index)
		2286	return -1;
2005	return altern[RANDOM()%index];	2287	return altern[RANDOM () % index];
2006	}	2288	}
2007		2289
2008	/*	2290	/*
2009	* find_first_free_spot(archetype, mapstruct, x, y) works like	2291	* find_first_free_spot(archetype, mapstruct, x, y) works like
2010	* find_free_spot(), but it will search max number of squares.	2292	* find_free_spot(), but it will search max number of squares.
2011	* But it will return the first available spot, not a random choice.	2293	* But it will return the first available spot, not a random choice.
2012	* Changed 0.93.2: Have it return -1 if there is no free spot available.	2294	* Changed 0.93.2: Have it return -1 if there is no free spot available.
2013	*/	2295	*/
2014		2296
		2297	int
2015	int find_first_free_spot(const object *ob, mapstruct *m,int x,int y) {	2298	find_first_free_spot (const object *ob, mapstruct *m, int x, int y)
		2299	{
		2300	int
2016	int i;	2301	i;
		2302
2017	for(i=0;i<SIZEOFFREE;i++) {	2303	for (i = 0; i < SIZEOFFREE; i++)
		2304	{
2018	if(!ob_blocked(ob,m,x+freearr_x[i],y+freearr_y[i]))	2305	if (!ob_blocked (ob, m, x + freearr_x[i], y + freearr_y[i]))
2019	return i;	2306	return i;
2020	}	2307	}
2021	return -1;	2308	return -1;
2022	}	2309	}
2023		2310
2024	/*	2311	/*
2025	* The function permute(arr, begin, end) randomly reorders the array	2312	* The function permute(arr, begin, end) randomly reorders the array
2026	* arr[begin..end-1].	2313	* arr[begin..end-1].
2027	*/	2314	*/
		2315	static void
2028	static void permute(int *arr, int begin, int end)	2316	permute (int *arr, int begin, int end)
2029	{	2317	{
2030	int i, j, tmp, len;	2318	int
		2319	i,
		2320	j,
		2321	tmp,
		2322	len;
2031		2323
2032	len = end-begin;	2324	len = end - begin;
2033	for(i = begin; i < end; i++)	2325	for (i = begin; i < end; i++)
2034	{	2326	{
2035	j = begin+RANDOM()%len;	2327	j = begin + RANDOM () % len;
2036		2328
2037	tmp = arr[i];	2329	tmp = arr[i];
2038	arr[i] = arr[j];	2330	arr[i] = arr[j];
2039	arr[j] = tmp;	2331	arr[j] = tmp;
2040	}	2332	}
2041	}	2333	}
2042		2334
2043	/* new function to make monster searching more efficient, and effective!	2335	/* new function to make monster searching more efficient, and effective!
2044	* This basically returns a randomized array (in the passed pointer) of	2336	* This basically returns a randomized array (in the passed pointer) of
2045	* the spaces to find monsters. In this way, it won't always look for	2337	* the spaces to find monsters. In this way, it won't always look for
2046	* monsters to the north first. However, the size of the array passed	2338	* monsters to the north first. However, the size of the array passed
2047	* covers all the spaces, so within that size, all the spaces within	2339	* covers all the spaces, so within that size, all the spaces within
2048	* the 3x3 area will be searched, just not in a predictable order.	2340	* the 3x3 area will be searched, just not in a predictable order.
2049	*/	2341	*/
		2342	void
2050	void get_search_arr(int *search_arr)	2343	get_search_arr (int *search_arr)
2051	{	2344	{
		2345	int
2052	int i;	2346	i;
2053		2347
2054	for(i = 0; i < SIZEOFFREE; i++)	2348	for (i = 0; i < SIZEOFFREE; i++)
2055	{	2349	{
2056	search_arr[i] = i;	2350	search_arr[i] = i;
2057	}	2351	}
2058		2352
2059	permute(search_arr, 1, SIZEOFFREE1+1);	2353	permute (search_arr, 1, SIZEOFFREE1 + 1);
2060	permute(search_arr, SIZEOFFREE1+1, SIZEOFFREE2+1);	2354	permute (search_arr, SIZEOFFREE1 + 1, SIZEOFFREE2 + 1);
2061	permute(search_arr, SIZEOFFREE2+1, SIZEOFFREE);	2355	permute (search_arr, SIZEOFFREE2 + 1, SIZEOFFREE);
2062	}	2356	}
2063		2357
2064	/*	2358	/*
2065	* find_dir(map, x, y, exclude) will search some close squares in the	2359	* find_dir(map, x, y, exclude) will search some close squares in the
2066	* given map at the given coordinates for live objects.	2360	* given map at the given coordinates for live objects.
…		…
2072	* is actually want is going to try and move there. We need this info	2366	* is actually want is going to try and move there. We need this info
2073	* because we have to know what movement the thing looking to move	2367	* because we have to know what movement the thing looking to move
2074	* there is capable of.	2368	* there is capable of.
2075	*/	2369	*/
2076		2370
		2371	int
2077	int find_dir(mapstruct *m, int x, int y, object *exclude) {	2372	find_dir (mapstruct *m, int x, int y, object *exclude)
		2373	{
		2374	int
		2375	i,
2078	int i,max=SIZEOFFREE, mflags;	2376	max = SIZEOFFREE, mflags;
2079	sint16 nx, ny;	2377	sint16
		2378	nx,
		2379	ny;
2080	object *tmp;	2380	object *
		2381	tmp;
2081	mapstruct *mp;	2382	mapstruct *
2082	MoveType blocked, move_type;	2383	mp;
		2384	MoveType
		2385	blocked,
		2386	move_type;
2083		2387
2084	if (exclude && exclude->head) {	2388	if (exclude && exclude->head)
		2389	{
2085	exclude = exclude->head;	2390	exclude = exclude->head;
2086	move_type = exclude->move_type;	2391	move_type = exclude->move_type;
2087	} else {	2392	}
		2393	else
		2394	{
2088	/* If we don't have anything, presume it can use all movement types. */	2395	/* If we don't have anything, presume it can use all movement types. */
2089	move_type=MOVE_ALL;	2396	move_type = MOVE_ALL;
		2397	}
		2398
		2399	for (i = 1; i < max; i++)
2090	}	2400	{
2091
2092	for(i=1;i<max;i++) {
2093	mp = m;	2401	mp = m;
2094	nx = x + freearr_x[i];	2402	nx = x + freearr_x[i];
2095	ny = y + freearr_y[i];	2403	ny = y + freearr_y[i];
2096		2404
2097	mflags = get_map_flags(m, &mp, nx, ny, &nx, &ny);	2405	mflags = get_map_flags (m, &mp, nx, ny, &nx, &ny);
2098	if (mflags & P_OUT_OF_MAP) {	2406	if (mflags & P_OUT_OF_MAP)
		2407	{
2099	max = maxfree[i];	2408	max = maxfree[i];
		2409	}
2100	} else {	2410	else
		2411	{
2101	blocked = GET_MAP_MOVE_BLOCK(mp, nx, ny);	2412	blocked = GET_MAP_MOVE_BLOCK (mp, nx, ny);
2102		2413
2103	if ((move_type & blocked) == move_type) {	2414	if ((move_type & blocked) == move_type)
		2415	{
2104	max=maxfree[i];	2416	max = maxfree[i];
		2417	}
2105	} else if (mflags & P_IS_ALIVE) {	2418	else if (mflags & P_IS_ALIVE)
		2419	{
2106	for (tmp=GET_MAP_OB(mp,nx,ny); tmp!= NULL; tmp=tmp->above) {	2420	for (tmp = GET_MAP_OB (mp, nx, ny); tmp != NULL; tmp = tmp->above)
2107	if ((QUERY_FLAG(tmp,FLAG_MONSTER) || tmp->type==PLAYER) &&	2421	{
2108	(tmp != exclude ||(tmp->head && tmp->head != exclude))) {	2422	if ((QUERY_FLAG (tmp, FLAG_MONSTER) || tmp->type == PLAYER) && (tmp != exclude || (tmp->head && tmp->head != exclude)))
		2423	{
2109	break;	2424	break;
2110	}	2425	}
2111	}	2426	}
2112	if(tmp) {	2427	if (tmp)
		2428	{
2113	return freedir[i];	2429	return freedir[i];
2114	}	2430	}
2115	}	2431	}
2116	}	2432	}
2117	}	2433	}
2118	return 0;	2434	return 0;
2119	}	2435	}
2120		2436
2121	/*	2437	/*
2122	* distance(object 1, object 2) will return the square of the	2438	* distance(object 1, object 2) will return the square of the
2123	* distance between the two given objects.	2439	* distance between the two given objects.
2124	*/	2440	*/
2125		2441
		2442	int
2126	int distance(const object *ob1, const object *ob2) {	2443	distance (const object *ob1, const object *ob2)
		2444	{
2127	int i;	2445	int
2128	i= (ob1->x - ob2->x)*(ob1->x - ob2->x)+	2446	i;
2129	(ob1->y - ob2->y)*(ob1->y - ob2->y);	2447
		2448	i = (ob1->x - ob2->x) * (ob1->x - ob2->x) + (ob1->y - ob2->y) * (ob1->y - ob2->y);
2130	return i;	2449	return i;
2131	}	2450	}
2132		2451
2133	/*	2452	/*
2134	* find_dir_2(delta-x,delta-y) will return a direction in which	2453	* find_dir_2(delta-x,delta-y) will return a direction in which
2135	* an object which has subtracted the x and y coordinates of another	2454	* an object which has subtracted the x and y coordinates of another
2136	* object, needs to travel toward it.	2455	* object, needs to travel toward it.
2137	*/	2456	*/
2138		2457
		2458	int
2139	int find_dir_2(int x, int y) {	2459	find_dir_2 (int x, int y)
		2460	{
2140	int q;	2461	int
		2462	q;
2141		2463
2142	if(y)	2464	if (y)
2143	q=x*100/y;	2465	q = x * 100 / y;
2144	else if (x)	2466	else if (x)
2145	q= -300*x;	2467	q = -300 * x;
2146	else	2468	else
2147	return 0;	2469	return 0;
2148		2470
2149	if(y>0) {	2471	if (y > 0)
		2472	{
2150	if(q < -242)	2473	if (q < -242)
2151	return 3 ;	2474	return 3;
2152	if (q < -41)	2475	if (q < -41)
2153	return 2 ;	2476	return 2;
2154	if (q < 41)	2477	if (q < 41)
2155	return 1 ;	2478	return 1;
2156	if (q < 242)	2479	if (q < 242)
2157	return 8 ;	2480	return 8;
2158	return 7 ;	2481	return 7;
2159	}	2482	}
2160		2483
2161	if (q < -242)	2484	if (q < -242)
2162	return 7 ;	2485	return 7;
2163	if (q < -41)	2486	if (q < -41)
2164	return 6 ;	2487	return 6;
2165	if (q < 41)	2488	if (q < 41)
2166	return 5 ;	2489	return 5;
2167	if (q < 242)	2490	if (q < 242)
2168	return 4 ;	2491	return 4;
2169		2492
2170	return 3 ;	2493	return 3;
2171	}	2494	}
2172		2495
2173	/*	2496	/*
2174	* absdir(int): Returns a number between 1 and 8, which represent	2497	* absdir(int): Returns a number between 1 and 8, which represent
2175	* the "absolute" direction of a number (it actually takes care of	2498	* the "absolute" direction of a number (it actually takes care of
2176	* "overflow" in previous calculations of a direction).	2499	* "overflow" in previous calculations of a direction).
2177	*/	2500	*/
2178		2501
		2502	int
2179	int absdir(int d) {	2503	absdir (int d)
2180	while(d<1) d+=8;	2504	{
2181	while(d>8) d-=8;	2505	while (d < 1)
		2506	d += 8;
		2507	while (d > 8)
		2508	d -= 8;
2182	return d;	2509	return d;
2183	}	2510	}
2184		2511
2185	/*	2512	/*
2186	* dirdiff(dir1, dir2) returns how many 45-degrees differences there is	2513	* dirdiff(dir1, dir2) returns how many 45-degrees differences there is
2187	* between two directions (which are expected to be absolute (see absdir())	2514	* between two directions (which are expected to be absolute (see absdir())
2188	*/	2515	*/
2189		2516
		2517	int
2190	int dirdiff(int dir1, int dir2) {	2518	dirdiff (int dir1, int dir2)
		2519	{
2191	int d;	2520	int
		2521	d;
		2522
2192	d = abs(dir1 - dir2);	2523	d = abs (dir1 - dir2);
2193	if(d>4)	2524	if (d > 4)
2194	d = 8 - d;	2525	d = 8 - d;
2195	return d;	2526	return d;
2196	}	2527	}
2197		2528
2198	/* peterm:	2529	/* peterm:
…		…
2203	* direction 4, 14, or 16 to get back to where we are.	2534	* direction 4, 14, or 16 to get back to where we are.
2204	* Moved from spell_util.c to object.c with the other related direction	2535	* Moved from spell_util.c to object.c with the other related direction
2205	* functions.	2536	* functions.
2206	*/	2537	*/
2207		2538
		2539	int
2208	int reduction_dir[SIZEOFFREE][3] = {	2540	reduction_dir[SIZEOFFREE][3] = {
2209	{0,0,0}, /* 0 */	2541	{0, 0, 0}, /* 0 */
2210	{0,0,0}, /* 1 */	2542	{0, 0, 0}, /* 1 */
2211	{0,0,0}, /* 2 */	2543	{0, 0, 0}, /* 2 */
2212	{0,0,0}, /* 3 */	2544	{0, 0, 0}, /* 3 */
2213	{0,0,0}, /* 4 */	2545	{0, 0, 0}, /* 4 */
2214	{0,0,0}, /* 5 */	2546	{0, 0, 0}, /* 5 */
2215	{0,0,0}, /* 6 */	2547	{0, 0, 0}, /* 6 */
2216	{0,0,0}, /* 7 */	2548	{0, 0, 0}, /* 7 */
2217	{0,0,0}, /* 8 */	2549	{0, 0, 0}, /* 8 */
2218	{8,1,2}, /* 9 */	2550	{8, 1, 2}, /* 9 */
2219	{1,2,-1}, /* 10 */	2551	{1, 2, -1}, /* 10 */
2220	{2,10,12}, /* 11 */	2552	{2, 10, 12}, /* 11 */
2221	{2,3,-1}, /* 12 */	2553	{2, 3, -1}, /* 12 */
2222	{2,3,4}, /* 13 */	2554	{2, 3, 4}, /* 13 */
2223	{3,4,-1}, /* 14 */	2555	{3, 4, -1}, /* 14 */
2224	{4,14,16}, /* 15 */	2556	{4, 14, 16}, /* 15 */
2225	{5,4,-1}, /* 16 */	2557	{5, 4, -1}, /* 16 */
2226	{4,5,6}, /* 17 */	2558	{4, 5, 6}, /* 17 */
2227	{6,5,-1}, /* 18 */	2559	{6, 5, -1}, /* 18 */
2228	{6,20,18}, /* 19 */	2560	{6, 20, 18}, /* 19 */
2229	{7,6,-1}, /* 20 */	2561	{7, 6, -1}, /* 20 */
2230	{6,7,8}, /* 21 */	2562	{6, 7, 8}, /* 21 */
2231	{7,8,-1}, /* 22 */	2563	{7, 8, -1}, /* 22 */
2232	{8,22,24}, /* 23 */	2564	{8, 22, 24}, /* 23 */
2233	{8,1,-1}, /* 24 */	2565	{8, 1, -1}, /* 24 */
2234	{24,9,10}, /* 25 */	2566	{24, 9, 10}, /* 25 */
2235	{9,10,-1}, /* 26 */	2567	{9, 10, -1}, /* 26 */
2236	{10,11,-1}, /* 27 */	2568	{10, 11, -1}, /* 27 */
2237	{27,11,29}, /* 28 */	2569	{27, 11, 29}, /* 28 */
2238	{11,12,-1}, /* 29 */	2570	{11, 12, -1}, /* 29 */
2239	{12,13,-1}, /* 30 */	2571	{12, 13, -1}, /* 30 */
2240	{12,13,14}, /* 31 */	2572	{12, 13, 14}, /* 31 */
2241	{13,14,-1}, /* 32 */	2573	{13, 14, -1}, /* 32 */
2242	{14,15,-1}, /* 33 */	2574	{14, 15, -1}, /* 33 */
2243	{33,15,35}, /* 34 */	2575	{33, 15, 35}, /* 34 */
2244	{16,15,-1}, /* 35 */	2576	{16, 15, -1}, /* 35 */
2245	{17,16,-1}, /* 36 */	2577	{17, 16, -1}, /* 36 */
2246	{18,17,16}, /* 37 */	2578	{18, 17, 16}, /* 37 */
2247	{18,17,-1}, /* 38 */	2579	{18, 17, -1}, /* 38 */
2248	{18,19,-1}, /* 39 */	2580	{18, 19, -1}, /* 39 */
2249	{41,19,39}, /* 40 */	2581	{41, 19, 39}, /* 40 */
2250	{19,20,-1}, /* 41 */	2582	{19, 20, -1}, /* 41 */
2251	{20,21,-1}, /* 42 */	2583	{20, 21, -1}, /* 42 */
2252	{20,21,22}, /* 43 */	2584	{20, 21, 22}, /* 43 */
2253	{21,22,-1}, /* 44 */	2585	{21, 22, -1}, /* 44 */
2254	{23,22,-1}, /* 45 */	2586	{23, 22, -1}, /* 45 */
2255	{45,47,23}, /* 46 */	2587	{45, 47, 23}, /* 46 */
2256	{23,24,-1}, /* 47 */	2588	{23, 24, -1}, /* 47 */
2257	{24,9,-1}}; /* 48 */	2589	{24, 9, -1}
		2590	}; /* 48 */
2258		2591
2259	/* Recursive routine to step back and see if we can	2592	/* Recursive routine to step back and see if we can
2260	* find a path to that monster that we found. If not,	2593	* find a path to that monster that we found. If not,
2261	* we don't bother going toward it. Returns 1 if we	2594	* we don't bother going toward it. Returns 1 if we
2262	* can see a direct way to get it	2595	* can see a direct way to get it
2263	* Modified to be map tile aware -.MSW	2596	* Modified to be map tile aware -.MSW
2264	*/	2597	*/
2265
2266		2598
		2599
		2600	int
2267	int can_see_monsterP(mapstruct *m, int x, int y,int dir) {	2601	can_see_monsterP (mapstruct *m, int x, int y, int dir)
2268	sint16 dx, dy;	2602	{
		2603	sint16
		2604	dx,
		2605	dy;
		2606	int
2269	int mflags;	2607	mflags;
2270		2608
		2609	if (dir < 0)
2271	if(dir<0) return 0; /* exit condition: invalid direction */	2610	return 0; /* exit condition: invalid direction */
2272		2611
2273	dx = x + freearr_x[dir];	2612	dx = x + freearr_x[dir];
2274	dy = y + freearr_y[dir];	2613	dy = y + freearr_y[dir];
2275		2614
2276	mflags = get_map_flags(m, &m, dx, dy, &dx, &dy);	2615	mflags = get_map_flags (m, &m, dx, dy, &dx, &dy);
2277		2616
2278	/* This functional arguably was incorrect before - it was	2617	/* This functional arguably was incorrect before - it was
2279	* checking for P_WALL - that was basically seeing if	2618	* checking for P_WALL - that was basically seeing if
2280	* we could move to the monster - this is being more	2619	* we could move to the monster - this is being more
2281	* literal on if we can see it. To know if we can actually	2620	* literal on if we can see it. To know if we can actually
2282	* move to the monster, we'd need the monster passed in or	2621	* move to the monster, we'd need the monster passed in or
2283	* at least its move type.	2622	* at least its move type.
2284	*/	2623	*/
2285	if (mflags & (P_OUT_OF_MAP | P_BLOCKSVIEW)) return 0;	2624	if (mflags & (P_OUT_OF_MAP | P_BLOCKSVIEW))
		2625	return 0;
2286		2626
2287	/* yes, can see. */	2627	/* yes, can see. */
2288	if(dir < 9) return 1;	2628	if (dir < 9)
		2629	return 1;
2289	return can_see_monsterP(m, x, y, reduction_dir[dir][0]) |	2630	return can_see_monsterP (m, x, y, reduction_dir[dir][0]) |
2290	can_see_monsterP(m,x,y, reduction_dir[dir][1]) |	2631	can_see_monsterP (m, x, y, reduction_dir[dir][1]) | can_see_monsterP (m, x, y, reduction_dir[dir][2]);
2291	can_see_monsterP(m,x,y, reduction_dir[dir][2]);
2292	}	2632	}
2293		2633
2294		2634
2295		2635
2296	/*	2636	/*
2297	* can_pick(picker, item): finds out if an object is possible to be	2637	* can_pick(picker, item): finds out if an object is possible to be
2298	* picked up by the picker. Returnes 1 if it can be	2638	* picked up by the picker. Returnes 1 if it can be
2299	* picked up, otherwise 0.	2639	* picked up, otherwise 0.
2300	*	2640	*
…		…
2302	* core dumps if they do.	2642	* core dumps if they do.
2303	*	2643	*
2304	* Add a check so we can't pick up invisible objects (0.93.8)	2644	* Add a check so we can't pick up invisible objects (0.93.8)
2305	*/	2645	*/
2306		2646
		2647	int
2307	int can_pick(const object *who, const object *item) {	2648	can_pick (const object *who, const object *item)
		2649	{
2308	return /*QUERY_FLAG(who,FLAG_WIZ)||*/	2650	return /*QUERY_FLAG(who,FLAG_WIZ)|| */
2309	(item->weight>0&&!QUERY_FLAG(item,FLAG_NO_PICK)&&	2651	(item->weight > 0 && !QUERY_FLAG (item, FLAG_NO_PICK) &&
2310	!QUERY_FLAG(item,FLAG_ALIVE)&&!item->invisible &&	2652	!QUERY_FLAG (item, FLAG_ALIVE) && !item->invisible && (who->type == PLAYER || item->weight < who->weight / 3));
2311	(who->type==PLAYER||item->weight<who->weight/3));
2312	}	2653	}
2313		2654
2314		2655
2315	/*	2656	/*
2316	* create clone from object to another	2657	* create clone from object to another
2317	*/	2658	*/
		2659	object *
2318	object *object_create_clone (object *asrc) {	2660	object_create_clone (object *asrc)
		2661	{
		2662	object *
2319	object *dst = NULL,*tmp,*src,*part,*prev, *item;	2663	dst = NULL, *tmp, *src, *part, *prev, *item;
2320		2664
		2665	if (!asrc)
2321	if(!asrc) return NULL;	2666	return NULL;
2322	src = asrc;	2667	src = asrc;
2323	if(src->head)	2668	if (src->head)
2324	src = src->head;	2669	src = src->head;
2325		2670
2326	prev = NULL;	2671	prev = NULL;
2327	for(part = src; part; part = part->more) {	2672	for (part = src; part; part = part->more)
		2673	{
2328	tmp = get_object();	2674	tmp = get_object ();
2329	copy_object(part,tmp);	2675	copy_object (part, tmp);
2330	tmp->x -= src->x;	2676	tmp->x -= src->x;
2331	tmp->y -= src->y;	2677	tmp->y -= src->y;
2332	if(!part->head) {	2678	if (!part->head)
		2679	{
2333	dst = tmp;	2680	dst = tmp;
2334	tmp->head = NULL;	2681	tmp->head = NULL;
		2682	}
2335	} else {	2683	else
		2684	{
2336	tmp->head = dst;	2685	tmp->head = dst;
2337	}	2686	}
2338	tmp->more = NULL;	2687	tmp->more = NULL;
2339	if(prev)	2688	if (prev)
2340	prev->more = tmp;	2689	prev->more = tmp;
2341	prev = tmp;	2690	prev = tmp;
2342	}	2691	}
		2692
2343	/*** copy inventory ***/	2693	/*** copy inventory ***/
2344	for(item = src->inv; item; item = item->below) {	2694	for (item = src->inv; item; item = item->below)
		2695	{
2345	(void) insert_ob_in_ob(object_create_clone(item),dst);	2696	(void) insert_ob_in_ob (object_create_clone (item), dst);
2346	}	2697	}
2347		2698
2348	return dst;	2699	return dst;
2349	}	2700	}
2350		2701
2351	/* return true if the object was destroyed, 0 otherwise */	2702	/* return true if the object was destroyed, 0 otherwise */
		2703	int
2352	int was_destroyed (const object *op, tag_t old_tag)	2704	was_destroyed (const object *op, tag_t old_tag)
2353	{	2705	{
2354	/* checking for FLAG_FREED isn't necessary, but makes this function more	2706	/* checking for FLAG_FREED isn't necessary, but makes this function more
2355	* robust */	2707	* robust */
2356	return op->count != old_tag || QUERY_FLAG (op, FLAG_FREED);	2708	return op->count != old_tag || QUERY_FLAG (op, FLAG_FREED);
2357	}	2709	}
2358		2710
2359	/* GROS - Creates an object using a string representing its content. */	2711	/* GROS - Creates an object using a string representing its content. */
		2712
2360	/* Basically, we save the content of the string to a temp file, then call */	2713	/* Basically, we save the content of the string to a temp file, then call */
		2714
2361	/* load_object on it. I admit it is a highly inefficient way to make things, */	2715	/* load_object on it. I admit it is a highly inefficient way to make things, */
		2716
2362	/* but it was simple to make and allows reusing the load_object function. */	2717	/* but it was simple to make and allows reusing the load_object function. */
		2718
2363	/* Remember not to use load_object_str in a time-critical situation. */	2719	/* Remember not to use load_object_str in a time-critical situation. */
		2720
2364	/* Also remember that multiparts objects are not supported for now. */	2721	/* Also remember that multiparts objects are not supported for now. */
2365		2722
		2723	object *
2366	object* load_object_str(const char *obstr)	2724	load_object_str (const char *obstr)
2367	{	2725	{
2368	object *op;	2726	object *
		2727	op;
		2728	char
2369	char filename[MAX_BUF];	2729	filename[MAX_BUF];
		2730
2370	sprintf(filename,"%s/cfloadobstr2044",settings.tmpdir);	2731	sprintf (filename, "%s/cfloadobstr2044", settings.tmpdir);
2371		2732
		2733	FILE *
2372	FILE *tempfile=fopen(filename,"w");	2734	tempfile = fopen (filename, "w");
		2735
2373	if (tempfile == NULL)	2736	if (tempfile == NULL)
2374	{	2737	{
2375	LOG(llevError,"Error - Unable to access load object temp file\n");	2738	LOG (llevError, "Error - Unable to access load object temp file\n");
2376	return NULL;	2739	return NULL;
2377	};	2740	};
2378	fprintf(tempfile,obstr);	2741	fprintf (tempfile, obstr);
2379	fclose(tempfile);	2742	fclose (tempfile);
2380		2743
2381	op=get_object();	2744	op = get_object ();
2382		2745
2383	object_thawer thawer (filename);	2746	object_thawer
		2747	thawer (filename);
2384		2748
2385	if (thawer)	2749	if (thawer)
2386	load_object(thawer,op,0);	2750	load_object (thawer, op, 0);
2387		2751
2388	LOG(llevDebug," load str completed, object=%s\n", &op->name);	2752	LOG (llevDebug, " load str completed, object=%s\n", &op->name);
2389	CLEAR_FLAG(op,FLAG_REMOVED);	2753	CLEAR_FLAG (op, FLAG_REMOVED);
2390		2754
2391	return op;	2755	return op;
2392	}	2756	}
2393		2757
2394	/* This returns the first object in who's inventory that	2758	/* This returns the first object in who's inventory that
2395	* has the same type and subtype match.	2759	* has the same type and subtype match.
2396	* returns NULL if no match.	2760	* returns NULL if no match.
2397	*/	2761	*/
		2762	object *
2398	object *find_obj_by_type_subtype(const object *who, int type, int subtype)	2763	find_obj_by_type_subtype (const object *who, int type, int subtype)
2399	{	2764	{
2400	object *tmp;	2765	object *
		2766	tmp;
2401		2767
2402	for (tmp=who->inv; tmp; tmp=tmp->below)	2768	for (tmp = who->inv; tmp; tmp = tmp->below)
2403	if (tmp->type == type && tmp->subtype == subtype) return tmp;	2769	if (tmp->type == type && tmp->subtype == subtype)
		2770	return tmp;
2404		2771
2405	return NULL;	2772	return NULL;
2406	}	2773	}
2407		2774
2408	/* If ob has a field named key, return the link from the list,	2775	/* If ob has a field named key, return the link from the list,
2409	* otherwise return NULL.	2776	* otherwise return NULL.
2410	*	2777	*
2411	* key must be a passed in shared string - otherwise, this won't	2778	* key must be a passed in shared string - otherwise, this won't
2412	* do the desired thing.	2779	* do the desired thing.
2413	*/	2780	*/
		2781	key_value *
2414	key_value * get_ob_key_link(const object * ob, const char * key) {	2782	get_ob_key_link (const object *ob, const char *key)
		2783	{
2415	key_value * link;	2784	key_value *
2416		2785	link;
		2786
2417	for (link = ob->key_values; link != NULL; link = link->next) {	2787	for (link = ob->key_values; link != NULL; link = link->next)
		2788	{
2418	if (link->key == key) {	2789	if (link->key == key)
		2790	{
2419	return link;	2791	return link;
2420	}	2792	}
2421	}
2422		2793	}
		2794
2423	return NULL;	2795	return NULL;
2424	}	2796	}
2425		2797
2426	/*	2798	/*
2427	* Returns the value of op has an extra_field for key, or NULL.	2799	* Returns the value of op has an extra_field for key, or NULL.
2428	*	2800	*
2429	* The argument doesn't need to be a shared string.	2801	* The argument doesn't need to be a shared string.
2430	*	2802	*
2431	* The returned string is shared.	2803	* The returned string is shared.
2432	*/	2804	*/
		2805	const char *
2433	const char * get_ob_key_value(const object * op, const char * const key) {	2806	get_ob_key_value (const object *op, const char *const key)
		2807	{
2434	key_value * link;	2808	key_value *
2435	const char * canonical_key;	2809	link;
2436		2810	const char *
		2811	canonical_key;
		2812
2437	canonical_key = shstr::find (key);	2813	canonical_key = shstr::find (key);
2438		2814
2439	if (canonical_key == NULL) {	2815	if (canonical_key == NULL)
		2816	{
2440	/* 1. There being a field named key on any object	2817	/* 1. There being a field named key on any object
2441	* implies there'd be a shared string to find.	2818	* implies there'd be a shared string to find.
2442	* 2. Since there isn't, no object has this field.	2819	* 2. Since there isn't, no object has this field.
2443	* 3. Therefore, *this* object doesn't have this field.	2820	* 3. Therefore, *this* object doesn't have this field.
2444	*/	2821	*/
2445	return NULL;	2822	return NULL;
2446	}	2823	}
2447		2824
2448	/* This is copied from get_ob_key_link() above -	2825	/* This is copied from get_ob_key_link() above -
2449	* only 4 lines, and saves the function call overhead.	2826	* only 4 lines, and saves the function call overhead.
2450	*/	2827	*/
2451	for (link = op->key_values; link != NULL; link = link->next) {	2828	for (link = op->key_values; link != NULL; link = link->next)
		2829	{
2452	if (link->key == canonical_key) {	2830	if (link->key == canonical_key)
		2831	{
2453	return link->value;	2832	return link->value;
2454	}	2833	}
2455	}	2834	}
2456	return NULL;	2835	return NULL;
2457	}	2836	}
2458		2837
2459		2838
2460	/*	2839	/*
2461	* Updates the canonical_key in op to value.	2840	* Updates the canonical_key in op to value.
…		…
2465	* Unless add_key is TRUE, it won't add fields, only change the value of existing	2844	* Unless add_key is TRUE, it won't add fields, only change the value of existing
2466	* keys.	2845	* keys.
2467	*	2846	*
2468	* Returns TRUE on success.	2847	* Returns TRUE on success.
2469	*/	2848	*/
		2849	int
2470	int set_ob_key_value_s(object * op, const shstr &canonical_key, const char * value, int add_key) {	2850	set_ob_key_value_s (object *op, const shstr & canonical_key, const char *value, int add_key)
		2851	{
		2852	key_value *
2471	key_value * field = NULL, *last=NULL;	2853	field = NULL, *last = NULL;
2472		2854
2473	for (field=op->key_values; field != NULL; field=field->next) {	2855	for (field = op->key_values; field != NULL; field = field->next)
		2856	{
2474	if (field->key != canonical_key) {	2857	if (field->key != canonical_key)
		2858	{
2475	last = field;	2859	last = field;
2476	continue;	2860	continue;
2477	}	2861	}
2478		2862
2479	if (value)	2863	if (value)
2480	field->value = value;	2864	field->value = value;
2481	else {	2865	else
		2866	{
2482	/* Basically, if the archetype has this key set,	2867	/* Basically, if the archetype has this key set,
2483	* we need to store the null value so when we save	2868	* we need to store the null value so when we save
2484	* it, we save the empty value so that when we load,	2869	* it, we save the empty value so that when we load,
2485	* we get this value back again.	2870	* we get this value back again.
2486	*/	2871	*/
2487	if (get_ob_key_link (&op->arch->clone, canonical_key))	2872	if (get_ob_key_link (&op->arch->clone, canonical_key))
2488	field->value = 0;	2873	field->value = 0;
		2874	else
		2875	{
		2876	if (last)
		2877	last->next = field->next;
2489	else	2878	else
2490	{
2491	if (last) last->next = field->next;
2492	else op->key_values = field->next;	2879	op->key_values = field->next;
2493		2880
		2881	delete
2494	delete field;	2882	field;
2495	}	2883	}
2496	}	2884	}
2497	return TRUE;	2885	return TRUE;
2498	}	2886	}
2499	/* IF we get here, key doesn't exist */	2887	/* IF we get here, key doesn't exist */
2500		2888
2501	/* No field, we'll have to add it. */	2889	/* No field, we'll have to add it. */
		2890
		2891	if (!add_key)
2502		2892	{
2503	if (!add_key) {
2504	return FALSE;	2893	return FALSE;
2505	}	2894	}
2506	/* There isn't any good reason to store a null	2895	/* There isn't any good reason to store a null
2507	* value in the key/value list. If the archetype has	2896	* value in the key/value list. If the archetype has
2508	* this key, then we should also have it, so shouldn't	2897	* this key, then we should also have it, so shouldn't
2509	* be here. If user wants to store empty strings,	2898	* be here. If user wants to store empty strings,
2510	* should pass in ""	2899	* should pass in ""
2511	*/	2900	*/
2512	if (value == NULL) return TRUE;	2901	if (value == NULL)
2513
2514	field = new key_value;
2515
2516	field->key = canonical_key;
2517	field->value = value;
2518	/* Usual prepend-addition. */
2519	field->next = op->key_values;
2520	op->key_values = field;
2521
2522	return TRUE;	2902	return TRUE;
		2903
		2904	field = new key_value;
		2905
		2906	field->key = canonical_key;
		2907	field->value = value;
		2908	/* Usual prepend-addition. */
		2909	field->next = op->key_values;
		2910	op->key_values = field;
		2911
		2912	return TRUE;
2523	}	2913	}
2524		2914
2525	/*	2915	/*
2526	* Updates the key in op to value.	2916	* Updates the key in op to value.
2527	*	2917	*
…		…
2529	* and not add new ones.	2919	* and not add new ones.
2530	* In general, should be little reason FALSE is ever passed in for add_key	2920	* In general, should be little reason FALSE is ever passed in for add_key
2531	*	2921	*
2532	* Returns TRUE on success.	2922	* Returns TRUE on success.
2533	*/	2923	*/
		2924	int
2534	int set_ob_key_value(object * op, const char *key, const char * value, int add_key)	2925	set_ob_key_value (object *op, const char *key, const char *value, int add_key)
2535	{	2926	{
		2927	shstr
2536	shstr key_ (key);	2928	key_ (key);
		2929
2537	return set_ob_key_value_s (op, key_, value, add_key);	2930	return set_ob_key_value_s (op, key_, value, add_key);
2538	}	2931	}

Diff Legend

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