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

Comparing deliantra/server/common/object.C (file contents):
Revision 1.217 by root, Wed Apr 23 07:13:23 2008 UTC vs.
Revision 1.254 by root, Sun Aug 31 01:17:11 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
…		…
334		318
335	// find player who can see this object	319	// find player who can see this object
336	object *	320	object *
337	object::visible_to () const	321	object::visible_to () const
338	{	322	{
339	if (!flag [FLAG_REMOVED])	323	if (client_visible () && !flag [FLAG_REMOVED])
340	{	324	{
341	// see if we are in a container of sorts	325	// see if we are in a container of sorts
342	if (env)	326	if (env)
343	{	327	{
344	// the player inventory itself is always visible	328	// the player inventory itself is always visible
…		…
358	else	342	else
359	{	343	{
360	// maybe there is a player standing on the same mapspace	344	// maybe there is a player standing on the same mapspace
361	// this will catch the case where "this" is a player	345	// this will catch the case where "this" is a player
362	if (object *pl = ms ().player ())	346	if (object *pl = ms ().player ())
		347	if ((!pl->container && pl->contr->ns && !pl->contr->ns->update_look)
		348	\|\| this == pl->container)
363	return pl;	349	return pl;
364	}	350	}
365	}	351	}
366		352
367	return 0;	353	return 0;
368	}	354	}
369		355
370	// adjust weight per container type ("of holding")	356	// adjust weight per container type ("of holding")
371	static sint32	357	static sint32
372	weight_adjust (object *op, sint32 weight)	358	weight_adjust_for (object *op, sint32 weight)
373	{	359	{
374	return op->type == CONTAINER	360	return op->type == CONTAINER
375	? lerp (weight, 0, 100, 0, 100 - op->stats.Str)	361	? lerp (weight, 0, 100, 0, 100 - op->stats.Str)
376	: weight;	362	: weight;
377	}	363	}
…		…
383	static void	369	static void
384	adjust_weight (object *op, sint32 weight)	370	adjust_weight (object *op, sint32 weight)
385	{	371	{
386	while (op)	372	while (op)
387	{	373	{
		374	// adjust by actual difference to account for rounding errors
		375	// i.e. (w2 - w1) / f != w2 / f - w1 / f and the latter is correct
388	weight = weight_adjust (op, weight);	376	weight = weight_adjust_for (op, op->carrying)
		377	- weight_adjust_for (op, op->carrying - weight);
389		378
390	if (!weight)	379	if (!weight)
391	return;	380	return;
392		381
393	op->carrying += weight;	382	op->carrying += weight;
…		…
416	op->update_weight ();	405	op->update_weight ();
417		406
418	sum += op->total_weight ();	407	sum += op->total_weight ();
419	}	408	}
420		409
421	sum = weight_adjust (this, sum);	410	sum = weight_adjust_for (this, sum);
422		411
423	if (sum != carrying)	412	if (sum != carrying)
424	{	413	{
425	carrying = sum;	414	carrying = sum;
426		415
…		…
488	*/	477	*/
489	object *	478	object *
490	find_object_name (const char *str)	479	find_object_name (const char *str)
491	{	480	{
492	shstr_cmp str_ (str);	481	shstr_cmp str_ (str);
493	object *op;
494		482
		483	if (str_)
495	for_all_objects (op)	484	for_all_objects (op)
496	if (op->name == str_)	485	if (op->name == str_)
497	break;	486	return op;
498		487
499	return op;	488	return 0;
500	}	489	}
501		490
502	/*	491	/*
503	* Sets the owner and sets the skill and exp pointers to owner's current	492	* Sets the owner and sets the skill and exp pointers to owner's current
504	* skill and experience objects.	493	* skill and experience objects.
…		…
602	}	591	}
603		592
604	op->key_values = 0;	593	op->key_values = 0;
605	}	594	}
606		595
607	object &	596	/*
608	object::operator =(const object &src)	597	* copy_to first frees everything allocated by the dst object,
		598	* and then copies the contents of itself into the second
		599	* object, allocating what needs to be allocated. Basically, any
		600	* data that is malloc'd needs to be re-malloc/copied. Otherwise,
		601	* if the first object is freed, the pointers in the new object
		602	* will point at garbage.
		603	*/
		604	void
		605	object::copy_to (object *dst)
609	{	606	{
610	bool is_freed = flag [FLAG_FREED];	607	dst->remove ();
611	bool is_removed = flag [FLAG_REMOVED];
612
613	(object_copy )this = src;	608	(object_copy )dst = *this;
614
615	flag [FLAG_FREED] = is_freed;
616	flag [FLAG_REMOVED] = is_removed;	609	dst->flag [FLAG_REMOVED] = true;
617		610
618	/* Copy over key_values, if any. */	611	/* Copy over key_values, if any. */
619	if (src.key_values)	612	if (key_values)
620	{	613	{
621	key_value *tail = 0;	614	key_value *tail = 0;
622	key_values = 0;	615	dst->key_values = 0;
623		616
624	for (key_value *i = src.key_values; i; i = i->next)	617	for (key_value *i = key_values; i; i = i->next)
625	{	618	{
626	key_value *new_link = new key_value;	619	key_value *new_link = new key_value;
627		620
628	new_link->next = 0;	621	new_link->next = 0;
629	new_link->key = i->key;	622	new_link->key = i->key;
630	new_link->value = i->value;	623	new_link->value = i->value;
631		624
632	/* Try and be clever here, too. */	625	/* Try and be clever here, too. */
633	if (!key_values)	626	if (!dst->key_values)
634	{	627	{
635	key_values = new_link;	628	dst->key_values = new_link;
636	tail = new_link;	629	tail = new_link;
637	}	630	}
638	else	631	else
639	{	632	{
640	tail->next = new_link;	633	tail->next = new_link;
641	tail = new_link;	634	tail = new_link;
642	}	635	}
643	}	636	}
644	}	637	}
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		638
660	if (speed < 0)	639	if (speed < 0)
661	dst->speed_left -= rndm ();	640	dst->speed_left -= rndm ();
662		641
663	dst->set_speed (dst->speed);	642	dst->set_speed (dst->speed);
…		…
685	object *	664	object *
686	object::clone ()	665	object::clone ()
687	{	666	{
688	object *neu = create ();	667	object *neu = create ();
689	copy_to (neu);	668	copy_to (neu);
		669	neu->map = map; // not copied by copy_to
690	return neu;	670	return neu;
691	}	671	}
692		672
693	/*	673	/*
694	* If an object with the IS_TURNABLE() flag needs to be turned due	674	* If an object with the IS_TURNABLE() flag needs to be turned due
…		…
745	* UP_OBJ_FACE: only the objects face has changed.	725	* UP_OBJ_FACE: only the objects face has changed.
746	*/	726	*/
747	void	727	void
748	update_object (object *op, int action)	728	update_object (object *op, int action)
749	{	729	{
750	if (op == NULL)	730	if (!op)
751	{	731	{
752	/* this should never happen */	732	/* this should never happen */
753	LOG (llevDebug, "update_object() called for NULL object.\n");	733	LOG (llevError \| logBacktrace, "update_object() called for NULL object.\n");
754	return;	734	return;
755	}	735	}
756		736
757	if (op->env)	737	if (!op->is_on_map ())
758	{	738	{
759	/* Animation is currently handled by client, so nothing	739	/* Animation is currently handled by client, so nothing
760	* to do in this case.	740	* to do in this case.
761	*/	741	*/
762	return;	742	return;
763	}	743	}
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		744
771	/* make sure the object is within map boundaries */	745	/* 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)	746	if (op->x < 0 \|\| op->x >= op->map->width \|\| op->y < 0 \|\| op->y >= op->map->height)
773	{	747	{
774	LOG (llevError, "update_object() called for object out of map!\n");	748	LOG (llevError, "update_object() called for object out of map!\n");
…		…
793	\|\| (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC))	767	\|\| (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC))
794	\|\| (m.move_on \| op->move_on ) != m.move_on	768	\|\| (m.move_on \| op->move_on ) != m.move_on
795	\|\| (m.move_off \| op->move_off ) != m.move_off	769	\|\| (m.move_off \| op->move_off ) != m.move_off
796	\|\| (m.move_slow \| op->move_slow) != m.move_slow	770	\|\| (m.move_slow \| op->move_slow) != m.move_slow
797	/* This isn't perfect, but I don't expect a lot of objects to	771	/* This isn't perfect, but I don't expect a lot of objects to
798	* to have move_allow right now.	772	* have move_allow right now.
799	*/	773	*/
800	\|\| ((m.move_block \| op->move_block) & ~op->move_allow) != m.move_block	774	\|\| ((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)	775	\|\| 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp)
802	m.flags_ = 0;	776	m.flags_ = 0;
803	}	777	}
…		…
929	* drop on that space.	903	* drop on that space.
930	*/	904	*/
931	if (!drop_to_ground	905	if (!drop_to_ground
932	\|\| !map	906	\|\| !map
933	\|\| map->in_memory != MAP_ACTIVE	907	\|\| map->in_memory != MAP_ACTIVE
934	\|\| map->nodrop	908	\|\| map->no_drop
935	\|\| ms ().move_block == MOVE_ALL)	909	\|\| ms ().move_block == MOVE_ALL)
936	{	910	{
937	while (inv)	911	while (inv)
938	{
939	inv->destroy_inv (false);
940	inv->destroy ();	912	inv->destroy (true);
941	}
942	}	913	}
943	else	914	else
944	{ /* Put objects in inventory onto this space */	915	{ /* Put objects in inventory onto this space */
945	while (inv)	916	while (inv)
946	{	917	{
…		…
964	object *op = new object;	935	object *op = new object;
965	op->link ();	936	op->link ();
966	return op;	937	return op;
967	}	938	}
968		939
		940	static struct freed_map : maptile
		941	{
		942	freed_map ()
		943	{
		944	path = "<freed objects map>";
		945	name = "/internal/freed_objects_map";
		946	width = 3;
		947	height = 3;
		948	no_drop = 1;
		949	no_reset = 1;
		950
		951	alloc ();
		952	in_memory = MAP_ACTIVE;
		953	}
		954
		955	~freed_map ()
		956	{
		957	destroy ();
		958	}
		959	} freed_map; // freed objects are moved here to avoid crashes
		960
969	void	961	void
970	object::do_destroy ()	962	object::do_destroy ()
971	{	963	{
972	if (flag [FLAG_IS_LINKED])	964	if (flag [FLAG_IS_LINKED])
973	remove_button_link (this);	965	remove_button_link (this);
…		…
983	unlink ();	975	unlink ();
984		976
985	flag [FLAG_FREED] = 1;	977	flag [FLAG_FREED] = 1;
986		978
987	// hack to ensure that freed objects still have a valid map	979	// 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;	980	map = &freed_map;
1006	x = 1;	981	x = 1;
1007	y = 1;	982	y = 1;
1008	}
1009		983
1010	if (more)	984	if (more)
1011	{	985	{
1012	more->destroy ();	986	more->destroy ();
1013	more = 0;	987	more = 0;
…		…
1025	void	999	void
1026	object::destroy (bool destroy_inventory)	1000	object::destroy (bool destroy_inventory)
1027	{	1001	{
1028	if (destroyed ())	1002	if (destroyed ())
1029	return;	1003	return;
		1004
		1005	if (!is_head () && !head->destroyed ())
		1006	{
		1007	LOG (llevError \| logBacktrace, "tried to destroy the tail of an object");
		1008	head->destroy (destroy_inventory);
		1009	return;
		1010	}
1030		1011
1031	destroy_inv (!destroy_inventory);	1012	destroy_inv (!destroy_inventory);
1032		1013
1033	if (is_head ())	1014	if (is_head ())
1034	if (sound_destroy)	1015	if (sound_destroy)
…		…
1055	if (flag [FLAG_REMOVED])	1036	if (flag [FLAG_REMOVED])
1056	return;	1037	return;
1057		1038
1058	INVOKE_OBJECT (REMOVE, this);	1039	INVOKE_OBJECT (REMOVE, this);
1059		1040
1060	if (object *pl = visible_to ())
1061	esrv_del_item (pl->contr, count);
1062
1063	flag [FLAG_REMOVED] = true;	1041	flag [FLAG_REMOVED] = true;
1064		1042
1065	if (more)	1043	if (more)
1066	more->remove ();	1044	more->remove ();
1067		1045
…		…
1069	* In this case, the object to be removed is in someones	1047	* In this case, the object to be removed is in someones
1070	* inventory.	1048	* inventory.
1071	*/	1049	*/
1072	if (env)	1050	if (env)
1073	{	1051	{
		1052	flag [FLAG_REMOVED] = false; // hack around the issue of visible_to checking flag_removed
		1053	if (object *pl = visible_to ())
		1054	esrv_del_item (pl->contr, count);
		1055	flag [FLAG_REMOVED] = true; // hack around the issue of visible_to checking flag_removed
		1056
1074	adjust_weight (env, -total_weight ());	1057	adjust_weight (env, -total_weight ());
1075
1076	*(above ? &above->below : &env->inv) = below;
1077
1078	if (below)
1079	below->above = above;
1080		1058
1081	/* we set up values so that it could be inserted into	1059	/* we set up values so that it could be inserted into
1082	* the map, but we don't actually do that - it is up	1060	* the map, but we don't actually do that - it is up
1083	* to the caller to decide what we want to do.	1061	* to the caller to decide what we want to do.
1084	*/	1062	*/
1085	map = env->map;	1063	map = env->map;
1086	x = env->x;	1064	x = env->x;
1087	y = env->y;	1065	y = env->y;
		1066
		1067	// make sure cmov optimisation is applicable
		1068	*(above ? &above->below : &env->inv) = below;
		1069	*(below ? &below->above : &above ) = above; // &above is just a dummy
		1070
1088	above = 0;	1071	above = 0;
1089	below = 0;	1072	below = 0;
1090	env = 0;	1073	env = 0;
1091		1074
1092	/* NO_FIX_PLAYER is set when a great many changes are being	1075	/* NO_FIX_PLAYER is set when a great many changes are being
…		…
1096	if ((otmp = in_player ()) && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))	1079	if ((otmp = in_player ()) && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
1097	otmp->update_stats ();	1080	otmp->update_stats ();
1098	}	1081	}
1099	else if (map)	1082	else if (map)
1100	{	1083	{
1101	if (type == PLAYER)
1102	{
1103	// leaving a spot always closes any open container on the ground
1104	if (container && !container->env)
1105	// this causes spurious floorbox updates, but it ensures
1106	// that the CLOSE event is being sent.
1107	close_container ();
1108
1109	--map->players;
1110	map->touch ();
1111	}
1112
1113	map->dirty = true;	1084	map->dirty = true;
1114	mapspace &ms = this->ms ();	1085	mapspace &ms = this->ms ();
1115		1086
		1087	if (object *pl = ms.player ())
		1088	{
		1089	if (type == PLAYER) // this == pl(!)
		1090	{
		1091	// leaving a spot always closes any open container on the ground
		1092	if (container && !container->env)
		1093	// this causes spurious floorbox updates, but it ensures
		1094	// that the CLOSE event is being sent.
		1095	close_container ();
		1096
		1097	--map->players;
		1098	map->touch ();
		1099	}
		1100	else if (pl->container == this)
		1101	{
		1102	// removing a container should close it
		1103	close_container ();
		1104	}
		1105
		1106	esrv_del_item (pl->contr, count);
		1107	}
		1108
1116	/* link the object above us */	1109	/* link the object above us */
1117	if (above)	1110	// re-link, make sure compiler can easily use cmove
1118	above->below = below;	1111	*(above ? &above->below : &ms.top) = below;
1119	else	1112	*(below ? &below->above : &ms.bot) = above;
1120	ms.top = below; /* we were top, set new top */
1121
1122	/* Relink the object below us, if there is one */
1123	if (below)
1124	below->above = above;
1125	else
1126	{
1127	/* Nothing below, which means we need to relink map object for this space
1128	* use translated coordinates in case some oddness with map tiling is
1129	* evident
1130	*/
1131	if (GET_MAP_OB (map, x, y) != this)
1132	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 ());
1133
1134	ms.bot = above; /* goes on above it. */
1135	}
1136		1113
1137	above = 0;	1114	above = 0;
1138	below = 0;	1115	below = 0;
		1116
		1117	ms.flags_ = 0;
1139		1118
1140	if (map->in_memory == MAP_SAVING)	1119	if (map->in_memory == MAP_SAVING)
1141	return;	1120	return;
1142		1121
1143	int check_walk_off = !flag [FLAG_NO_APPLY];	1122	int check_walk_off = !flag [FLAG_NO_APPLY];
…		…
1149	* removed (most likely destroyed), update the player view	1128	* removed (most likely destroyed), update the player view
1150	* appropriately.	1129	* appropriately.
1151	*/	1130	*/
1152	pl->close_container ();	1131	pl->close_container ();
1153		1132
		1133	//TODO: the floorbox prev/next might need updating
1154	esrv_del_item (pl, count);	1134	//esrv_del_item (pl->contr, count);
		1135	//TODO: update floorbox to preserve ordering
		1136	if (pl->contr->ns)
		1137	pl->contr->ns->floorbox_update ();
1155	}	1138	}
1156		1139
1157	for (tmp = ms.bot; tmp; tmp = tmp->above)	1140	for (tmp = ms.bot; tmp; tmp = tmp->above)
1158	{	1141	{
1159	/* No point updating the players look faces if he is the object	1142	/* No point updating the players look faces if he is the object
…		…
1172	}	1155	}
1173		1156
1174	last = tmp;	1157	last = tmp;
1175	}	1158	}
1176		1159
1177	/* last == NULL if there are no objects on this space */
1178	//TODO: this makes little sense, why only update the topmost object?
1179	if (!last)
1180	map->at (x, y).flags_ = 0;
1181	else
1182	update_object (last, UP_OBJ_REMOVE);
1183
1184	if (flag [FLAG_BLOCKSVIEW] \|\| glow_radius)	1160	if (flag [FLAG_BLOCKSVIEW] \|\| glow_radius)
1185	update_all_los (map, x, y);	1161	update_all_los (map, x, y);
1186	}	1162	}
1187	}	1163	}
1188		1164
…		…
1251	* job preparing multi-part monsters.	1227	* job preparing multi-part monsters.
1252	*/	1228	*/
1253	object *	1229	object *
1254	insert_ob_in_map_at (object op, maptile m, object *originator, int flag, int x, int y)	1230	insert_ob_in_map_at (object op, maptile m, object *originator, int flag, int x, int y)
1255	{	1231	{
		1232	op->remove ();
		1233
1256	for (object *tmp = op->head_ (); tmp; tmp = tmp->more)	1234	for (object *tmp = op->head_ (); tmp; tmp = tmp->more)
1257	{	1235	{
1258	tmp->x = x + tmp->arch->x;	1236	tmp->x = x + tmp->arch->x;
1259	tmp->y = y + tmp->arch->y;	1237	tmp->y = y + tmp->arch->y;
1260	}	1238	}
…		…
1283	* just 'op' otherwise	1261	* just 'op' otherwise
1284	*/	1262	*/
1285	object *	1263	object *
1286	insert_ob_in_map (object op, maptile m, object *originator, int flag)	1264	insert_ob_in_map (object op, maptile m, object *originator, int flag)
1287	{	1265	{
1288	assert (!op->flag [FLAG_FREED]);	1266	if (op->is_on_map ())
		1267	{
		1268	LOG (llevError, "insert_ob_in_map called for object already on map");
		1269	abort ();
		1270	}
1289		1271
1290	object top, floor = NULL;	1272	if (op->env)
1291		1273	{
		1274	LOG (llevError, "insert_ob_in_map called for object in an inventory (proceeding)");
1292	op->remove ();	1275	op->remove ();
		1276	}
		1277
		1278	if (op->face && !face_info (op->face))//D TODO: remove soon
		1279	{//D
		1280	LOG (llevError \| logBacktrace, "op->face out of bounds: %s", op->debug_desc ());//D
		1281	op->face = 1;//D
		1282	}//D
1293		1283
1294	/* Ideally, the caller figures this out. However, it complicates a lot	1284	/* Ideally, the caller figures this out. However, it complicates a lot
1295	* of areas of callers (eg, anything that uses find_free_spot would now	1285	* of areas of callers (eg, anything that uses find_free_spot would now
1296	* need extra work	1286	* need extra work
1297	*/	1287	*/
1298	if (!xy_normalise (m, op->x, op->y))	1288	if (!xy_normalise (m, op->x, op->y))
1299	{	1289	{
1300	op->destroy (1);	1290	op->head_ ()->destroy (1);// remove head_ once all tail object destroyers found
1301	return 0;	1291	return 0;
1302	}	1292	}
1303		1293
1304	if (object *more = op->more)	1294	if (object *more = op->more)
1305	if (!insert_ob_in_map (more, m, originator, flag))	1295	if (!insert_ob_in_map (more, m, originator, flag))
…		…
1314	*/	1304	*/
1315	if (op->nrof && !(flag & INS_NO_MERGE))	1305	if (op->nrof && !(flag & INS_NO_MERGE))
1316	for (object *tmp = ms.bot; tmp; tmp = tmp->above)	1306	for (object *tmp = ms.bot; tmp; tmp = tmp->above)
1317	if (object::can_merge (op, tmp))	1307	if (object::can_merge (op, tmp))
1318	{	1308	{
		1309	// TODO: we actually want to update tmp, not op,
		1310	// but some caller surely breaks when we return tmp
		1311	// from here :/
1319	op->nrof += tmp->nrof;	1312	op->nrof += tmp->nrof;
1320	tmp->destroy (1);	1313	tmp->destroy (1);
1321	}	1314	}
1322		1315
1323	CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */	1316	CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */
…		…
1332	{	1325	{
1333	LOG (llevError, "insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n");	1326	LOG (llevError, "insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n");
1334	abort ();	1327	abort ();
1335	}	1328	}
1336		1329
		1330	if (!originator->is_on_map ())
		1331	LOG (llevDebug \| logBacktrace, "insert_ob_in_map(%s) called with INS_BELOW_ORIGINATOR when originator '%s' not on map",
		1332	op->debug_desc (), originator->debug_desc ());
		1333
1337	op->above = originator;	1334	op->above = originator;
1338	op->below = originator->below;	1335	op->below = originator->below;
1339
1340	if (op->below)
1341	op->below->above = op;
1342	else
1343	ms.bot = op;
1344
1345	/* since below originator, no need to update top */
1346	originator->below = op;	1336	originator->below = op;
		1337
		1338	*(op->below ? &op->below->above : &ms.bot) = op;
1347	}	1339	}
1348	else	1340	else
1349	{	1341	{
1350	top = ms.bot;	1342	object *floor = 0;
		1343	object *top = ms.top;
1351		1344
1352	/* If there are other objects, then */	1345	/* If there are other objects, then */
1353	if (top)	1346	if (top)
1354	{	1347	{
1355	object *last = 0;
1356
1357	/*	1348	/*
1358	* If there are multiple objects on this space, we do some trickier handling.	1349	* If there are multiple objects on this space, we do some trickier handling.
1359	* We've already dealt with merging if appropriate.	1350	* We've already dealt with merging if appropriate.
1360	* Generally, we want to put the new object on top. But if	1351	* Generally, we want to put the new object on top. But if
1361	* flag contains INS_ABOVE_FLOOR_ONLY, once we find the last	1352	* flag contains INS_ABOVE_FLOOR_ONLY, once we find the last
…		…
1364	* once we get to them. This reduces the need to traverse over all of	1355	* once we get to them. This reduces the need to traverse over all of
1365	* them when adding another one - this saves quite a bit of cpu time	1356	* them when adding another one - this saves quite a bit of cpu time
1366	* when lots of spells are cast in one area. Currently, it is presumed	1357	* when lots of spells are cast in one area. Currently, it is presumed
1367	* that flying non pickable objects are spell objects.	1358	* that flying non pickable objects are spell objects.
1368	*/	1359	*/
1369	for (top = ms.bot; top; top = top->above)	1360	for (object *tmp = ms.bot; tmp; tmp = tmp->above)
1370	{	1361	{
1371	if (QUERY_FLAG (top, FLAG_IS_FLOOR) \|\| QUERY_FLAG (top, FLAG_OVERLAY_FLOOR))	1362	if (QUERY_FLAG (tmp, FLAG_IS_FLOOR) \|\| QUERY_FLAG (tmp, FLAG_OVERLAY_FLOOR))
1372	floor = top;	1363	floor = tmp;
1373		1364
1374	if (QUERY_FLAG (top, FLAG_NO_PICK) && (top->move_type & (MOVE_FLY_LOW \| MOVE_FLY_HIGH)) && !QUERY_FLAG (top, FLAG_IS_FLOOR))	1365	if (QUERY_FLAG (tmp, FLAG_NO_PICK) && (tmp->move_type & (MOVE_FLY_LOW \| MOVE_FLY_HIGH)) && !QUERY_FLAG (tmp, FLAG_IS_FLOOR))
1375	{	1366	{
1376	/* We insert above top, so we want this object below this */	1367	/* We insert above top, so we want this object below this */
1377	top = top->below;	1368	top = tmp->below;
1378	break;	1369	break;
1379	}	1370	}
1380		1371
1381	last = top;	1372	top = tmp;
1382	}	1373	}
1383
1384	/* Don't want top to be NULL, so set it to the last valid object */
1385	top = last;
1386		1374
1387	/* We let update_position deal with figuring out what the space	1375	/* We let update_position deal with figuring out what the space
1388	* looks like instead of lots of conditions here.	1376	* looks like instead of lots of conditions here.
1389	* makes things faster, and effectively the same result.	1377	* makes things faster, and effectively the same result.
1390	*/	1378	*/
…		…
1397	*/	1385	*/
1398	if (!(flag & INS_ON_TOP)	1386	if (!(flag & INS_ON_TOP)
1399	&& ms.flags () & P_BLOCKSVIEW	1387	&& ms.flags () & P_BLOCKSVIEW
1400	&& (op->face && !faces [op->face].visibility))	1388	&& (op->face && !faces [op->face].visibility))
1401	{	1389	{
		1390	object *last;
		1391
1402	for (last = top; last != floor; last = last->below)	1392	for (last = top; last != floor; last = last->below)
1403	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))	1393	if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))
1404	break;	1394	break;
1405		1395
1406	/* Check to see if we found the object that blocks view,	1396	/* Check to see if we found the object that blocks view,
…		…
1414	} /* If objects on this space */	1404	} /* If objects on this space */
1415		1405
1416	if (flag & INS_ABOVE_FLOOR_ONLY)	1406	if (flag & INS_ABOVE_FLOOR_ONLY)
1417	top = floor;	1407	top = floor;
1418		1408
1419	/* Top is the object that our object (op) is going to get inserted above.	1409	// insert object above top, or bottom-most if top = 0
1420	*/
1421
1422	/* First object on this space */
1423	if (!top)	1410	if (!top)
1424	{	1411	{
		1412	op->below = 0;
1425	op->above = ms.bot;	1413	op->above = ms.bot;
1426
1427	if (op->above)
1428	op->above->below = op;
1429
1430	op->below = 0;
1431	ms.bot = op;	1414	ms.bot = op;
		1415
		1416	*(op->above ? &op->above->below : &ms.top) = op;
1432	}	1417	}
1433	else	1418	else
1434	{ /* get inserted into the stack above top */	1419	{
1435	op->above = top->above;	1420	op->above = top->above;
1436
1437	if (op->above)
1438	op->above->below = op;	1421	top->above = op;
1439		1422
1440	op->below = top;	1423	op->below = top;
1441	top->above = op;	1424	*(op->above ? &op->above->below : &ms.top) = op;
1442	}	1425	}
1443		1426	}
1444	if (!op->above)
1445	ms.top = op;
1446	} /* else not INS_BELOW_ORIGINATOR */
1447		1427
1448	if (op->type == PLAYER)	1428	if (op->type == PLAYER)
1449	{	1429	{
1450	op->contr->do_los = 1;	1430	op->contr->do_los = 1;
1451	++op->map->players;	1431	++op->map->players;
…		…
1453	}	1433	}
1454		1434
1455	op->map->dirty = true;	1435	op->map->dirty = true;
1456		1436
1457	if (object *pl = ms.player ())	1437	if (object *pl = ms.player ())
		1438	//TODO: the floorbox prev/next might need updating
1458	esrv_send_item (pl, op);	1439	//esrv_send_item (pl, op);
		1440	//TODO: update floorbox to preserve ordering
		1441	if (pl->contr->ns)
		1442	pl->contr->ns->floorbox_update ();
1459		1443
1460	/* If this object glows, it may affect lighting conditions that are	1444	/* If this object glows, it may affect lighting conditions that are
1461	* visible to others on this map. But update_all_los is really	1445	* visible to others on this map. But update_all_los is really
1462	* an inefficient way to do this, as it means los for all players	1446	* an inefficient way to do this, as it means los for all players
1463	* on the map will get recalculated. The players could very well	1447	* on the map will get recalculated. The players could very well
…		…
1543	if (!nr)	1527	if (!nr)
1544	return true;	1528	return true;
1545		1529
1546	nr = min (nr, nrof);	1530	nr = min (nr, nrof);
1547		1531
		1532	if (nrof > nr)
		1533	{
1548	nrof -= nr;	1534	nrof -= nr;
1549
1550	if (nrof)
1551	{
1552	adjust_weight (env, -weight * nr); // carrying == 0	1535	adjust_weight (env, -weight * max (1, nr)); // carrying == 0
1553		1536
1554	if (object *pl = visible_to ())	1537	if (object *pl = visible_to ())
1555	esrv_update_item (UPD_NROF, pl, this);	1538	esrv_update_item (UPD_NROF, pl, this);
1556		1539
1557	return true;	1540	return true;
1558	}	1541	}
1559	else	1542	else
1560	{	1543	{
1561	destroy (1);	1544	destroy ();
1562	return false;	1545	return false;
1563	}	1546	}
1564	}	1547	}
1565		1548
1566	/*	1549	/*
…		…
1583	}	1566	}
1584	else	1567	else
1585	{	1568	{
1586	decrease (nr);	1569	decrease (nr);
1587		1570
1588	object *op = object_create_clone (this);	1571	object *op = deep_clone ();
1589	op->nrof = nr;	1572	op->nrof = nr;
1590	return op;	1573	return op;
1591	}	1574	}
1592	}	1575	}
1593		1576
…		…
1814	LOG (llevError, "Present_arch called outside map.\n");	1797	LOG (llevError, "Present_arch called outside map.\n");
1815	return NULL;	1798	return NULL;
1816	}	1799	}
1817		1800
1818	for (object *tmp = m->at (x, y).bot; tmp; tmp = tmp->above)	1801	for (object *tmp = m->at (x, y).bot; tmp; tmp = tmp->above)
1819	if (tmp->arch == at)	1802	if (tmp->arch->archname == at->archname)
1820	return tmp;	1803	return tmp;
1821		1804
1822	return NULL;	1805	return NULL;
1823	}	1806	}
1824		1807
…		…
1888	* The first matching object is returned, or NULL if none.	1871	* The first matching object is returned, or NULL if none.
1889	*/	1872	*/
1890	object *	1873	object *
1891	present_arch_in_ob (const archetype at, const object op)	1874	present_arch_in_ob (const archetype at, const object op)
1892	{	1875	{
1893	for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below)	1876	for (object *tmp = op->inv; tmp; tmp = tmp->below)
1894	if (tmp->arch == at)	1877	if (tmp->arch->archname == at->archname)
1895	return tmp;	1878	return tmp;
1896		1879
1897	return NULL;	1880	return NULL;
1898	}	1881	}
1899		1882
…		…
2308		2291
2309	/*	2292	/*
2310	* create clone from object to another	2293	* create clone from object to another
2311	*/	2294	*/
2312	object *	2295	object *
2313	object_create_clone (object *asrc)	2296	object::deep_clone ()
2314	{	2297	{
2315	object dst = 0, tmp, src, prev, *item;	2298	assert (("deep_clone called on non-head object", is_head ()));
2316		2299
2317	if (!asrc)	2300	object *dst = clone ();
2318	return 0;
2319		2301
2320	src = asrc->head_ ();	2302	object *prev = dst;
2321
2322	prev = 0;
2323	for (object *part = src; part; part = part->more)	2303	for (object *part = this->more; part; part = part->more)
2324	{	2304	{
2325	tmp = part->clone ();	2305	object *tmp = part->clone ();
2326	tmp->x -= src->x;
2327	tmp->y -= src->y;
2328
2329	if (!part->head)
2330	{
2331	dst = tmp;
2332	tmp->head = 0;
2333	}
2334	else
2335	tmp->head = dst;	2306	tmp->head = dst;
2336
2337	tmp->more = 0;
2338
2339	if (prev)
2340	prev->more = tmp;	2307	prev->more = tmp;
2341
2342	prev = tmp;	2308	prev = tmp;
2343	}	2309	}
2344		2310
2345	for (item = src->inv; item; item = item->below)	2311	for (object *item = inv; item; item = item->below)
2346	insert_ob_in_ob (object_create_clone (item), dst);	2312	insert_ob_in_ob (item->deep_clone (), dst);
2347		2313
2348	return dst;	2314	return dst;
2349	}	2315	}
2350		2316
2351	/* This returns the first object in who's inventory that	2317	/* This returns the first object in who's inventory that
…		…
2360	return tmp;	2326	return tmp;
2361		2327
2362	return 0;	2328	return 0;
2363	}	2329	}
2364		2330
2365	/* If ob has a field named key, return the link from the list,	2331	const shstr &
2366	* otherwise return NULL.	2332	object::kv_get (const shstr &key) const
2367	*
2368	* key must be a passed in shared string - otherwise, this won't
2369	* do the desired thing.
2370	*/
2371	key_value *
2372	get_ob_key_link (const object ob, const char key)
2373	{	2333	{
2374	for (key_value *link = ob->key_values; link; link = link->next)	2334	for (key_value *kv = key_values; kv; kv = kv->next)
2375	if (link->key == key)	2335	if (kv->key == key)
2376	return link;
2377
2378	return 0;
2379	}
2380
2381	/*
2382	* Returns the value of op has an extra_field for key, or NULL.
2383	*
2384	* The argument doesn't need to be a shared string.
2385	*
2386	* The returned string is shared.
2387	*/
2388	const char *
2389	get_ob_key_value (const object op, const char const key)
2390	{
2391	key_value *link;
2392	shstr_cmp canonical_key (key);
2393
2394	if (!canonical_key)
2395	{
2396	/* 1. There being a field named key on any object
2397	* implies there'd be a shared string to find.
2398	* 2. Since there isn't, no object has this field.
2399	* 3. Therefore, this object doesn't have this field.
2400	*/
2401	return 0;
2402	}
2403
2404	/* This is copied from get_ob_key_link() above -
2405	* only 4 lines, and saves the function call overhead.
2406	*/
2407	for (link = op->key_values; link; link = link->next)
2408	if (link->key == canonical_key)
2409	return link->value;	2336	return kv->value;
2410		2337
2411	return 0;	2338	return shstr_null;
2412	}	2339	}
2413		2340
2414	/*	2341	void
2415	* Updates the canonical_key in op to value.	2342	object::kv_set (const shstr &key, const shstr &value)
2416	*
2417	* canonical_key is a shared string (value doesn't have to be).
2418	*
2419	* Unless add_key is TRUE, it won't add fields, only change the value of existing
2420	* keys.
2421	*
2422	* Returns TRUE on success.
2423	*/
2424	int
2425	set_ob_key_value_s (object op, const shstr & canonical_key, const char value, int add_key)
2426	{	2343	{
2427	key_value field = NULL, last = NULL;	2344	for (key_value *kv = key_values; kv; kv = kv->next)
2428		2345	if (kv->key == key)
2429	for (field = op->key_values; field != NULL; field = field->next)
2430	{
2431	if (field->key != canonical_key)
2432	{	2346	{
2433	last = field;	2347	kv->value = value;
2434	continue;	2348	return;
2435	}	2349	}
2436		2350
2437	if (value)	2351	key_value *kv = new key_value;
2438	field->value = value;	2352
2439	else	2353	kv->next = key_values;
		2354	kv->key = key;
		2355	kv->value = value;
		2356
		2357	key_values = kv;
		2358	}
		2359
		2360	void
		2361	object::kv_del (const shstr &key)
		2362	{
		2363	for (key_value *kvp = &key_values; kvp; kvp = &(*kvp)->next)
		2364	if ((*kvp)->key == key)
2440	{	2365	{
2441	/* Basically, if the archetype has this key set,	2366	key_value kv = kvp;
2442	* we need to store the null value so when we save	2367	kvp = (kvp)->next;
2443	* it, we save the empty value so that when we load,	2368	delete kv;
2444	* we get this value back again.	2369	return;
2445	*/
2446	if (get_ob_key_link (op->arch, canonical_key))
2447	field->value = 0;
2448	else
2449	{
2450	if (last)
2451	last->next = field->next;
2452	else
2453	op->key_values = field->next;
2454
2455	delete field;
2456	}
2457	}	2370	}
2458	return TRUE;
2459	}
2460	/* IF we get here, key doesn't exist */
2461
2462	/* No field, we'll have to add it. */
2463
2464	if (!add_key)
2465	return FALSE;
2466
2467	/* There isn't any good reason to store a null
2468	* value in the key/value list. If the archetype has
2469	* this key, then we should also have it, so shouldn't
2470	* be here. If user wants to store empty strings,
2471	* should pass in ""
2472	*/
2473	if (value == NULL)
2474	return TRUE;
2475
2476	field = new key_value;
2477
2478	field->key = canonical_key;
2479	field->value = value;
2480	/* Usual prepend-addition. */
2481	field->next = op->key_values;
2482	op->key_values = field;
2483
2484	return TRUE;
2485	}
2486
2487	/*
2488	* Updates the key in op to value.
2489	*
2490	* If add_key is FALSE, this will only update existing keys,
2491	* and not add new ones.
2492	* In general, should be little reason FALSE is ever passed in for add_key
2493	*
2494	* Returns TRUE on success.
2495	*/
2496	int
2497	set_ob_key_value (object op, const char key, const char *value, int add_key)
2498	{
2499	shstr key_ (key);
2500
2501	return set_ob_key_value_s (op, key_, value, add_key);
2502	}	2371	}
2503		2372
2504	object::depth_iterator::depth_iterator (object *container)	2373	object::depth_iterator::depth_iterator (object *container)
2505	: iterator_base (container)	2374	: iterator_base (container)
2506	{	2375	{
…		…
2556	{	2425	{
2557	char flagdesc[512];	2426	char flagdesc[512];
2558	char info2[256 * 4];	2427	char info2[256 * 4];
2559	char *p = info;	2428	char *p = info;
2560		2429
2561	p += snprintf (p, 512, "{cnt:%d,uuid:%s,name:\"%s\"%s%s,flags:[%s],type:%d}",	2430	p += snprintf (p, 512, "{cnt:%d,uuid:%s,name:\"%s\"%s%s%s,flags:[%s],type:%d}",
2562	count,	2431	count,
2563	uuid.c_str (),	2432	uuid.c_str (),
2564	&name,	2433	&name,
2565	title ? "\",title:\"" : "",	2434	title ? ",title:\"" : "",
2566	title ? (const char *)title : "",	2435	title ? (const char *)title : "",
		2436	title ? "\"" : "",
2567	flag_desc (flagdesc, 512), type);	2437	flag_desc (flagdesc, 512), type);
2568		2438
2569	if (!flag[FLAG_REMOVED] && env)	2439	if (!flag[FLAG_REMOVED] && env)
2570	p += snprintf (p, 256, "(in %s)", env->debug_desc (info2));	2440	p += snprintf (p, 256, "(in %s)", env->debug_desc (info2));
2571		2441
…		…
2604	object::open_container (object *new_container)	2474	object::open_container (object *new_container)
2605	{	2475	{
2606	if (container == new_container)	2476	if (container == new_container)
2607	return;	2477	return;
2608		2478
2609	if (object *old_container = container)	2479	object *old_container = container;
		2480
		2481	if (old_container)
2610	{	2482	{
2611	if (INVOKE_OBJECT (CLOSE, old_container, ARG_OBJECT (this)))	2483	if (INVOKE_OBJECT (CLOSE, old_container, ARG_OBJECT (this)))
2612	return;	2484	return;
2613		2485
2614	#if 0	2486	#if 0
…		…
2616	if (object *closer = old_container->inv)	2488	if (object *closer = old_container->inv)
2617	if (closer->type == CLOSE_CON)	2489	if (closer->type == CLOSE_CON)
2618	closer->destroy ();	2490	closer->destroy ();
2619	#endif	2491	#endif
2620		2492
		2493	// make sure the container is available
		2494	esrv_send_item (this, old_container);
		2495
2621	old_container->flag [FLAG_APPLIED] = false;	2496	old_container->flag [FLAG_APPLIED] = false;
2622	container = 0;	2497	container = 0;
2623		2498
		2499	// client needs item update to make it work, client bug requires this to be separate
2624	esrv_update_item (UPD_FLAGS, this, old_container);	2500	esrv_update_item (UPD_FLAGS, this, old_container);
		2501
2625	new_draw_info_format (NDI_UNIQUE, 0, this, "You close %s.", query_name (old_container));	2502	new_draw_info_format (NDI_UNIQUE, 0, this, "You close %s.", query_name (old_container));
2626	play_sound (sound_find ("chest_close"));	2503	play_sound (sound_find ("chest_close"));
2627	}	2504	}
2628		2505
2629	if (new_container)	2506	if (new_container)
…		…
2642	}	2519	}
2643	#endif	2520	#endif
2644		2521
2645	new_draw_info_format (NDI_UNIQUE, 0, this, "You open %s.", query_name (new_container));	2522	new_draw_info_format (NDI_UNIQUE, 0, this, "You open %s.", query_name (new_container));
2646		2523
		2524	// make sure the container is available, client bug requires this to be separate
		2525	esrv_send_item (this, new_container);
		2526
2647	new_container->flag [FLAG_APPLIED] = true;	2527	new_container->flag [FLAG_APPLIED] = true;
2648	container = new_container;	2528	container = new_container;
2649		2529
		2530	// client needs flag change
2650	esrv_update_item (UPD_FLAGS, this, new_container);	2531	esrv_update_item (UPD_FLAGS, this, new_container);
2651	esrv_send_inventory (this, new_container);	2532	esrv_send_inventory (this, new_container);
2652	play_sound (sound_find ("chest_open"));	2533	play_sound (sound_find ("chest_open"));
2653	}	2534	}
		2535	// else if (!old_container->env && contr && contr->ns)
		2536	// contr->ns->floorbox_reset ();
2654	}	2537	}
2655		2538
2656	object *	2539	object *
2657	object::force_find (const shstr name)	2540	object::force_find (const shstr name)
2658	{	2541	{

Diff Legend

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

Comparing deliantra/server/common/object.C (file contents): Revision 1.217 by root, Wed Apr 23 07:13:23 2008 UTC vs. Revision 1.254 by root, Sun Aug 31 01:17:11 2008 UTC

Diff Legend

Comparing deliantra/server/common/object.C (file contents):
Revision 1.217 by root, Wed Apr 23 07:13:23 2008 UTC vs.
Revision 1.254 by root, Sun Aug 31 01:17:11 2008 UTC