[ViewVC] Diff of: cvs/deliantra/server/common/object.C

Comparing deliantra/server/common/object.C (file contents):
Revision 1.219 by root, Wed Apr 23 21:09:10 2008 UTC vs.
Revision 1.255 by root, Sun Aug 31 02:01:41 2008 UTC

…		…
139		139
140	/* Returns TRUE if every key_values in wants has a partner with the same value in has. */	140	/* Returns TRUE if every key_values in wants has a partner with the same value in has. */
141	static bool	141	static bool
142	compare_ob_value_lists_one (const object wants, const object has)	142	compare_ob_value_lists_one (const object wants, const object has)
143	{	143	{
144	key_value *wants_field;
145
146	/* n-squared behaviour (see get_ob_key_link()), but I'm hoping both	144	/* n-squared behaviour (see kv_get), but I'm hoping both
147	* objects with lists are rare, and lists stay short. If not, use a	145	* objects with lists are rare, and lists stay short. If not, use a
148	* different structure or at least keep the lists sorted...	146	* different structure or at least keep the lists sorted...
149	*/	147	*/
150		148
151	/* For each field in wants, */	149	/* For each field in wants, */
152	for (wants_field = wants->key_values; wants_field; wants_field = wants_field->next)	150	for (key_value *kv = wants->key_values; kv; kv = kv->next)
153	{	151	if (has->kv_get (kv->key) != kv->value)
154	key_value *has_field;	152	return false;
155
156	/* Look for a field in has with the same key. */
157	has_field = get_ob_key_link (has, wants_field->key);
158
159	if (!has_field)
160	return 0; /* No field with that name. */
161
162	/* Found the matching field. */
163	if (has_field->value != wants_field->value)
164	return 0; /* Values don't match, so this half of the comparison is false. */
165
166	/* If we get here, we found a match. Now for the next field in wants. */
167	}
168		153
169	/* If we get here, every field in wants has a matching field in has. */	154	/* If we get here, every field in wants has a matching field in has. */
170	return 1;	155	return true;
171	}	156	}
172		157
173	/* Returns TRUE if ob1 has the same key_values as ob2. */	158	/* Returns TRUE if ob1 has the same key_values as ob2. */
174	static bool	159	static bool
175	compare_ob_value_lists (const object ob1, const object ob2)	160	compare_ob_value_lists (const object ob1, const object ob2)
176	{	161	{
177	/* However, there may be fields in has which aren't partnered in wants,	162	/* However, there may be fields in has which aren't partnered in wants,
178	* so we need to run the comparison twice. :(	163	* so we need to run the comparison twice. :(
179	*/	164	*/
180	return compare_ob_value_lists_one (ob1, ob2) && compare_ob_value_lists_one (ob2, ob1);	165	return compare_ob_value_lists_one (ob1, ob2)
		166	&& compare_ob_value_lists_one (ob2, ob1);
181	}	167	}
182		168
183	/* Function examines the 2 objects given to it, and returns true if	169	/* Function examines the 2 objects given to it, and returns true if
184	* they can be merged together.	170	* they can be merged together.
185	*	171	*
…		…
200	\|\| ob1->speed != ob2->speed	186	\|\| ob1->speed != ob2->speed
201	\|\| ob1->value != ob2->value	187	\|\| ob1->value != ob2->value
202	\|\| ob1->name != ob2->name)	188	\|\| ob1->name != ob2->name)
203	return 0;	189	return 0;
204		190
205	/* Do not merge objects if nrof would overflow. First part checks	191	/* Do not merge objects if nrof would overflow, assume nrof
206	* for unsigned overflow (2c), second part checks whether the result	192	* is always 0 .. 2*31-1 /
207	* would fit into a 32 bit signed int, which is often used to hold	193	if (ob1->nrof > 0x7fffffff - ob2->nrof)
208	* nrof values.
209	*/
210	if (~ob1->nrof < ob2->nrof \|\| ob1->nrof + ob2->nrof > (1UL << 31))
211	return 0;	194	return 0;
212		195
213	/* If the objects have been identified, set the BEEN_APPLIED flag.	196	/* If the objects have been identified, set the BEEN_APPLIED flag.
214	* This is to the comparison of the flags below will be OK. We	197	* This is to the comparison of the flags below will be OK. We
215	* just can't ignore the been applied or identified flags, as they	198	* just can't ignore the been applied or identified flags, as they
…		…
220	SET_FLAG (ob1, FLAG_BEEN_APPLIED);	203	SET_FLAG (ob1, FLAG_BEEN_APPLIED);
221		204
222	if (QUERY_FLAG (ob2, FLAG_IDENTIFIED))	205	if (QUERY_FLAG (ob2, FLAG_IDENTIFIED))
223	SET_FLAG (ob2, FLAG_BEEN_APPLIED);	206	SET_FLAG (ob2, FLAG_BEEN_APPLIED);
224		207
225	if (ob1->arch->name != ob2->arch->name	208	if (ob1->arch->archname != ob2->arch->archname
226	\|\| ob1->name != ob2->name	209	\|\| ob1->name != ob2->name
227	\|\| ob1->title != ob2->title	210	\|\| ob1->title != ob2->title
228	\|\| ob1->msg != ob2->msg	211	\|\| ob1->msg != ob2->msg
229	\|\| ob1->weight != ob2->weight	212	\|\| ob1->weight != ob2->weight
230	\|\| ob1->attacktype != ob2->attacktype	213	\|\| ob1->attacktype != ob2->attacktype
231	\|\| ob1->magic != ob2->magic	214	\|\| ob1->magic != ob2->magic
232	\|\| ob1->slaying != ob2->slaying	215	\|\| ob1->slaying != ob2->slaying
233	\|\| ob1->skill != ob2->skill	216	\|\| ob1->skill != ob2->skill
234	\|\| ob1->value != ob2->value	217	\|\| ob1->value != ob2->value
235	\|\| ob1->animation_id != ob2->animation_id	218	\|\| ob1->animation_id != ob2->animation_id
		219	\|\| (ob1->face != ob2->face && !ob1->animation_id) // face and animation are dependent on each other
236	\|\| ob1->client_type != ob2->client_type	220	\|\| ob1->client_type != ob2->client_type
237	\|\| ob1->materialname != ob2->materialname	221	\|\| ob1->materialname != ob2->materialname
238	\|\| ob1->lore != ob2->lore	222	\|\| ob1->lore != ob2->lore
239	\|\| ob1->subtype != ob2->subtype	223	\|\| ob1->subtype != ob2->subtype
240	\|\| ob1->move_type != ob2->move_type	224	\|\| ob1->move_type != ob2->move_type
…		…
248	\|\| memcmp (&ob1->stats , &ob2->stats , sizeof (ob1->stats)))	232	\|\| memcmp (&ob1->stats , &ob2->stats , sizeof (ob1->stats)))
249	return 0;	233	return 0;
250		234
251	if ((ob1->flag ^ ob2->flag)	235	if ((ob1->flag ^ ob2->flag)
252	.reset (FLAG_INV_LOCKED)	236	.reset (FLAG_INV_LOCKED)
253	.reset (FLAG_CLIENT_SENT)
254	.reset (FLAG_REMOVED)	237	.reset (FLAG_REMOVED)
255	.any ())	238	.any ())
256	return 0;	239	return 0;
257		240
258	/* This is really a spellbook check - we should in general	241	/* This is really a spellbook check - we should in general
…		…
334		317
335	// find player who can see this object	318	// find player who can see this object
336	object *	319	object *
337	object::visible_to () const	320	object::visible_to () const
338	{	321	{
339	if (!flag [FLAG_REMOVED])	322	if (client_visible () && !flag [FLAG_REMOVED])
340	{	323	{
341	// see if we are in a container of sorts	324	// see if we are in a container of sorts
342	if (env)	325	if (env)
343	{	326	{
344	// the player inventory itself is always visible	327	// the player inventory itself is always visible
…		…
358	else	341	else
359	{	342	{
360	// maybe there is a player standing on the same mapspace	343	// maybe there is a player standing on the same mapspace
361	// this will catch the case where "this" is a player	344	// this will catch the case where "this" is a player
362	if (object *pl = ms ().player ())	345	if (object *pl = ms ().player ())
		346	if ((!pl->container && pl->contr->ns && !pl->contr->ns->update_look)
		347	\|\| this == pl->container)
363	return pl;	348	return pl;
364	}	349	}
365	}	350	}
366		351
367	return 0;	352	return 0;
368	}	353	}
369		354
370	// adjust weight per container type ("of holding")	355	// adjust weight per container type ("of holding")
371	static sint32	356	static sint32
372	weight_adjust (object *op, sint32 weight)	357	weight_adjust_for (object *op, sint32 weight)
373	{	358	{
374	return op->type == CONTAINER	359	return op->type == CONTAINER
375	? lerp (weight, 0, 100, 0, 100 - op->stats.Str)	360	? lerp (weight, 0, 100, 0, 100 - op->stats.Str)
376	: weight;	361	: weight;
377	}	362	}
…		…
383	static void	368	static void
384	adjust_weight (object *op, sint32 weight)	369	adjust_weight (object *op, sint32 weight)
385	{	370	{
386	while (op)	371	while (op)
387	{	372	{
		373	// adjust by actual difference to account for rounding errors
		374	// i.e. (w2 - w1) / f != w2 / f - w1 / f and the latter is correct
388	weight = weight_adjust (op, weight);	375	weight = weight_adjust_for (op, op->carrying)
		376	- weight_adjust_for (op, op->carrying - weight);
389		377
390	if (!weight)	378	if (!weight)
391	return;	379	return;
392		380
393	op->carrying += weight;	381	op->carrying += weight;
…		…
416	op->update_weight ();	404	op->update_weight ();
417		405
418	sum += op->total_weight ();	406	sum += op->total_weight ();
419	}	407	}
420		408
421	sum = weight_adjust (this, sum);	409	sum = weight_adjust_for (this, sum);
422		410
423	if (sum != carrying)	411	if (sum != carrying)
424	{	412	{
425	carrying = sum;	413	carrying = sum;
426		414
…		…
488	*/	476	*/
489	object *	477	object *
490	find_object_name (const char *str)	478	find_object_name (const char *str)
491	{	479	{
492	shstr_cmp str_ (str);	480	shstr_cmp str_ (str);
493	object *op;
494		481
		482	if (str_)
495	for_all_objects (op)	483	for_all_objects (op)
496	if (op->name == str_)	484	if (op->name == str_)
497	break;	485	return op;
498		486
499	return op;	487	return 0;
500	}	488	}
501		489
502	/*	490	/*
503	* Sets the owner and sets the skill and exp pointers to owner's current	491	* Sets the owner and sets the skill and exp pointers to owner's current
504	* skill and experience objects.	492	* skill and experience objects.
…		…
602	}	590	}
603		591
604	op->key_values = 0;	592	op->key_values = 0;
605	}	593	}
606		594
607	object &	595	/*
608	object::operator =(const object &src)	596	* copy_to first frees everything allocated by the dst object,
		597	* and then copies the contents of itself into the second
		598	* object, allocating what needs to be allocated. Basically, any
		599	* data that is malloc'd needs to be re-malloc/copied. Otherwise,
		600	* if the first object is freed, the pointers in the new object
		601	* will point at garbage.
		602	*/
		603	void
		604	object::copy_to (object *dst)
609	{	605	{
610	bool is_freed = flag [FLAG_FREED];	606	dst->remove ();
611	bool is_removed = flag [FLAG_REMOVED];
612
613	(object_copy )this = src;	607	(object_copy )dst = *this;
614
615	flag [FLAG_FREED] = is_freed;
616	flag [FLAG_REMOVED] = is_removed;	608	dst->flag [FLAG_REMOVED] = true;
617		609
618	/* Copy over key_values, if any. */	610	/* Copy over key_values, if any. */
619	if (src.key_values)	611	if (key_values)
620	{	612	{
621	key_value *tail = 0;	613	key_value *tail = 0;
622	key_values = 0;	614	dst->key_values = 0;
623		615
624	for (key_value *i = src.key_values; i; i = i->next)	616	for (key_value *i = key_values; i; i = i->next)
625	{	617	{
626	key_value *new_link = new key_value;	618	key_value *new_link = new key_value;
627		619
628	new_link->next = 0;	620	new_link->next = 0;
629	new_link->key = i->key;	621	new_link->key = i->key;
630	new_link->value = i->value;	622	new_link->value = i->value;
631		623
632	/* Try and be clever here, too. */	624	/* Try and be clever here, too. */
633	if (!key_values)	625	if (!dst->key_values)
634	{	626	{
635	key_values = new_link;	627	dst->key_values = new_link;
636	tail = new_link;	628	tail = new_link;
637	}	629	}
638	else	630	else
639	{	631	{
640	tail->next = new_link;	632	tail->next = new_link;
641	tail = new_link;	633	tail = new_link;
642	}	634	}
643	}	635	}
644	}	636	}
645	}
646
647	/*
648	* copy_to first frees everything allocated by the dst object,
649	* and then copies the contents of itself into the second
650	* object, allocating what needs to be allocated. Basically, any
651	* data that is malloc'd needs to be re-malloc/copied. Otherwise,
652	* if the first object is freed, the pointers in the new object
653	* will point at garbage.
654	*/
655	void
656	object::copy_to (object *dst)
657	{
658	dst = this;
659		637
660	if (speed < 0)	638	if (speed < 0)
661	dst->speed_left -= rndm ();	639	dst->speed_left -= rndm ();
662		640
663	dst->set_speed (dst->speed);	641	dst->set_speed (dst->speed);
…		…
685	object *	663	object *
686	object::clone ()	664	object::clone ()
687	{	665	{
688	object *neu = create ();	666	object *neu = create ();
689	copy_to (neu);	667	copy_to (neu);
		668	neu->map = map; // not copied by copy_to
690	return neu;	669	return neu;
691	}	670	}
692		671
693	/*	672	/*
694	* If an object with the IS_TURNABLE() flag needs to be turned due	673	* If an object with the IS_TURNABLE() flag needs to be turned due
…		…
745	* UP_OBJ_FACE: only the objects face has changed.	724	* UP_OBJ_FACE: only the objects face has changed.
746	*/	725	*/
747	void	726	void
748	update_object (object *op, int action)	727	update_object (object *op, int action)
749	{	728	{
750	if (op == NULL)	729	if (!op)
751	{	730	{
752	/* this should never happen */	731	/* this should never happen */
753	LOG (llevDebug, "update_object() called for NULL object.\n");	732	LOG (llevError \| logBacktrace, "update_object() called for NULL object.\n");
754	return;	733	return;
755	}	734	}
756		735
757	if (op->env)	736	if (!op->is_on_map ())
758	{	737	{
759	/* Animation is currently handled by client, so nothing	738	/* Animation is currently handled by client, so nothing
760	* to do in this case.	739	* to do in this case.
761	*/	740	*/
762	return;	741	return;
763	}	742	}
764
765	/* If the map is saving, don't do anything as everything is
766	* going to get freed anyways.
767	*/
768	if (!op->map \|\| op->map->in_memory == MAP_SAVING)
769	return;
770		743
771	/* make sure the object is within map boundaries */	744	/* make sure the object is within map boundaries */
772	if (op->x < 0 \|\| op->x >= op->map->width \|\| op->y < 0 \|\| op->y >= op->map->height)	745	if (op->x < 0 \|\| op->x >= op->map->width \|\| op->y < 0 \|\| op->y >= op->map->height)
773	{	746	{
774	LOG (llevError, "update_object() called for object out of map!\n");	747	LOG (llevError, "update_object() called for object out of map!\n");
…		…
793	\|\| (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC))	766	\|\| (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC))
794	\|\| (m.move_on \| op->move_on ) != m.move_on	767	\|\| (m.move_on \| op->move_on ) != m.move_on
795	\|\| (m.move_off \| op->move_off ) != m.move_off	768	\|\| (m.move_off \| op->move_off ) != m.move_off
796	\|\| (m.move_slow \| op->move_slow) != m.move_slow	769	\|\| (m.move_slow \| op->move_slow) != m.move_slow
797	/* This isn't perfect, but I don't expect a lot of objects to	770	/* This isn't perfect, but I don't expect a lot of objects to
798	* to have move_allow right now.	771	* have move_allow right now.
799	*/	772	*/
800	\|\| ((m.move_block \| op->move_block) & ~op->move_allow) != m.move_block	773	\|\| ((m.move_block \| op->move_block) & ~op->move_allow) != m.move_block
801	\|\| 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp)	774	\|\| 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp)
802	m.flags_ = 0;	775	m.flags_ = 0;
803	}	776	}
…		…
929	* drop on that space.	902	* drop on that space.
930	*/	903	*/
931	if (!drop_to_ground	904	if (!drop_to_ground
932	\|\| !map	905	\|\| !map
933	\|\| map->in_memory != MAP_ACTIVE	906	\|\| map->in_memory != MAP_ACTIVE
934	\|\| map->nodrop	907	\|\| map->no_drop
935	\|\| ms ().move_block == MOVE_ALL)	908	\|\| ms ().move_block == MOVE_ALL)
936	{	909	{
937	while (inv)	910	while (inv)
938	{
939	inv->destroy_inv (false);
940	inv->destroy ();	911	inv->destroy (true);
941	}
942	}	912	}
943	else	913	else
944	{ /* Put objects in inventory onto this space */	914	{ /* Put objects in inventory onto this space */
945	while (inv)	915	while (inv)
946	{	916	{
…		…
964	object *op = new object;	934	object *op = new object;
965	op->link ();	935	op->link ();
966	return op;	936	return op;
967	}	937	}
968		938
		939	static struct freed_map : maptile
		940	{
		941	freed_map ()
		942	{
		943	path = "<freed objects map>";
		944	name = "/internal/freed_objects_map";
		945	width = 3;
		946	height = 3;
		947	no_drop = 1;
		948	no_reset = 1;
		949
		950	alloc ();
		951	in_memory = MAP_ACTIVE;
		952	}
		953
		954	~freed_map ()
		955	{
		956	destroy ();
		957	}
		958	} freed_map; // freed objects are moved here to avoid crashes
		959
969	void	960	void
970	object::do_destroy ()	961	object::do_destroy ()
971	{	962	{
972	if (flag [FLAG_IS_LINKED])	963	if (flag [FLAG_IS_LINKED])
973	remove_button_link (this);	964	remove_button_link (this);
…		…
983	unlink ();	974	unlink ();
984		975
985	flag [FLAG_FREED] = 1;	976	flag [FLAG_FREED] = 1;
986		977
987	// hack to ensure that freed objects still have a valid map	978	// hack to ensure that freed objects still have a valid map
988	{
989	static maptile *freed_map; // freed objects are moved here to avoid crashes
990
991	if (!freed_map)
992	{
993	freed_map = new maptile;
994
995	freed_map->path = "<freed objects map>";
996	freed_map->name = "/internal/freed_objects_map";
997	freed_map->width = 3;
998	freed_map->height = 3;
999	freed_map->nodrop = 1;
1000
1001	freed_map->alloc ();
1002	freed_map->in_memory = MAP_ACTIVE;
1003	}
1004
1005	map = freed_map;	979	map = &freed_map;
1006	x = 1;	980	x = 1;
1007	y = 1;	981	y = 1;
1008	}
1009		982
1010	if (more)	983	if (more)
1011	{	984	{
1012	more->destroy ();	985	more->destroy ();
1013	more = 0;	986	more = 0;
…		…
1030		1003
1031	if (!is_head () && !head->destroyed ())	1004	if (!is_head () && !head->destroyed ())
1032	{	1005	{
1033	LOG (llevError \| logBacktrace, "tried to destroy the tail of an object");	1006	LOG (llevError \| logBacktrace, "tried to destroy the tail of an object");
1034	head->destroy (destroy_inventory);	1007	head->destroy (destroy_inventory);
		1008	return;
1035	}	1009	}
1036		1010
1037	destroy_inv (!destroy_inventory);	1011	destroy_inv (!destroy_inventory);
1038		1012
1039	if (is_head ())	1013	if (is_head ())
…		…
1061	if (flag [FLAG_REMOVED])	1035	if (flag [FLAG_REMOVED])
1062	return;	1036	return;
1063		1037
1064	INVOKE_OBJECT (REMOVE, this);	1038	INVOKE_OBJECT (REMOVE, this);
1065		1039
1066	if (object *pl = visible_to ())
1067	esrv_del_item (pl->contr, count);
1068
1069	flag [FLAG_REMOVED] = true;	1040	flag [FLAG_REMOVED] = true;
1070		1041
1071	if (more)	1042	if (more)
1072	more->remove ();	1043	more->remove ();
1073		1044
…		…
1075	* In this case, the object to be removed is in someones	1046	* In this case, the object to be removed is in someones
1076	* inventory.	1047	* inventory.
1077	*/	1048	*/
1078	if (env)	1049	if (env)
1079	{	1050	{
		1051	flag [FLAG_REMOVED] = false; // hack around the issue of visible_to checking flag_removed
		1052	if (object *pl = visible_to ())
		1053	esrv_del_item (pl->contr, count);
		1054	flag [FLAG_REMOVED] = true; // hack around the issue of visible_to checking flag_removed
		1055
1080	adjust_weight (env, -total_weight ());	1056	adjust_weight (env, -total_weight ());
1081
1082	*(above ? &above->below : &env->inv) = below;
1083
1084	if (below)
1085	below->above = above;
1086		1057
1087	/* we set up values so that it could be inserted into	1058	/* we set up values so that it could be inserted into
1088	* the map, but we don't actually do that - it is up	1059	* the map, but we don't actually do that - it is up
1089	* to the caller to decide what we want to do.	1060	* to the caller to decide what we want to do.
1090	*/	1061	*/
1091	map = env->map;	1062	map = env->map;
1092	x = env->x;	1063	x = env->x;
1093	y = env->y;	1064	y = env->y;
		1065
		1066	// make sure cmov optimisation is applicable
		1067	*(above ? &above->below : &env->inv) = below;
		1068	*(below ? &below->above : &above ) = above; // &above is just a dummy
		1069
1094	above = 0;	1070	above = 0;
1095	below = 0;	1071	below = 0;
1096	env = 0;	1072	env = 0;
1097		1073
1098	/* NO_FIX_PLAYER is set when a great many changes are being	1074	/* NO_FIX_PLAYER is set when a great many changes are being
…		…
1102	if ((otmp = in_player ()) && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))	1078	if ((otmp = in_player ()) && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
1103	otmp->update_stats ();	1079	otmp->update_stats ();
1104	}	1080	}
1105	else if (map)	1081	else if (map)
1106	{	1082	{
1107	if (type == PLAYER)
1108	{
1109	// leaving a spot always closes any open container on the ground
1110	if (container && !container->env)
1111	// this causes spurious floorbox updates, but it ensures
1112	// that the CLOSE event is being sent.
1113	close_container ();
1114
1115	--map->players;
1116	map->touch ();
1117	}
1118
1119	map->dirty = true;	1083	map->dirty = true;
1120	mapspace &ms = this->ms ();	1084	mapspace &ms = this->ms ();
1121		1085
		1086	if (object *pl = ms.player ())
		1087	{
		1088	if (type == PLAYER) // this == pl(!)
		1089	{
		1090	// leaving a spot always closes any open container on the ground
		1091	if (container && !container->env)
		1092	// this causes spurious floorbox updates, but it ensures
		1093	// that the CLOSE event is being sent.
		1094	close_container ();
		1095
		1096	--map->players;
		1097	map->touch ();
		1098	}
		1099	else if (pl->container == this)
		1100	{
		1101	// removing a container should close it
		1102	close_container ();
		1103	}
		1104
		1105	esrv_del_item (pl->contr, count);
		1106	}
		1107
1122	/* link the object above us */	1108	/* link the object above us */
1123	if (above)	1109	// re-link, make sure compiler can easily use cmove
1124	above->below = below;	1110	*(above ? &above->below : &ms.top) = below;
1125	else	1111	*(below ? &below->above : &ms.bot) = above;
1126	ms.top = below; /* we were top, set new top */
1127
1128	/* Relink the object below us, if there is one */
1129	if (below)
1130	below->above = above;
1131	else
1132	{
1133	/* Nothing below, which means we need to relink map object for this space
1134	* use translated coordinates in case some oddness with map tiling is
1135	* evident
1136	*/
1137	if (GET_MAP_OB (map, x, y) != this)
1138	LOG (llevError, "remove_ob: GET_MAP_OB does not return object to be removed even though it appears to be on the bottom? %s\n", debug_desc ());
1139
1140	ms.bot = above; /* goes on above it. */
1141	}
1142		1112
1143	above = 0;	1113	above = 0;
1144	below = 0;	1114	below = 0;
		1115
		1116	ms.flags_ = 0;
1145		1117
1146	if (map->in_memory == MAP_SAVING)	1118	if (map->in_memory == MAP_SAVING)
1147	return;	1119	return;
1148		1120
1149	int check_walk_off = !flag [FLAG_NO_APPLY];	1121	int check_walk_off = !flag [FLAG_NO_APPLY];
…		…
1156	* appropriately.	1128	* appropriately.
1157	*/	1129	*/
1158	pl->close_container ();	1130	pl->close_container ();
1159		1131
1160	//TODO: the floorbox prev/next might need updating	1132	//TODO: the floorbox prev/next might need updating
1161	esrv_del_item (pl->contr, count);	1133	//esrv_del_item (pl->contr, count);
		1134	//TODO: update floorbox to preserve ordering
		1135	if (pl->contr->ns)
		1136	pl->contr->ns->floorbox_update ();
1162	}	1137	}
1163		1138
1164	for (tmp = ms.bot; tmp; tmp = tmp->above)	1139	for (tmp = ms.bot; tmp; tmp = tmp->above)
1165	{	1140	{
1166	/* No point updating the players look faces if he is the object	1141	/* No point updating the players look faces if he is the object
…		…
1179	}	1154	}
1180		1155
1181	last = tmp;	1156	last = tmp;
1182	}	1157	}
1183		1158
1184	/* last == NULL if there are no objects on this space */
1185	//TODO: this makes little sense, why only update the topmost object?
1186	if (!last)
1187	map->at (x, y).flags_ = 0;
1188	else
1189	update_object (last, UP_OBJ_REMOVE);
1190
1191	if (flag [FLAG_BLOCKSVIEW] \|\| glow_radius)	1159	if (flag [FLAG_BLOCKSVIEW] \|\| glow_radius)
1192	update_all_los (map, x, y);	1160	update_all_los (map, x, y);
1193	}	1161	}
1194	}	1162	}
1195		1163
…		…
1258	* job preparing multi-part monsters.	1226	* job preparing multi-part monsters.
1259	*/	1227	*/
1260	object *	1228	object *
1261	insert_ob_in_map_at (object op, maptile m, object *originator, int flag, int x, int y)	1229	insert_ob_in_map_at (object op, maptile m, object *originator, int flag, int x, int y)
1262	{	1230	{
		1231	op->remove ();
		1232
1263	for (object *tmp = op->head_ (); tmp; tmp = tmp->more)	1233	for (object *tmp = op->head_ (); tmp; tmp = tmp->more)
1264	{	1234	{
1265	tmp->x = x + tmp->arch->x;	1235	tmp->x = x + tmp->arch->x;
1266	tmp->y = y + tmp->arch->y;	1236	tmp->y = y + tmp->arch->y;
1267	}	1237	}
…		…
1290	* just 'op' otherwise	1260	* just 'op' otherwise
1291	*/	1261	*/
1292	object *	1262	object *
1293	insert_ob_in_map (object op, maptile m, object *originator, int flag)	1263	insert_ob_in_map (object op, maptile m, object *originator, int flag)
1294	{	1264	{
1295	assert (!op->flag [FLAG_FREED]);	1265	if (op->is_on_map ())
		1266	{
		1267	LOG (llevError, "insert_ob_in_map called for object already on map");
		1268	abort ();
		1269	}
1296		1270
		1271	if (op->env)
		1272	{
		1273	LOG (llevError, "insert_ob_in_map called for object in an inventory (proceeding)");
1297	op->remove ();	1274	op->remove ();
		1275	}
		1276
		1277	if (op->face && !face_info (op->face))//D TODO: remove soon
		1278	{//D
		1279	LOG (llevError \| logBacktrace, "op->face out of bounds: %s", op->debug_desc ());//D
		1280	op->face = 1;//D
		1281	}//D
1298		1282
1299	/* Ideally, the caller figures this out. However, it complicates a lot	1283	/* Ideally, the caller figures this out. However, it complicates a lot
1300	* of areas of callers (eg, anything that uses find_free_spot would now	1284	* of areas of callers (eg, anything that uses find_free_spot would now
1301	* need extra work	1285	* need extra work
1302	*/	1286	*/
1303	if (!xy_normalise (m, op->x, op->y))	1287	if (!xy_normalise (m, op->x, op->y))
1304	{	1288	{
1305	op->destroy (1);	1289	op->head_ ()->destroy (1);// remove head_ once all tail object destroyers found
1306	return 0;	1290	return 0;
1307	}	1291	}
1308		1292
1309	if (object *more = op->more)	1293	if (object *more = op->more)
1310	if (!insert_ob_in_map (more, m, originator, flag))	1294	if (!insert_ob_in_map (more, m, originator, flag))
…		…
1319	*/	1303	*/
1320	if (op->nrof && !(flag & INS_NO_MERGE))	1304	if (op->nrof && !(flag & INS_NO_MERGE))
1321	for (object *tmp = ms.bot; tmp; tmp = tmp->above)	1305	for (object *tmp = ms.bot; tmp; tmp = tmp->above)
1322	if (object::can_merge (op, tmp))	1306	if (object::can_merge (op, tmp))
1323	{	1307	{
1324	// TODO: we atcually want to update tmp, not op,	1308	// TODO: we actually want to update tmp, not op,
1325	// but some caller surely breaks when we return tmp	1309	// but some caller surely breaks when we return tmp
1326	// from here :/	1310	// from here :/
1327	op->nrof += tmp->nrof;	1311	op->nrof += tmp->nrof;
1328	tmp->destroy (1);	1312	tmp->destroy (1);
1329	}	1313	}
…		…
1340	{	1324	{
1341	LOG (llevError, "insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n");	1325	LOG (llevError, "insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n");
1342	abort ();	1326	abort ();
1343	}	1327	}
1344		1328
		1329	if (!originator->is_on_map ())
		1330	LOG (llevDebug \| logBacktrace, "insert_ob_in_map(%s) called with INS_BELOW_ORIGINATOR when originator '%s' not on map",
		1331	op->debug_desc (), originator->debug_desc ());
		1332
1345	op->above = originator;	1333	op->above = originator;
1346	op->below = originator->below;	1334	op->below = originator->below;
1347
1348	if (op->below)
1349	op->below->above = op;
1350	else
1351	ms.bot = op;
1352
1353	/* since below originator, no need to update top */
1354	originator->below = op;	1335	originator->below = op;
		1336
		1337	*(op->below ? &op->below->above : &ms.bot) = op;
1355	}	1338	}
1356	else	1339	else
1357	{	1340	{
1358	object top, floor = NULL;	1341	object *floor = 0;
1359		1342	object *top = ms.top;
1360	top = ms.bot;
1361		1343
1362	/* If there are other objects, then */	1344	/* If there are other objects, then */
1363	if (top)	1345	if (top)
1364	{	1346	{
1365	object *last = 0;
1366
1367	/*	1347	/*
1368	* If there are multiple objects on this space, we do some trickier handling.	1348	* If there are multiple objects on this space, we do some trickier handling.
1369	* We've already dealt with merging if appropriate.	1349	* We've already dealt with merging if appropriate.
1370	* Generally, we want to put the new object on top. But if	1350	* Generally, we want to put the new object on top. But if
1371	* flag contains INS_ABOVE_FLOOR_ONLY, once we find the last	1351	* flag contains INS_ABOVE_FLOOR_ONLY, once we find the last
…		…
1374	* once we get to them. This reduces the need to traverse over all of	1354	* once we get to them. This reduces the need to traverse over all of
1375	* them when adding another one - this saves quite a bit of cpu time	1355	* them when adding another one - this saves quite a bit of cpu time
1376	* when lots of spells are cast in one area. Currently, it is presumed	1356	* when lots of spells are cast in one area. Currently, it is presumed
1377	* that flying non pickable objects are spell objects.	1357	* that flying non pickable objects are spell objects.
1378	*/	1358	*/
1379	for (top = ms.bot; top; top = top->above)	1359	for (object *tmp = ms.bot; tmp; tmp = tmp->above)
1380	{	1360	{
1381	if (QUERY_FLAG (top, FLAG_IS_FLOOR) \|\| QUERY_FLAG (top, FLAG_OVERLAY_FLOOR))	1361	if (QUERY_FLAG (tmp, FLAG_IS_FLOOR) \|\| QUERY_FLAG (tmp, FLAG_OVERLAY_FLOOR))
1382	floor = top;	1362	floor = tmp;
1383		1363
1384	if (QUERY_FLAG (top, FLAG_NO_PICK) && (top->move_type & (MOVE_FLY_LOW \| MOVE_FLY_HIGH)) && !QUERY_FLAG (top, FLAG_IS_FLOOR))	1364	if (QUERY_FLAG (tmp, FLAG_NO_PICK) && (tmp->move_type & (MOVE_FLY_LOW \| MOVE_FLY_HIGH)) && !QUERY_FLAG (tmp, FLAG_IS_FLOOR))
1385	{	1365	{
1386	/* We insert above top, so we want this object below this */	1366	/* We insert above top, so we want this object below this */
1387	top = top->below;	1367	top = tmp->below;
1388	break;	1368	break;
1389	}	1369	}
1390		1370
1391	last = top;	1371	top = tmp;
1392	}	1372	}
1393
1394	/* Don't want top to be NULL, so set it to the last valid object */
1395	top = last;
1396		1373
1397	/* We let update_position deal with figuring out what the space	1374	/* We let update_position deal with figuring out what the space
1398	* looks like instead of lots of conditions here.	1375	* looks like instead of lots of conditions here.
1399	* makes things faster, and effectively the same result.	1376	* makes things faster, and effectively the same result.
1400	*/	1377	*/
…		…
1407	*/	1384	*/
1408	if (!(flag & INS_ON_TOP)	1385	if (!(flag & INS_ON_TOP)
1409	&& ms.flags () & P_BLOCKSVIEW	1386	&& ms.flags () & P_BLOCKSVIEW
1410	&& (op->face && !faces [op->face].visibility))	1387	&& (op->face && !faces [op->face].visibility))
1411	{	1388	{
		1389	object *last;
		1390
1412	for (last = top; last != floor; last = last->below)	1391	for (last = top; last != floor; last = last->below)
1413	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))	1392	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))
1414	break;	1393	break;
1415		1394
1416	/* Check to see if we found the object that blocks view,	1395	/* Check to see if we found the object that blocks view,
…		…
1424	} /* If objects on this space */	1403	} /* If objects on this space */
1425		1404
1426	if (flag & INS_ABOVE_FLOOR_ONLY)	1405	if (flag & INS_ABOVE_FLOOR_ONLY)
1427	top = floor;	1406	top = floor;
1428		1407
1429	/* Top is the object that our object (op) is going to get inserted above.	1408	// insert object above top, or bottom-most if top = 0
1430	*/
1431
1432	/* First object on this space */
1433	if (!top)	1409	if (!top)
1434	{	1410	{
		1411	op->below = 0;
1435	op->above = ms.bot;	1412	op->above = ms.bot;
1436
1437	if (op->above)
1438	op->above->below = op;
1439
1440	op->below = 0;
1441	ms.bot = op;	1413	ms.bot = op;
		1414
		1415	*(op->above ? &op->above->below : &ms.top) = op;
1442	}	1416	}
1443	else	1417	else
1444	{ /* get inserted into the stack above top */	1418	{
1445	op->above = top->above;	1419	op->above = top->above;
1446
1447	if (op->above)
1448	op->above->below = op;	1420	top->above = op;
1449		1421
1450	op->below = top;	1422	op->below = top;
1451	top->above = op;	1423	*(op->above ? &op->above->below : &ms.top) = op;
1452	}	1424	}
1453		1425	}
1454	if (!op->above)
1455	ms.top = op;
1456	} /* else not INS_BELOW_ORIGINATOR */
1457		1426
1458	if (op->type == PLAYER)	1427	if (op->type == PLAYER)
1459	{	1428	{
1460	op->contr->do_los = 1;	1429	op->contr->do_los = 1;
1461	++op->map->players;	1430	++op->map->players;
…		…
1464		1433
1465	op->map->dirty = true;	1434	op->map->dirty = true;
1466		1435
1467	if (object *pl = ms.player ())	1436	if (object *pl = ms.player ())
1468	//TODO: the floorbox prev/next might need updating	1437	//TODO: the floorbox prev/next might need updating
1469	esrv_send_item (pl, op);	1438	//esrv_send_item (pl, op);
		1439	//TODO: update floorbox to preserve ordering
		1440	if (pl->contr->ns)
		1441	pl->contr->ns->floorbox_update ();
1470		1442
1471	/* If this object glows, it may affect lighting conditions that are	1443	/* If this object glows, it may affect lighting conditions that are
1472	* visible to others on this map. But update_all_los is really	1444	* visible to others on this map. But update_all_los is really
1473	* an inefficient way to do this, as it means los for all players	1445	* an inefficient way to do this, as it means los for all players
1474	* on the map will get recalculated. The players could very well	1446	* on the map will get recalculated. The players could very well
…		…
1554	if (!nr)	1526	if (!nr)
1555	return true;	1527	return true;
1556		1528
1557	nr = min (nr, nrof);	1529	nr = min (nr, nrof);
1558		1530
		1531	if (nrof > nr)
		1532	{
1559	nrof -= nr;	1533	nrof -= nr;
1560
1561	if (nrof)
1562	{
1563	adjust_weight (env, -weight * nr); // carrying == 0	1534	adjust_weight (env, -weight * max (1, nr)); // carrying == 0
1564		1535
1565	if (object *pl = visible_to ())	1536	if (object *pl = visible_to ())
1566	esrv_update_item (UPD_NROF, pl, this);	1537	esrv_update_item (UPD_NROF, pl, this);
1567		1538
1568	return true;	1539	return true;
1569	}	1540	}
1570	else	1541	else
1571	{	1542	{
1572	destroy (1);	1543	destroy ();
1573	return false;	1544	return false;
1574	}	1545	}
1575	}	1546	}
1576		1547
1577	/*	1548	/*
…		…
1594	}	1565	}
1595	else	1566	else
1596	{	1567	{
1597	decrease (nr);	1568	decrease (nr);
1598		1569
1599	object *op = object_create_clone (this);	1570	object *op = deep_clone ();
1600	op->nrof = nr;	1571	op->nrof = nr;
1601	return op;	1572	return op;
1602	}	1573	}
1603	}	1574	}
1604		1575
…		…
1825	LOG (llevError, "Present_arch called outside map.\n");	1796	LOG (llevError, "Present_arch called outside map.\n");
1826	return NULL;	1797	return NULL;
1827	}	1798	}
1828		1799
1829	for (object *tmp = m->at (x, y).bot; tmp; tmp = tmp->above)	1800	for (object *tmp = m->at (x, y).bot; tmp; tmp = tmp->above)
1830	if (tmp->arch == at)	1801	if (tmp->arch->archname == at->archname)
1831	return tmp;	1802	return tmp;
1832		1803
1833	return NULL;	1804	return NULL;
1834	}	1805	}
1835		1806
…		…
1899	* The first matching object is returned, or NULL if none.	1870	* The first matching object is returned, or NULL if none.
1900	*/	1871	*/
1901	object *	1872	object *
1902	present_arch_in_ob (const archetype at, const object op)	1873	present_arch_in_ob (const archetype at, const object op)
1903	{	1874	{
1904	for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below)	1875	for (object *tmp = op->inv; tmp; tmp = tmp->below)
1905	if (tmp->arch == at)	1876	if (tmp->arch->archname == at->archname)
1906	return tmp;	1877	return tmp;
1907		1878
1908	return NULL;	1879	return NULL;
1909	}	1880	}
1910		1881
…		…
2319		2290
2320	/*	2291	/*
2321	* create clone from object to another	2292	* create clone from object to another
2322	*/	2293	*/
2323	object *	2294	object *
2324	object_create_clone (object *asrc)	2295	object::deep_clone ()
2325	{	2296	{
2326	object dst = 0, tmp, src, prev, *item;	2297	assert (("deep_clone called on non-head object", is_head ()));
2327		2298
2328	if (!asrc)	2299	object *dst = clone ();
2329	return 0;
2330		2300
2331	src = asrc->head_ ();	2301	object *prev = dst;
2332
2333	prev = 0;
2334	for (object *part = src; part; part = part->more)	2302	for (object *part = this->more; part; part = part->more)
2335	{	2303	{
2336	tmp = part->clone ();	2304	object *tmp = part->clone ();
2337	tmp->x -= src->x;
2338	tmp->y -= src->y;
2339
2340	if (!part->head)
2341	{
2342	dst = tmp;
2343	tmp->head = 0;
2344	}
2345	else
2346	tmp->head = dst;	2305	tmp->head = dst;
2347
2348	tmp->more = 0;
2349
2350	if (prev)
2351	prev->more = tmp;	2306	prev->more = tmp;
2352
2353	prev = tmp;	2307	prev = tmp;
2354	}	2308	}
2355		2309
2356	for (item = src->inv; item; item = item->below)	2310	for (object *item = inv; item; item = item->below)
2357	insert_ob_in_ob (object_create_clone (item), dst);	2311	insert_ob_in_ob (item->deep_clone (), dst);
2358		2312
2359	return dst;	2313	return dst;
2360	}	2314	}
2361		2315
2362	/* This returns the first object in who's inventory that	2316	/* This returns the first object in who's inventory that
…		…
2371	return tmp;	2325	return tmp;
2372		2326
2373	return 0;	2327	return 0;
2374	}	2328	}
2375		2329
2376	/* If ob has a field named key, return the link from the list,	2330	const shstr &
2377	* otherwise return NULL.	2331	object::kv_get (const shstr &key) const
2378	*
2379	* key must be a passed in shared string - otherwise, this won't
2380	* do the desired thing.
2381	*/
2382	key_value *
2383	get_ob_key_link (const object ob, const char key)
2384	{	2332	{
2385	for (key_value *link = ob->key_values; link; link = link->next)	2333	for (key_value *kv = key_values; kv; kv = kv->next)
2386	if (link->key == key)	2334	if (kv->key == key)
2387	return link;
2388
2389	return 0;
2390	}
2391
2392	/*
2393	* Returns the value of op has an extra_field for key, or NULL.
2394	*
2395	* The argument doesn't need to be a shared string.
2396	*
2397	* The returned string is shared.
2398	*/
2399	const char *
2400	get_ob_key_value (const object op, const char const key)
2401	{
2402	key_value *link;
2403	shstr_cmp canonical_key (key);
2404
2405	if (!canonical_key)
2406	{
2407	/* 1. There being a field named key on any object
2408	* implies there'd be a shared string to find.
2409	* 2. Since there isn't, no object has this field.
2410	* 3. Therefore, this object doesn't have this field.
2411	*/
2412	return 0;
2413	}
2414
2415	/* This is copied from get_ob_key_link() above -
2416	* only 4 lines, and saves the function call overhead.
2417	*/
2418	for (link = op->key_values; link; link = link->next)
2419	if (link->key == canonical_key)
2420	return link->value;	2335	return kv->value;
2421		2336
2422	return 0;	2337	return shstr_null;
2423	}	2338	}
2424		2339
2425	/*	2340	void
2426	* Updates the canonical_key in op to value.	2341	object::kv_set (const shstr &key, const shstr &value)
2427	*
2428	* canonical_key is a shared string (value doesn't have to be).
2429	*
2430	* Unless add_key is TRUE, it won't add fields, only change the value of existing
2431	* keys.
2432	*
2433	* Returns TRUE on success.
2434	*/
2435	int
2436	set_ob_key_value_s (object op, const shstr & canonical_key, const char value, int add_key)
2437	{	2342	{
2438	key_value field = NULL, last = NULL;	2343	for (key_value *kv = key_values; kv; kv = kv->next)
2439		2344	if (kv->key == key)
2440	for (field = op->key_values; field != NULL; field = field->next)
2441	{
2442	if (field->key != canonical_key)
2443	{	2345	{
2444	last = field;	2346	kv->value = value;
2445	continue;	2347	return;
2446	}	2348	}
2447		2349
2448	if (value)	2350	key_value *kv = new key_value;
2449	field->value = value;	2351
2450	else	2352	kv->next = key_values;
		2353	kv->key = key;
		2354	kv->value = value;
		2355
		2356	key_values = kv;
		2357	}
		2358
		2359	void
		2360	object::kv_del (const shstr &key)
		2361	{
		2362	for (key_value *kvp = &key_values; kvp; kvp = &(*kvp)->next)
		2363	if ((*kvp)->key == key)
2451	{	2364	{
2452	/* Basically, if the archetype has this key set,	2365	key_value kv = kvp;
2453	* we need to store the null value so when we save	2366	kvp = (kvp)->next;
2454	* it, we save the empty value so that when we load,	2367	delete kv;
2455	* we get this value back again.	2368	return;
2456	*/
2457	if (get_ob_key_link (op->arch, canonical_key))
2458	field->value = 0;
2459	else
2460	{
2461	if (last)
2462	last->next = field->next;
2463	else
2464	op->key_values = field->next;
2465
2466	delete field;
2467	}
2468	}	2369	}
2469	return TRUE;
2470	}
2471	/* IF we get here, key doesn't exist */
2472
2473	/* No field, we'll have to add it. */
2474
2475	if (!add_key)
2476	return FALSE;
2477
2478	/* There isn't any good reason to store a null
2479	* value in the key/value list. If the archetype has
2480	* this key, then we should also have it, so shouldn't
2481	* be here. If user wants to store empty strings,
2482	* should pass in ""
2483	*/
2484	if (value == NULL)
2485	return TRUE;
2486
2487	field = new key_value;
2488
2489	field->key = canonical_key;
2490	field->value = value;
2491	/* Usual prepend-addition. */
2492	field->next = op->key_values;
2493	op->key_values = field;
2494
2495	return TRUE;
2496	}
2497
2498	/*
2499	* Updates the key in op to value.
2500	*
2501	* If add_key is FALSE, this will only update existing keys,
2502	* and not add new ones.
2503	* In general, should be little reason FALSE is ever passed in for add_key
2504	*
2505	* Returns TRUE on success.
2506	*/
2507	int
2508	set_ob_key_value (object op, const char key, const char *value, int add_key)
2509	{
2510	shstr key_ (key);
2511
2512	return set_ob_key_value_s (op, key_, value, add_key);
2513	}	2370	}
2514		2371
2515	object::depth_iterator::depth_iterator (object *container)	2372	object::depth_iterator::depth_iterator (object *container)
2516	: iterator_base (container)	2373	: iterator_base (container)
2517	{	2374	{
…		…
2567	{	2424	{
2568	char flagdesc[512];	2425	char flagdesc[512];
2569	char info2[256 * 4];	2426	char info2[256 * 4];
2570	char *p = info;	2427	char *p = info;
2571		2428
2572	p += snprintf (p, 512, "{cnt:%d,uuid:%s,name:\"%s\"%s%s,flags:[%s],type:%d}",	2429	p += snprintf (p, 512, "{cnt:%d,uuid:%s,name:\"%s\"%s%s%s,flags:[%s],type:%d}",
2573	count,	2430	count,
2574	uuid.c_str (),	2431	uuid.c_str (),
2575	&name,	2432	&name,
2576	title ? "\",title:\"" : "",	2433	title ? ",title:\"" : "",
2577	title ? (const char *)title : "",	2434	title ? (const char *)title : "",
		2435	title ? "\"" : "",
2578	flag_desc (flagdesc, 512), type);	2436	flag_desc (flagdesc, 512), type);
2579		2437
2580	if (!flag[FLAG_REMOVED] && env)	2438	if (!flag[FLAG_REMOVED] && env)
2581	p += snprintf (p, 256, "(in %s)", env->debug_desc (info2));	2439	p += snprintf (p, 256, "(in %s)", env->debug_desc (info2));
2582		2440
…		…
2615	object::open_container (object *new_container)	2473	object::open_container (object *new_container)
2616	{	2474	{
2617	if (container == new_container)	2475	if (container == new_container)
2618	return;	2476	return;
2619		2477
2620	if (object *old_container = container)	2478	object *old_container = container;
		2479
		2480	if (old_container)
2621	{	2481	{
2622	if (INVOKE_OBJECT (CLOSE, old_container, ARG_OBJECT (this)))	2482	if (INVOKE_OBJECT (CLOSE, old_container, ARG_OBJECT (this)))
2623	return;	2483	return;
2624		2484
2625	#if 0	2485	#if 0
…		…
2627	if (object *closer = old_container->inv)	2487	if (object *closer = old_container->inv)
2628	if (closer->type == CLOSE_CON)	2488	if (closer->type == CLOSE_CON)
2629	closer->destroy ();	2489	closer->destroy ();
2630	#endif	2490	#endif
2631		2491
		2492	// make sure the container is available
		2493	esrv_send_item (this, old_container);
		2494
2632	old_container->flag [FLAG_APPLIED] = false;	2495	old_container->flag [FLAG_APPLIED] = false;
2633	container = 0;	2496	container = 0;
2634		2497
		2498	// client needs item update to make it work, client bug requires this to be separate
2635	esrv_update_item (UPD_FLAGS, this, old_container);	2499	esrv_update_item (UPD_FLAGS, this, old_container);
		2500
2636	new_draw_info_format (NDI_UNIQUE, 0, this, "You close %s.", query_name (old_container));	2501	new_draw_info_format (NDI_UNIQUE, 0, this, "You close %s.", query_name (old_container));
2637	play_sound (sound_find ("chest_close"));	2502	play_sound (sound_find ("chest_close"));
2638	}	2503	}
2639		2504
2640	if (new_container)	2505	if (new_container)
…		…
2653	}	2518	}
2654	#endif	2519	#endif
2655		2520
2656	new_draw_info_format (NDI_UNIQUE, 0, this, "You open %s.", query_name (new_container));	2521	new_draw_info_format (NDI_UNIQUE, 0, this, "You open %s.", query_name (new_container));
2657		2522
		2523	// make sure the container is available, client bug requires this to be separate
		2524	esrv_send_item (this, new_container);
		2525
2658	new_container->flag [FLAG_APPLIED] = true;	2526	new_container->flag [FLAG_APPLIED] = true;
2659	container = new_container;	2527	container = new_container;
2660		2528
		2529	// client needs flag change
2661	esrv_update_item (UPD_FLAGS, this, new_container);	2530	esrv_update_item (UPD_FLAGS, this, new_container);
2662	esrv_send_inventory (this, new_container);	2531	esrv_send_inventory (this, new_container);
2663	play_sound (sound_find ("chest_open"));	2532	play_sound (sound_find ("chest_open"));
2664	}	2533	}
		2534	// else if (!old_container->env && contr && contr->ns)
		2535	// contr->ns->floorbox_reset ();
2665	}	2536	}
2666		2537
2667	object *	2538	object *
2668	object::force_find (const shstr name)	2539	object::force_find (const shstr name)
2669	{	2540	{

Diff Legend

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

Comparing deliantra/server/common/object.C (file contents): Revision 1.219 by root, Wed Apr 23 21:09:10 2008 UTC vs. Revision 1.255 by root, Sun Aug 31 02:01:41 2008 UTC

Diff Legend

Comparing deliantra/server/common/object.C (file contents):
Revision 1.219 by root, Wed Apr 23 21:09:10 2008 UTC vs.
Revision 1.255 by root, Sun Aug 31 02:01:41 2008 UTC