ViewVC Help
View File | Revision Log | Show Annotations | Download File
/cvs/deliantra/server/common/object.C
(Generate patch)

Comparing deliantra/server/common/object.C (file contents):
Revision 1.222 by root, Wed Apr 30 05:06:36 2008 UTC vs.
Revision 1.289 by root, Sun Oct 11 05:31:54 2009 UTC

1/* 1/*
2 * This file is part of Deliantra, the Roguelike Realtime MMORPG. 2 * This file is part of Deliantra, the Roguelike Realtime MMORPG.
3 * 3 *
4 * Copyright (©) 2005,2006,2007,2008 Marc Alexander Lehmann / Robin Redeker / the Deliantra team 4 * Copyright (©) 2005,2006,2007,2008,2009 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
5 * Copyright (©) 2001,2007 Mark Wedel & Crossfire Development Team 5 * Copyright (©) 2001,2007 Mark Wedel & Crossfire Development Team
6 * Copyright (©) 1992,2007 Frank Tore Johansen 6 * Copyright (©) 1992,2007 Frank Tore Johansen
7 * 7 *
8 * Deliantra is free software: you can redistribute it and/or modify 8 * Deliantra is free software: you can redistribute it and/or modify
9 * it under the terms of the GNU General Public License as published by 9 * it under the terms of the GNU General Public License as published by
32#include <bitset> 32#include <bitset>
33 33
34UUID UUID::cur; 34UUID UUID::cur;
35static uint64_t seq_next_save; 35static uint64_t seq_next_save;
36static const uint64 UUID_GAP = 1<<19; 36static const uint64 UUID_GAP = 1<<19;
37uint32_t mapspace::smellcount = 10000;
37 38
38objectvec objects; 39objectvec objects;
39activevec actives; 40activevec actives;
40 41
41short freearr_x[SIZEOFFREE] = { 42short freearr_x[SIZEOFFREE] = {
139 140
140/* Returns TRUE if every key_values in wants has a partner with the same value in has. */ 141/* Returns TRUE if every key_values in wants has a partner with the same value in has. */
141static bool 142static bool
142compare_ob_value_lists_one (const object *wants, const object *has) 143compare_ob_value_lists_one (const object *wants, const object *has)
143{ 144{
144 key_value *wants_field;
145
146 /* n-squared behaviour (see get_ob_key_link()), but I'm hoping both 145 /* 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 146 * objects with lists are rare, and lists stay short. If not, use a
148 * different structure or at least keep the lists sorted... 147 * different structure or at least keep the lists sorted...
149 */ 148 */
150 149
151 /* For each field in wants, */ 150 /* For each field in wants, */
152 for (wants_field = wants->key_values; wants_field; wants_field = wants_field->next) 151 for (key_value *kv = wants->key_values; kv; kv = kv->next)
153 { 152 if (has->kv_get (kv->key) != kv->value)
154 key_value *has_field; 153 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 154
169 /* If we get here, every field in wants has a matching field in has. */ 155 /* If we get here, every field in wants has a matching field in has. */
170 return 1; 156 return true;
171} 157}
172 158
173/* Returns TRUE if ob1 has the same key_values as ob2. */ 159/* Returns TRUE if ob1 has the same key_values as ob2. */
174static bool 160static bool
175compare_ob_value_lists (const object *ob1, const object *ob2) 161compare_ob_value_lists (const object *ob1, const object *ob2)
176{ 162{
177 /* However, there may be fields in has which aren't partnered in wants, 163 /* However, there may be fields in has which aren't partnered in wants,
178 * so we need to run the comparison *twice*. :( 164 * so we need to run the comparison *twice*. :(
179 */ 165 */
180 return compare_ob_value_lists_one (ob1, ob2) && compare_ob_value_lists_one (ob2, ob1); 166 return compare_ob_value_lists_one (ob1, ob2)
167 && compare_ob_value_lists_one (ob2, ob1);
181} 168}
182 169
183/* Function examines the 2 objects given to it, and returns true if 170/* Function examines the 2 objects given to it, and returns true if
184 * they can be merged together. 171 * they can be merged together.
185 * 172 *
200 || ob1->speed != ob2->speed 187 || ob1->speed != ob2->speed
201 || ob1->value != ob2->value 188 || ob1->value != ob2->value
202 || ob1->name != ob2->name) 189 || ob1->name != ob2->name)
203 return 0; 190 return 0;
204 191
205 /* Do not merge objects if nrof would overflow. First part checks 192 /* Do not merge objects if nrof would overflow, assume nrof
206 * for unsigned overflow (2c), second part checks whether the result 193 * is always 0 .. 2**31-1 */
207 * would fit into a 32 bit signed int, which is often used to hold 194 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; 195 return 0;
212 196
213 /* If the objects have been identified, set the BEEN_APPLIED flag. 197 /* 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 198 * 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 199 * just can't ignore the been applied or identified flags, as they
220 SET_FLAG (ob1, FLAG_BEEN_APPLIED); 204 SET_FLAG (ob1, FLAG_BEEN_APPLIED);
221 205
222 if (QUERY_FLAG (ob2, FLAG_IDENTIFIED)) 206 if (QUERY_FLAG (ob2, FLAG_IDENTIFIED))
223 SET_FLAG (ob2, FLAG_BEEN_APPLIED); 207 SET_FLAG (ob2, FLAG_BEEN_APPLIED);
224 208
225 if (ob1->arch->name != ob2->arch->name 209 if (ob1->arch->archname != ob2->arch->archname
226 || ob1->name != ob2->name 210 || ob1->name != ob2->name
227 || ob1->title != ob2->title 211 || ob1->title != ob2->title
228 || ob1->msg != ob2->msg 212 || ob1->msg != ob2->msg
229 || ob1->weight != ob2->weight 213 || ob1->weight != ob2->weight
230 || ob1->attacktype != ob2->attacktype 214 || ob1->attacktype != ob2->attacktype
231 || ob1->magic != ob2->magic 215 || ob1->magic != ob2->magic
232 || ob1->slaying != ob2->slaying 216 || ob1->slaying != ob2->slaying
233 || ob1->skill != ob2->skill 217 || ob1->skill != ob2->skill
234 || ob1->value != ob2->value 218 || ob1->value != ob2->value
235 || ob1->animation_id != ob2->animation_id 219 || ob1->animation_id != ob2->animation_id
220 || (ob1->face != ob2->face && !ob1->animation_id) // face and animation are dependent on each other
236 || ob1->client_type != ob2->client_type 221 || ob1->client_type != ob2->client_type
237 || ob1->materialname != ob2->materialname 222 || ob1->materialname != ob2->materialname
238 || ob1->lore != ob2->lore 223 || ob1->lore != ob2->lore
239 || ob1->subtype != ob2->subtype 224 || ob1->subtype != ob2->subtype
240 || ob1->move_type != ob2->move_type 225 || ob1->move_type != ob2->move_type
248 || memcmp (&ob1->stats , &ob2->stats , sizeof (ob1->stats))) 233 || memcmp (&ob1->stats , &ob2->stats , sizeof (ob1->stats)))
249 return 0; 234 return 0;
250 235
251 if ((ob1->flag ^ ob2->flag) 236 if ((ob1->flag ^ ob2->flag)
252 .reset (FLAG_INV_LOCKED) 237 .reset (FLAG_INV_LOCKED)
253 .reset (FLAG_CLIENT_SENT)
254 .reset (FLAG_REMOVED) 238 .reset (FLAG_REMOVED)
255 .any ()) 239 .any ())
256 return 0; 240 return 0;
257 241
258 /* This is really a spellbook check - we should in general 242 /* This is really a spellbook check - we should in general
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
345 if (env->type == PLAYER) 329 if (env->is_player ())
346 return env; 330 return env;
347 331
348 // else a player could have our env open 332 // else a player could have our env open
349 object *envest = env->outer_env (); 333 object *envest = env->outer_env_or_self ();
350 334
351 // the player itself is always on a map, so we will find him here 335 // the player itself is always on a map, so we will find him here
352 // even if our inv is in a player. 336 // even if our inv is in a player.
353 if (envest->is_on_map ()) 337 if (envest->is_on_map ())
354 if (object *pl = envest->ms ().player ()) 338 if (object *pl = envest->ms ().player ())
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)
363 if (!pl->container || this == pl->container) 348 || this == pl->container)
364 return pl; 349 return pl;
365 } 350 }
366 } 351 }
367 352
368 return 0; 353 return 0;
369} 354}
370 355
371// adjust weight per container type ("of holding") 356// adjust weight per container type ("of holding")
372static sint32 357static sint32
373weight_adjust (object *op, sint32 weight) 358weight_adjust_for (object *op, sint32 weight)
374{ 359{
375 return op->type == CONTAINER 360 return op->type == CONTAINER
376 ? lerp (weight, 0, 100, 0, 100 - op->stats.Str) 361 ? lerp (weight, 0, 100, 0, 100 - op->stats.Str)
377 : weight; 362 : weight;
378} 363}
384static void 369static void
385adjust_weight (object *op, sint32 weight) 370adjust_weight (object *op, sint32 weight)
386{ 371{
387 while (op) 372 while (op)
388 { 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
389 weight = weight_adjust (op, weight); 376 weight = weight_adjust_for (op, op->carrying)
377 - weight_adjust_for (op, op->carrying - weight);
390 378
391 if (!weight) 379 if (!weight)
392 return; 380 return;
393 381
394 op->carrying += weight; 382 op->carrying += weight;
417 op->update_weight (); 405 op->update_weight ();
418 406
419 sum += op->total_weight (); 407 sum += op->total_weight ();
420 } 408 }
421 409
422 sum = weight_adjust (this, sum); 410 sum = weight_adjust_for (this, sum);
423 411
424 if (sum != carrying) 412 if (sum != carrying)
425 { 413 {
426 carrying = sum; 414 carrying = sum;
427 415
441 return strdup ("[NULLOBJ]"); 429 return strdup ("[NULLOBJ]");
442 430
443 object_freezer freezer; 431 object_freezer freezer;
444 op->write (freezer); 432 op->write (freezer);
445 return freezer.as_string (); 433 return freezer.as_string ();
434}
435
436char *
437object::as_string ()
438{
439 return dump_object (this);
446} 440}
447 441
448/* 442/*
449 * get_nearest_part(multi-object, object 2) returns the part of the 443 * get_nearest_part(multi-object, object 2) returns the part of the
450 * multi-object 1 which is closest to the second object. 444 * multi-object 1 which is closest to the second object.
489 */ 483 */
490object * 484object *
491find_object_name (const char *str) 485find_object_name (const char *str)
492{ 486{
493 shstr_cmp str_ (str); 487 shstr_cmp str_ (str);
494 object *op;
495 488
489 if (str_)
496 for_all_objects (op) 490 for_all_objects (op)
497 if (op->name == str_) 491 if (op->name == str_)
498 break; 492 return op;
499 493
500 return op; 494 return 0;
501} 495}
502 496
503/* 497/*
504 * Sets the owner and sets the skill and exp pointers to owner's current 498 * Sets the owner and sets the skill and exp pointers to owner's current
505 * skill and experience objects. 499 * skill and experience objects.
567 update_stats (); 561 update_stats ();
568 562
569 new_draw_info_format (NDI_UNIQUE, 0, this, 563 new_draw_info_format (NDI_UNIQUE, 0, this,
570 "You try to balance all your items at once, " 564 "You try to balance all your items at once, "
571 "but the %s is just too much for your body. " 565 "but the %s is just too much for your body. "
572 "[You need to unapply some items first.]", &ob->name); 566 "[You need to unapply some items first - use the 'body' command to see "
567 "how many items you cna wera on a specific body part.]", &ob->name);
573 return false; 568 return false;
574 } 569 }
575 570
576 //new_draw_info_format (NDI_UNIQUE, 0, this, "You switch to your %s.", &ob->name); 571 //new_draw_info_format (NDI_UNIQUE, 0, this, "You switch to your %s.", &ob->name);
577 } 572 }
603 } 598 }
604 599
605 op->key_values = 0; 600 op->key_values = 0;
606} 601}
607 602
608object & 603/*
609object::operator =(const object &src) 604 * copy_to first frees everything allocated by the dst object,
605 * and then copies the contents of itself into the second
606 * object, allocating what needs to be allocated. Basically, any
607 * data that is malloc'd needs to be re-malloc/copied. Otherwise,
608 * if the first object is freed, the pointers in the new object
609 * will point at garbage.
610 */
611void
612object::copy_to (object *dst)
610{ 613{
611 bool is_freed = flag [FLAG_FREED]; 614 dst->remove ();
612 bool is_removed = flag [FLAG_REMOVED];
613
614 *(object_copy *)this = src; 615 *(object_copy *)dst = *this;
615
616 flag [FLAG_FREED] = is_freed;
617 flag [FLAG_REMOVED] = is_removed; 616 dst->flag [FLAG_REMOVED] = true;
618 617
619 /* Copy over key_values, if any. */ 618 /* Copy over key_values, if any. */
620 if (src.key_values) 619 if (key_values)
621 { 620 {
622 key_value *tail = 0; 621 key_value *tail = 0;
623 key_values = 0; 622 dst->key_values = 0;
624 623
625 for (key_value *i = src.key_values; i; i = i->next) 624 for (key_value *i = key_values; i; i = i->next)
626 { 625 {
627 key_value *new_link = new key_value; 626 key_value *new_link = new key_value;
628 627
629 new_link->next = 0; 628 new_link->next = 0;
630 new_link->key = i->key; 629 new_link->key = i->key;
631 new_link->value = i->value; 630 new_link->value = i->value;
632 631
633 /* Try and be clever here, too. */ 632 /* Try and be clever here, too. */
634 if (!key_values) 633 if (!dst->key_values)
635 { 634 {
636 key_values = new_link; 635 dst->key_values = new_link;
637 tail = new_link; 636 tail = new_link;
638 } 637 }
639 else 638 else
640 { 639 {
641 tail->next = new_link; 640 tail->next = new_link;
642 tail = new_link; 641 tail = new_link;
643 } 642 }
644 } 643 }
645 } 644 }
646}
647
648/*
649 * copy_to first frees everything allocated by the dst object,
650 * and then copies the contents of itself into the second
651 * object, allocating what needs to be allocated. Basically, any
652 * data that is malloc'd needs to be re-malloc/copied. Otherwise,
653 * if the first object is freed, the pointers in the new object
654 * will point at garbage.
655 */
656void
657object::copy_to (object *dst)
658{
659 *dst = *this;
660 645
661 if (speed < 0) 646 if (speed < 0)
662 dst->speed_left -= rndm (); 647 dst->speed_left -= rndm ();
663 648
664 dst->set_speed (dst->speed); 649 dst->activate ();
665} 650}
666 651
667void 652void
668object::instantiate () 653object::instantiate ()
669{ 654{
686object * 671object *
687object::clone () 672object::clone ()
688{ 673{
689 object *neu = create (); 674 object *neu = create ();
690 copy_to (neu); 675 copy_to (neu);
676 neu->map = map; // not copied by copy_to
691 return neu; 677 return neu;
692} 678}
693 679
694/* 680/*
695 * If an object with the IS_TURNABLE() flag needs to be turned due 681 * If an object with the IS_TURNABLE() flag needs to be turned due
780 else if (action == UP_OBJ_INSERT) 766 else if (action == UP_OBJ_INSERT)
781 { 767 {
782 // this is likely overkill, TODO: revisit (schmorp) 768 // this is likely overkill, TODO: revisit (schmorp)
783 if ((QUERY_FLAG (op, FLAG_BLOCKSVIEW) && !(m.flags_ & P_BLOCKSVIEW)) 769 if ((QUERY_FLAG (op, FLAG_BLOCKSVIEW) && !(m.flags_ & P_BLOCKSVIEW))
784 || (QUERY_FLAG (op, FLAG_NO_MAGIC) && !(m.flags_ & P_NO_MAGIC)) 770 || (QUERY_FLAG (op, FLAG_NO_MAGIC) && !(m.flags_ & P_NO_MAGIC))
785 || (op->type == PLAYER && !(m.flags_ & P_PLAYER)) 771 || (op->is_player () && !(m.flags_ & P_PLAYER))
786 || (op->type == SAFE_GROUND && !(m.flags_ & P_SAFE)) 772 || (op->type == SAFE_GROUND && !(m.flags_ & P_SAFE))
787 || (QUERY_FLAG (op, FLAG_ALIVE) && !(m.flags_ & P_IS_ALIVE)) 773 || (QUERY_FLAG (op, FLAG_ALIVE) && !(m.flags_ & P_IS_ALIVE))
788 || (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC)) 774 || (QUERY_FLAG (op, FLAG_DAMNED) && !(m.flags_ & P_NO_CLERIC))
789 || (m.move_on | op->move_on ) != m.move_on 775 || (m.move_on | op->move_on ) != m.move_on
790 || (m.move_off | op->move_off ) != m.move_off 776 || (m.move_off | op->move_off ) != m.move_off
791 || (m.move_slow | op->move_slow) != m.move_slow 777 || (m.move_slow | op->move_slow) != m.move_slow
792 /* This isn't perfect, but I don't expect a lot of objects to 778 /* This isn't perfect, but I don't expect a lot of objects to
793 * to have move_allow right now. 779 * have move_allow right now.
794 */ 780 */
795 || ((m.move_block | op->move_block) & ~op->move_allow) != m.move_block 781 || ((m.move_block | op->move_block) & ~op->move_allow) != m.move_block
796 || 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp) 782 || 1) // the above is not strong enough a test to skip updating. los maybe? TODO (Schmorp)
797 m.flags_ = 0; 783 m.invalidate ();
798 } 784 }
799 /* if the object is being removed, we can't make intelligent 785 /* if the object is being removed, we can't make intelligent
800 * decisions, because remove_ob can't really pass the object 786 * decisions, because remove_ob can't really pass the object
801 * that is being removed. 787 * that is being removed.
802 */ 788 */
803 else if (action == UP_OBJ_CHANGE || action == UP_OBJ_REMOVE) 789 else if (action == UP_OBJ_CHANGE || action == UP_OBJ_REMOVE)
804 m.flags_ = 0; 790 m.invalidate ();
805 else if (action == UP_OBJ_FACE) 791 else if (action == UP_OBJ_FACE)
806 /* Nothing to do for that case */ ; 792 /* Nothing to do for that case */ ;
807 else 793 else
808 LOG (llevError, "update_object called with invalid action: %d\n", action); 794 LOG (llevError, "update_object called with invalid action: %d\n", action);
809 795
813 799
814object::object () 800object::object ()
815{ 801{
816 SET_FLAG (this, FLAG_REMOVED); 802 SET_FLAG (this, FLAG_REMOVED);
817 803
818 expmul = 1.0; 804 //expmul = 1.0; declared const for the time being
819 face = blank_face; 805 face = blank_face;
820} 806}
821 807
822object::~object () 808object::~object ()
823{ 809{
853 /* If already on active list, don't do anything */ 839 /* If already on active list, don't do anything */
854 if (active) 840 if (active)
855 return; 841 return;
856 842
857 if (has_active_speed ()) 843 if (has_active_speed ())
844 {
845 if (flag [FLAG_FREED])
846 LOG (llevError | logBacktrace, "BUG: tried to activate freed object %s\n", debug_desc ());//D
847
858 actives.insert (this); 848 actives.insert (this);
849 }
859} 850}
860 851
861void 852void
862object::activate_recursive () 853object::activate_recursive ()
863{ 854{
924 * drop on that space. 915 * drop on that space.
925 */ 916 */
926 if (!drop_to_ground 917 if (!drop_to_ground
927 || !map 918 || !map
928 || map->in_memory != MAP_ACTIVE 919 || map->in_memory != MAP_ACTIVE
929 || map->nodrop 920 || map->no_drop
930 || ms ().move_block == MOVE_ALL) 921 || ms ().move_block == MOVE_ALL)
931 { 922 {
932 while (inv) 923 while (inv)
933 {
934 inv->destroy_inv (false);
935 inv->destroy (); 924 inv->destroy ();
936 }
937 } 925 }
938 else 926 else
939 { /* Put objects in inventory onto this space */ 927 { /* Put objects in inventory onto this space */
940 while (inv) 928 while (inv)
941 { 929 {
945 || op->flag [FLAG_NO_DROP] 933 || op->flag [FLAG_NO_DROP]
946 || op->type == RUNE 934 || op->type == RUNE
947 || op->type == TRAP 935 || op->type == TRAP
948 || op->flag [FLAG_IS_A_TEMPLATE] 936 || op->flag [FLAG_IS_A_TEMPLATE]
949 || op->flag [FLAG_DESTROY_ON_DEATH]) 937 || op->flag [FLAG_DESTROY_ON_DEATH])
950 op->destroy (true); 938 op->destroy ();
951 else 939 else
952 map->insert (op, x, y); 940 map->insert (op, x, y);
953 } 941 }
954 } 942 }
955} 943}
959 object *op = new object; 947 object *op = new object;
960 op->link (); 948 op->link ();
961 return op; 949 return op;
962} 950}
963 951
952static struct freed_map : maptile
953{
954 freed_map ()
955 {
956 path = "<freed objects map>";
957 name = "/internal/freed_objects_map";
958 width = 3;
959 height = 3;
960 no_drop = 1;
961 no_reset = 1;
962
963 alloc ();
964 in_memory = MAP_ACTIVE;
965 }
966
967 ~freed_map ()
968 {
969 destroy ();
970 }
971} freed_map; // freed objects are moved here to avoid crashes
972
964void 973void
965object::do_destroy () 974object::do_destroy ()
966{ 975{
967 if (flag [FLAG_IS_LINKED]) 976 if (flag [FLAG_IS_LINKED])
968 remove_button_link (this); 977 remove_link ();
969 978
970 if (flag [FLAG_FRIENDLY]) 979 if (flag [FLAG_FRIENDLY])
971 remove_friendly_object (this); 980 remove_friendly_object (this);
972 981
973 remove (); 982 remove ();
978 unlink (); 987 unlink ();
979 988
980 flag [FLAG_FREED] = 1; 989 flag [FLAG_FREED] = 1;
981 990
982 // hack to ensure that freed objects still have a valid map 991 // hack to ensure that freed objects still have a valid map
983 {
984 static maptile *freed_map; // freed objects are moved here to avoid crashes
985
986 if (!freed_map)
987 {
988 freed_map = new maptile;
989
990 freed_map->path = "<freed objects map>";
991 freed_map->name = "/internal/freed_objects_map";
992 freed_map->width = 3;
993 freed_map->height = 3;
994 freed_map->nodrop = 1;
995
996 freed_map->alloc ();
997 freed_map->in_memory = MAP_ACTIVE;
998 }
999
1000 map = freed_map; 992 map = &freed_map;
1001 x = 1; 993 x = 1;
1002 y = 1; 994 y = 1;
1003 }
1004 995
1005 if (more) 996 if (more)
1006 { 997 {
1007 more->destroy (); 998 more->destroy ();
1008 more = 0; 999 more = 0;
1016 attacked_by = 0; 1007 attacked_by = 0;
1017 current_weapon = 0; 1008 current_weapon = 0;
1018} 1009}
1019 1010
1020void 1011void
1021object::destroy (bool destroy_inventory) 1012object::destroy ()
1022{ 1013{
1023 if (destroyed ()) 1014 if (destroyed ())
1024 return; 1015 return;
1025 1016
1026 if (!is_head () && !head->destroyed ()) 1017 if (!is_head () && !head->destroyed ())
1027 { 1018 {
1028 LOG (llevError | logBacktrace, "tried to destroy the tail of an object"); 1019 LOG (llevError | logBacktrace, "tried to destroy the tail of an object");
1029 head->destroy (destroy_inventory); 1020 head->destroy ();
1021 return;
1030 } 1022 }
1031 1023
1032 destroy_inv (!destroy_inventory); 1024 destroy_inv (false);
1033 1025
1034 if (is_head ()) 1026 if (is_head ())
1035 if (sound_destroy) 1027 if (sound_destroy)
1036 play_sound (sound_destroy); 1028 play_sound (sound_destroy);
1037 else if (flag [FLAG_MONSTER]) 1029 else if (flag [FLAG_MONSTER])
1074 esrv_del_item (pl->contr, count); 1066 esrv_del_item (pl->contr, count);
1075 flag [FLAG_REMOVED] = true; // hack around the issue of visible_to checking flag_removed 1067 flag [FLAG_REMOVED] = true; // hack around the issue of visible_to checking flag_removed
1076 1068
1077 adjust_weight (env, -total_weight ()); 1069 adjust_weight (env, -total_weight ());
1078 1070
1079 *(above ? &above->below : &env->inv) = below; 1071 object *pl = in_player ();
1080
1081 if (below)
1082 below->above = above;
1083 1072
1084 /* we set up values so that it could be inserted into 1073 /* we set up values so that it could be inserted into
1085 * the map, but we don't actually do that - it is up 1074 * the map, but we don't actually do that - it is up
1086 * to the caller to decide what we want to do. 1075 * to the caller to decide what we want to do.
1087 */ 1076 */
1088 map = env->map; 1077 map = env->map;
1089 x = env->x; 1078 x = env->x;
1090 y = env->y; 1079 y = env->y;
1080
1081 // make sure cmov optimisation is applicable
1082 *(above ? &above->below : &env->inv) = below;
1083 *(below ? &below->above : &above ) = above; // &above is just a dummy
1084
1091 above = 0; 1085 above = 0;
1092 below = 0; 1086 below = 0;
1093 env = 0; 1087 env = 0;
1094 1088
1095 /* NO_FIX_PLAYER is set when a great many changes are being 1089 /* NO_FIX_PLAYER is set when a great many changes are being
1096 * made to players inventory. If set, avoiding the call 1090 * made to players inventory. If set, avoiding the call
1097 * to save cpu time. 1091 * to save cpu time.
1098 */ 1092 */
1099 if ((otmp = in_player ()) && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER)) 1093 if (pl)
1094 if (pl->is_player () && (glow_radius || !QUERY_FLAG (pl, FLAG_NO_FIX_PLAYER)))
1095 {
1100 otmp->update_stats (); 1096 pl->update_stats ();
1097
1098 if (glow_radius && pl->is_on_map ())
1099 update_all_los (pl->map, pl->x, pl->y);
1100 }
1101 } 1101 }
1102 else if (map) 1102 else if (map)
1103 { 1103 {
1104 map->dirty = true; 1104 map->dirty = true;
1105 mapspace &ms = this->ms (); 1105 mapspace &ms = this->ms ();
1106 1106
1107 if (object *pl = ms.player ()) 1107 if (object *pl = ms.player ())
1108 { 1108 {
1109 if (type == PLAYER) // this == pl(!) 1109 if (is_player ())
1110 { 1110 {
1111 if (!flag [FLAG_WIZPASS])
1112 ms.smell = ++mapspace::smellcount; // remember the smell of the player
1113
1111 // leaving a spot always closes any open container on the ground 1114 // leaving a spot always closes any open container on the ground
1112 if (container && !container->env) 1115 if (container && !container->env)
1113 // this causes spurious floorbox updates, but it ensures 1116 // this causes spurious floorbox updates, but it ensures
1114 // that the CLOSE event is being sent. 1117 // that the CLOSE event is being sent.
1115 close_container (); 1118 close_container ();
1125 1128
1126 esrv_del_item (pl->contr, count); 1129 esrv_del_item (pl->contr, count);
1127 } 1130 }
1128 1131
1129 /* link the object above us */ 1132 /* link the object above us */
1130 if (above) 1133 // re-link, make sure compiler can easily use cmove
1131 above->below = below; 1134 *(above ? &above->below : &ms.top) = below;
1132 else 1135 *(below ? &below->above : &ms.bot) = above;
1133 ms.top = below; /* we were top, set new top */
1134
1135 /* Relink the object below us, if there is one */
1136 if (below)
1137 below->above = above;
1138 else
1139 {
1140 /* Nothing below, which means we need to relink map object for this space
1141 * use translated coordinates in case some oddness with map tiling is
1142 * evident
1143 */
1144 if (GET_MAP_OB (map, x, y) != this)
1145 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 ());
1146
1147 ms.bot = above; /* goes on above it. */
1148 }
1149 1136
1150 above = 0; 1137 above = 0;
1151 below = 0; 1138 below = 0;
1139
1140 ms.invalidate ();
1152 1141
1153 if (map->in_memory == MAP_SAVING) 1142 if (map->in_memory == MAP_SAVING)
1154 return; 1143 return;
1155 1144
1156 int check_walk_off = !flag [FLAG_NO_APPLY]; 1145 int check_walk_off = !flag [FLAG_NO_APPLY];
1163 * appropriately. 1152 * appropriately.
1164 */ 1153 */
1165 pl->close_container (); 1154 pl->close_container ();
1166 1155
1167 //TODO: the floorbox prev/next might need updating 1156 //TODO: the floorbox prev/next might need updating
1168 esrv_del_item (pl->contr, count); 1157 //esrv_del_item (pl->contr, count);
1158 //TODO: update floorbox to preserve ordering
1159 if (pl->contr->ns)
1160 pl->contr->ns->floorbox_update ();
1169 } 1161 }
1170 1162
1171 for (tmp = ms.bot; tmp; tmp = tmp->above) 1163 for (tmp = ms.bot; tmp; tmp = tmp->above)
1172 { 1164 {
1173 /* No point updating the players look faces if he is the object 1165 /* No point updating the players look faces if he is the object
1186 } 1178 }
1187 1179
1188 last = tmp; 1180 last = tmp;
1189 } 1181 }
1190 1182
1191 /* last == NULL if there are no objects on this space */ 1183 if (affects_los ())
1192 //TODO: this makes little sense, why only update the topmost object?
1193 if (!last)
1194 map->at (x, y).flags_ = 0;
1195 else
1196 update_object (last, UP_OBJ_REMOVE);
1197
1198 if (flag [FLAG_BLOCKSVIEW] || glow_radius)
1199 update_all_los (map, x, y); 1184 update_all_los (map, x, y);
1200 } 1185 }
1201} 1186}
1202 1187
1203/* 1188/*
1227 esrv_update_item (UPD_NROF, pl, top); 1212 esrv_update_item (UPD_NROF, pl, top);
1228 1213
1229 op->weight = 0; // cancel the addition above 1214 op->weight = 0; // cancel the addition above
1230 op->carrying = 0; // must be 0 already 1215 op->carrying = 0; // must be 0 already
1231 1216
1232 op->destroy (1); 1217 op->destroy ();
1233 1218
1234 return top; 1219 return top;
1235 } 1220 }
1236 1221
1237 return 0; 1222 return 0;
1265 * job preparing multi-part monsters. 1250 * job preparing multi-part monsters.
1266 */ 1251 */
1267object * 1252object *
1268insert_ob_in_map_at (object *op, maptile *m, object *originator, int flag, int x, int y) 1253insert_ob_in_map_at (object *op, maptile *m, object *originator, int flag, int x, int y)
1269{ 1254{
1255 op->remove ();
1256
1270 for (object *tmp = op->head_ (); tmp; tmp = tmp->more) 1257 for (object *tmp = op->head_ (); tmp; tmp = tmp->more)
1271 { 1258 {
1272 tmp->x = x + tmp->arch->x; 1259 tmp->x = x + tmp->arch->x;
1273 tmp->y = y + tmp->arch->y; 1260 tmp->y = y + tmp->arch->y;
1274 } 1261 }
1297 * just 'op' otherwise 1284 * just 'op' otherwise
1298 */ 1285 */
1299object * 1286object *
1300insert_ob_in_map (object *op, maptile *m, object *originator, int flag) 1287insert_ob_in_map (object *op, maptile *m, object *originator, int flag)
1301{ 1288{
1302 assert (!op->flag [FLAG_FREED]);
1303
1304 op->remove (); 1289 op->remove ();
1290
1291 if (m == &freed_map)//D TODO: remove soon
1292 {//D
1293 LOG (llevError | logBacktrace, "tries to insret object on freed objects map: %s", op->debug_desc ());//D
1294 }//D
1305 1295
1306 /* Ideally, the caller figures this out. However, it complicates a lot 1296 /* Ideally, the caller figures this out. However, it complicates a lot
1307 * of areas of callers (eg, anything that uses find_free_spot would now 1297 * of areas of callers (eg, anything that uses find_free_spot would now
1308 * need extra work 1298 * need extra work
1309 */ 1299 */
1300 maptile *newmap = m;
1310 if (!xy_normalise (m, op->x, op->y)) 1301 if (!xy_normalise (newmap, op->x, op->y))
1311 { 1302 {
1312 op->destroy (1); 1303 op->head_ ()->destroy ();// remove head_ once all tail object destroyers found
1313 return 0; 1304 return 0;
1314 } 1305 }
1315 1306
1316 if (object *more = op->more) 1307 if (object *more = op->more)
1317 if (!insert_ob_in_map (more, m, originator, flag)) 1308 if (!insert_ob_in_map (more, m, originator, flag))
1318 return 0; 1309 return 0;
1319 1310
1320 CLEAR_FLAG (op, FLAG_REMOVED); 1311 op->flag [FLAG_REMOVED] = false;
1321 1312 op->env = 0;
1322 op->map = m; 1313 op->map = newmap;
1314
1323 mapspace &ms = op->ms (); 1315 mapspace &ms = op->ms ();
1324 1316
1325 /* this has to be done after we translate the coordinates. 1317 /* this has to be done after we translate the coordinates.
1326 */ 1318 */
1327 if (op->nrof && !(flag & INS_NO_MERGE)) 1319 if (op->nrof && !(flag & INS_NO_MERGE))
1328 for (object *tmp = ms.bot; tmp; tmp = tmp->above) 1320 for (object *tmp = ms.bot; tmp; tmp = tmp->above)
1329 if (object::can_merge (op, tmp)) 1321 if (object::can_merge (op, tmp))
1330 { 1322 {
1331 // TODO: we atcually want to update tmp, not op, 1323 // TODO: we actually want to update tmp, not op,
1332 // but some caller surely breaks when we return tmp 1324 // but some caller surely breaks when we return tmp
1333 // from here :/ 1325 // from here :/
1334 op->nrof += tmp->nrof; 1326 op->nrof += tmp->nrof;
1335 tmp->destroy (1); 1327 tmp->destroy ();
1336 } 1328 }
1337 1329
1338 CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */ 1330 CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */
1339 CLEAR_FLAG (op, FLAG_INV_LOCKED); 1331 CLEAR_FLAG (op, FLAG_INV_LOCKED);
1340 1332
1347 { 1339 {
1348 LOG (llevError, "insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n"); 1340 LOG (llevError, "insert_ob_in_map called with INS_BELOW_ORIGINATOR when originator not on same space!\n");
1349 abort (); 1341 abort ();
1350 } 1342 }
1351 1343
1344 if (!originator->is_on_map ())
1345 {
1346 LOG (llevError, "insert_ob_in_map(%s) called with INS_BELOW_ORIGINATOR when originator '%s' not on map",
1347 op->debug_desc (), originator->debug_desc ());
1348 abort ();
1349 }
1350
1352 op->above = originator; 1351 op->above = originator;
1353 op->below = originator->below; 1352 op->below = originator->below;
1354
1355 if (op->below)
1356 op->below->above = op;
1357 else
1358 ms.bot = op;
1359
1360 /* since *below* originator, no need to update top */
1361 originator->below = op; 1353 originator->below = op;
1354
1355 *(op->below ? &op->below->above : &ms.bot) = op;
1362 } 1356 }
1363 else 1357 else
1364 { 1358 {
1365 object *top, *floor = NULL; 1359 object *floor = 0;
1366 1360 object *top = ms.top;
1367 top = ms.bot;
1368 1361
1369 /* If there are other objects, then */ 1362 /* If there are other objects, then */
1370 if (top) 1363 if (top)
1371 { 1364 {
1372 object *last = 0;
1373
1374 /* 1365 /*
1375 * If there are multiple objects on this space, we do some trickier handling. 1366 * If there are multiple objects on this space, we do some trickier handling.
1376 * We've already dealt with merging if appropriate. 1367 * We've already dealt with merging if appropriate.
1377 * Generally, we want to put the new object on top. But if 1368 * Generally, we want to put the new object on top. But if
1378 * flag contains INS_ABOVE_FLOOR_ONLY, once we find the last 1369 * flag contains INS_ABOVE_FLOOR_ONLY, once we find the last
1381 * once we get to them. This reduces the need to traverse over all of 1372 * once we get to them. This reduces the need to traverse over all of
1382 * them when adding another one - this saves quite a bit of cpu time 1373 * them when adding another one - this saves quite a bit of cpu time
1383 * when lots of spells are cast in one area. Currently, it is presumed 1374 * when lots of spells are cast in one area. Currently, it is presumed
1384 * that flying non pickable objects are spell objects. 1375 * that flying non pickable objects are spell objects.
1385 */ 1376 */
1386 for (top = ms.bot; top; top = top->above) 1377 for (object *tmp = ms.bot; tmp; tmp = tmp->above)
1387 { 1378 {
1388 if (QUERY_FLAG (top, FLAG_IS_FLOOR) || QUERY_FLAG (top, FLAG_OVERLAY_FLOOR)) 1379 if (QUERY_FLAG (tmp, FLAG_IS_FLOOR) || QUERY_FLAG (tmp, FLAG_OVERLAY_FLOOR))
1389 floor = top; 1380 floor = tmp;
1390 1381
1391 if (QUERY_FLAG (top, FLAG_NO_PICK) && (top->move_type & (MOVE_FLY_LOW | MOVE_FLY_HIGH)) && !QUERY_FLAG (top, FLAG_IS_FLOOR)) 1382 if (QUERY_FLAG (tmp, FLAG_NO_PICK) && (tmp->move_type & (MOVE_FLY_LOW | MOVE_FLY_HIGH)) && !QUERY_FLAG (tmp, FLAG_IS_FLOOR))
1392 { 1383 {
1393 /* We insert above top, so we want this object below this */ 1384 /* We insert above top, so we want this object below this */
1394 top = top->below; 1385 top = tmp->below;
1395 break; 1386 break;
1396 } 1387 }
1397 1388
1398 last = top; 1389 top = tmp;
1399 } 1390 }
1400
1401 /* Don't want top to be NULL, so set it to the last valid object */
1402 top = last;
1403 1391
1404 /* We let update_position deal with figuring out what the space 1392 /* We let update_position deal with figuring out what the space
1405 * looks like instead of lots of conditions here. 1393 * looks like instead of lots of conditions here.
1406 * makes things faster, and effectively the same result. 1394 * makes things faster, and effectively the same result.
1407 */ 1395 */
1414 */ 1402 */
1415 if (!(flag & INS_ON_TOP) 1403 if (!(flag & INS_ON_TOP)
1416 && ms.flags () & P_BLOCKSVIEW 1404 && ms.flags () & P_BLOCKSVIEW
1417 && (op->face && !faces [op->face].visibility)) 1405 && (op->face && !faces [op->face].visibility))
1418 { 1406 {
1407 object *last;
1408
1419 for (last = top; last != floor; last = last->below) 1409 for (last = top; last != floor; last = last->below)
1420 if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT)) 1410 if (QUERY_FLAG (last, FLAG_BLOCKSVIEW) && (last->type != EXIT))
1421 break; 1411 break;
1422 1412
1423 /* Check to see if we found the object that blocks view, 1413 /* Check to see if we found the object that blocks view,
1431 } /* If objects on this space */ 1421 } /* If objects on this space */
1432 1422
1433 if (flag & INS_ABOVE_FLOOR_ONLY) 1423 if (flag & INS_ABOVE_FLOOR_ONLY)
1434 top = floor; 1424 top = floor;
1435 1425
1436 /* Top is the object that our object (op) is going to get inserted above. 1426 // insert object above top, or bottom-most if top = 0
1437 */
1438
1439 /* First object on this space */
1440 if (!top) 1427 if (!top)
1441 { 1428 {
1429 op->below = 0;
1442 op->above = ms.bot; 1430 op->above = ms.bot;
1443
1444 if (op->above)
1445 op->above->below = op;
1446
1447 op->below = 0;
1448 ms.bot = op; 1431 ms.bot = op;
1432
1433 *(op->above ? &op->above->below : &ms.top) = op;
1449 } 1434 }
1450 else 1435 else
1451 { /* get inserted into the stack above top */ 1436 {
1452 op->above = top->above; 1437 op->above = top->above;
1453
1454 if (op->above)
1455 op->above->below = op; 1438 top->above = op;
1456 1439
1457 op->below = top; 1440 op->below = top;
1458 top->above = op; 1441 *(op->above ? &op->above->below : &ms.top) = op;
1459 } 1442 }
1443 }
1460 1444
1461 if (!op->above) 1445 if (op->is_player ())
1462 ms.top = op;
1463 } /* else not INS_BELOW_ORIGINATOR */
1464
1465 if (op->type == PLAYER)
1466 { 1446 {
1467 op->contr->do_los = 1; 1447 op->contr->do_los = 1;
1468 ++op->map->players; 1448 ++op->map->players;
1469 op->map->touch (); 1449 op->map->touch ();
1470 } 1450 }
1471 1451
1472 op->map->dirty = true; 1452 op->map->dirty = true;
1473 1453
1474 if (object *pl = ms.player ()) 1454 if (object *pl = ms.player ())
1475 //TODO: the floorbox prev/next might need updating 1455 //TODO: the floorbox prev/next might need updating
1476 esrv_send_item (pl, op); 1456 //esrv_send_item (pl, op);
1457 //TODO: update floorbox to preserve ordering
1458 if (pl->contr->ns)
1459 pl->contr->ns->floorbox_update ();
1477 1460
1478 /* If this object glows, it may affect lighting conditions that are 1461 /* If this object glows, it may affect lighting conditions that are
1479 * visible to others on this map. But update_all_los is really 1462 * visible to others on this map. But update_all_los is really
1480 * an inefficient way to do this, as it means los for all players 1463 * an inefficient way to do this, as it means los for all players
1481 * on the map will get recalculated. The players could very well 1464 * on the map will get recalculated. The players could very well
1482 * be far away from this change and not affected in any way - 1465 * be far away from this change and not affected in any way -
1483 * this should get redone to only look for players within range, 1466 * this should get redone to only look for players within range,
1484 * or just updating the P_UPTODATE for spaces within this area 1467 * or just updating the P_UPTODATE for spaces within this area
1485 * of effect may be sufficient. 1468 * of effect may be sufficient.
1486 */ 1469 */
1487 if (op->map->darkness && (op->glow_radius != 0)) 1470 if (op->affects_los ())
1471 {
1472 op->ms ().invalidate ();
1488 update_all_los (op->map, op->x, op->y); 1473 update_all_los (op->map, op->x, op->y);
1474 }
1489 1475
1490 /* updates flags (blocked, alive, no magic, etc) for this map space */ 1476 /* updates flags (blocked, alive, no magic, etc) for this map space */
1491 update_object (op, UP_OBJ_INSERT); 1477 update_object (op, UP_OBJ_INSERT);
1492 1478
1493 INVOKE_OBJECT (INSERT, op); 1479 INVOKE_OBJECT (INSERT, op);
1500 * blocked() and wall() work properly), and these flags are updated by 1486 * blocked() and wall() work properly), and these flags are updated by
1501 * update_object(). 1487 * update_object().
1502 */ 1488 */
1503 1489
1504 /* if this is not the head or flag has been passed, don't check walk on status */ 1490 /* if this is not the head or flag has been passed, don't check walk on status */
1505 if (!(flag & INS_NO_WALK_ON) && op->head_ () == op) 1491 if (!(flag & INS_NO_WALK_ON) && op->is_head ())
1506 { 1492 {
1507 if (check_move_on (op, originator)) 1493 if (check_move_on (op, originator))
1508 return 0; 1494 return 0;
1509 1495
1510 /* If we are a multi part object, lets work our way through the check 1496 /* If we are a multi part object, lets work our way through the check
1521/* this function inserts an object in the map, but if it 1507/* this function inserts an object in the map, but if it
1522 * finds an object of its own type, it'll remove that one first. 1508 * finds an object of its own type, it'll remove that one first.
1523 * op is the object to insert it under: supplies x and the map. 1509 * op is the object to insert it under: supplies x and the map.
1524 */ 1510 */
1525void 1511void
1526replace_insert_ob_in_map (const char *arch_string, object *op) 1512replace_insert_ob_in_map (shstr_tmp archname, object *op)
1527{ 1513{
1528 /* first search for itself and remove any old instances */ 1514 /* first search for itself and remove any old instances */
1529 1515
1530 for (object *tmp = op->ms ().bot; tmp; tmp = tmp->above) 1516 for (object *tmp = op->ms ().bot; tmp; tmp = tmp->above)
1531 if (!strcmp (tmp->arch->archname, arch_string)) /* same archetype */ 1517 if (tmp->arch->archname == archname) /* same archetype */
1532 tmp->destroy (1); 1518 tmp->destroy ();
1533 1519
1534 object *tmp = arch_to_object (archetype::find (arch_string)); 1520 object *tmp = arch_to_object (archetype::find (archname));
1535 1521
1536 tmp->x = op->x; 1522 tmp->x = op->x;
1537 tmp->y = op->y; 1523 tmp->y = op->y;
1538 1524
1539 insert_ob_in_map (tmp, op->map, op, 0); 1525 insert_ob_in_map (tmp, op->map, op, 0);
1561 if (!nr) 1547 if (!nr)
1562 return true; 1548 return true;
1563 1549
1564 nr = min (nr, nrof); 1550 nr = min (nr, nrof);
1565 1551
1552 if (nrof > nr)
1553 {
1566 nrof -= nr; 1554 nrof -= nr;
1567
1568 if (nrof)
1569 {
1570 adjust_weight (env, -weight * nr); // carrying == 0 1555 adjust_weight (env, -weight * max (1, nr)); // carrying == 0
1571 1556
1572 if (object *pl = visible_to ()) 1557 if (object *pl = visible_to ())
1573 esrv_update_item (UPD_NROF, pl, this); 1558 esrv_update_item (UPD_NROF, pl, this);
1574 1559
1575 return true; 1560 return true;
1576 } 1561 }
1577 else 1562 else
1578 { 1563 {
1579 destroy (1); 1564 destroy ();
1580 return false; 1565 return false;
1581 } 1566 }
1582} 1567}
1583 1568
1584/* 1569/*
1601 } 1586 }
1602 else 1587 else
1603 { 1588 {
1604 decrease (nr); 1589 decrease (nr);
1605 1590
1606 object *op = object_create_clone (this); 1591 object *op = deep_clone ();
1607 op->nrof = nr; 1592 op->nrof = nr;
1608 return op; 1593 return op;
1609 } 1594 }
1610} 1595}
1611 1596
1661 if (object *pl = tmp->visible_to ()) 1646 if (object *pl = tmp->visible_to ())
1662 esrv_update_item (UPD_NROF, pl, tmp); 1647 esrv_update_item (UPD_NROF, pl, tmp);
1663 1648
1664 adjust_weight (this, op->total_weight ()); 1649 adjust_weight (this, op->total_weight ());
1665 1650
1666 op->destroy (1); 1651 op->destroy ();
1667 op = tmp; 1652 op = tmp;
1668 goto inserted; 1653 goto inserted;
1669 } 1654 }
1670 1655
1671 op->owner = 0; // it's his/hers now. period. 1656 op->owner = 0; // it's his/hers now. period.
1689 1674
1690 adjust_weight (this, op->total_weight ()); 1675 adjust_weight (this, op->total_weight ());
1691 1676
1692inserted: 1677inserted:
1693 /* reset the light list and los of the players on the map */ 1678 /* reset the light list and los of the players on the map */
1694 if (op->glow_radius && map && map->darkness) 1679 if (op->glow_radius && is_on_map ())
1680 {
1681 update_stats ();
1695 update_all_los (map, x, y); 1682 update_all_los (map, x, y);
1696 1683 }
1684 else if (is_player () && !flag [FLAG_NO_FIX_PLAYER])
1697 // if this is a player's inventory, update stats 1685 // if this is a player's inventory, update stats
1698 if (type == PLAYER && !flag [FLAG_NO_FIX_PLAYER])
1699 update_stats (); 1686 update_stats ();
1700 1687
1701 INVOKE_OBJECT (INSERT, this); 1688 INVOKE_OBJECT (INSERT, this);
1702 1689
1703 return op; 1690 return op;
1724 * on top. 1711 * on top.
1725 */ 1712 */
1726int 1713int
1727check_move_on (object *op, object *originator) 1714check_move_on (object *op, object *originator)
1728{ 1715{
1716 if (QUERY_FLAG (op, FLAG_NO_APPLY))
1717 return 0;
1718
1729 object *tmp; 1719 object *tmp;
1730 maptile *m = op->map; 1720 maptile *m = op->map;
1731 int x = op->x, y = op->y; 1721 int x = op->x, y = op->y;
1732 1722
1733 MoveType move_on, move_slow, move_block; 1723 mapspace &ms = m->at (x, y);
1734 1724
1735 if (QUERY_FLAG (op, FLAG_NO_APPLY)) 1725 ms.update ();
1736 return 0;
1737 1726
1738 move_on = GET_MAP_MOVE_ON (op->map, op->x, op->y); 1727 MoveType move_on = ms.move_on;
1739 move_slow = GET_MAP_MOVE_SLOW (op->map, op->x, op->y); 1728 MoveType move_slow = ms.move_slow;
1740 move_block = GET_MAP_MOVE_BLOCK (op->map, op->x, op->y); 1729 MoveType move_block = ms.move_block;
1741 1730
1742 /* if nothing on this space will slow op down or be applied, 1731 /* if nothing on this space will slow op down or be applied,
1743 * no need to do checking below. have to make sure move_type 1732 * no need to do checking below. have to make sure move_type
1744 * is set, as lots of objects don't have it set - we treat that 1733 * is set, as lots of objects don't have it set - we treat that
1745 * as walking. 1734 * as walking.
1756 return 0; 1745 return 0;
1757 1746
1758 /* The objects have to be checked from top to bottom. 1747 /* The objects have to be checked from top to bottom.
1759 * Hence, we first go to the top: 1748 * Hence, we first go to the top:
1760 */ 1749 */
1761 1750 for (object *next, *tmp = ms.top; tmp; tmp = next)
1762 for (tmp = op->ms ().bot; tmp && tmp->above; tmp = tmp->above)
1763 {
1764 /* Trim the search when we find the first other spell effect
1765 * this helps performance so that if a space has 50 spell objects,
1766 * we don't need to check all of them.
1767 */
1768 if ((tmp->move_type & MOVE_FLY_LOW) && QUERY_FLAG (tmp, FLAG_NO_PICK))
1769 break;
1770 } 1751 {
1752 next = tmp->below;
1771 1753
1772 for (; tmp; tmp = tmp->below)
1773 {
1774 if (tmp == op) 1754 if (tmp == op)
1775 continue; /* Can't apply yourself */ 1755 continue; /* Can't apply yourself */
1776 1756
1777 /* Check to see if one of the movement types should be slowed down. 1757 /* Check to see if one of the movement types should be slowed down.
1778 * Second check makes sure that the movement types not being slowed 1758 * Second check makes sure that the movement types not being slowed
1783 if (!QUERY_FLAG (op, FLAG_WIZPASS)) 1763 if (!QUERY_FLAG (op, FLAG_WIZPASS))
1784 { 1764 {
1785 if ((!op->move_type && tmp->move_slow & MOVE_WALK) || 1765 if ((!op->move_type && tmp->move_slow & MOVE_WALK) ||
1786 ((op->move_type & tmp->move_slow) && (op->move_type & ~tmp->move_slow & ~tmp->move_block) == 0)) 1766 ((op->move_type & tmp->move_slow) && (op->move_type & ~tmp->move_slow & ~tmp->move_block) == 0))
1787 { 1767 {
1788
1789 float
1790 diff = tmp->move_slow_penalty * fabs (op->speed); 1768 float diff = tmp->move_slow_penalty * fabs (op->speed);
1791 1769
1792 if (op->type == PLAYER) 1770 if (op->is_player ())
1793 if ((QUERY_FLAG (tmp, FLAG_IS_HILLY) && find_skill_by_number (op, SK_CLIMBING)) || 1771 if ((tmp->flag [FLAG_IS_HILLY ] && find_skill_by_number (op, SK_CLIMBING)) ||
1794 (QUERY_FLAG (tmp, FLAG_IS_WOODED) && find_skill_by_number (op, SK_WOODSMAN))) 1772 (tmp->flag [FLAG_IS_WOODED] && find_skill_by_number (op, SK_WOODSMAN)))
1795 diff /= 4.0; 1773 diff /= 4.0;
1796 1774
1797 op->speed_left -= diff; 1775 op->speed_left -= diff;
1798 } 1776 }
1799 } 1777 }
1832 LOG (llevError, "Present_arch called outside map.\n"); 1810 LOG (llevError, "Present_arch called outside map.\n");
1833 return NULL; 1811 return NULL;
1834 } 1812 }
1835 1813
1836 for (object *tmp = m->at (x, y).bot; tmp; tmp = tmp->above) 1814 for (object *tmp = m->at (x, y).bot; tmp; tmp = tmp->above)
1837 if (tmp->arch == at) 1815 if (tmp->arch->archname == at->archname)
1838 return tmp; 1816 return tmp;
1839 1817
1840 return NULL; 1818 return NULL;
1841} 1819}
1842 1820
1906 * The first matching object is returned, or NULL if none. 1884 * The first matching object is returned, or NULL if none.
1907 */ 1885 */
1908object * 1886object *
1909present_arch_in_ob (const archetype *at, const object *op) 1887present_arch_in_ob (const archetype *at, const object *op)
1910{ 1888{
1911 for (object *tmp = op->inv; tmp != NULL; tmp = tmp->below) 1889 for (object *tmp = op->inv; tmp; tmp = tmp->below)
1912 if (tmp->arch == at) 1890 if (tmp->arch->archname == at->archname)
1913 return tmp; 1891 return tmp;
1914 1892
1915 return NULL; 1893 return NULL;
1916} 1894}
1917 1895
2005 * head of the object should correspond for the entire object. 1983 * head of the object should correspond for the entire object.
2006 */ 1984 */
2007 if (OB_TYPE_MOVE_BLOCK (ob, ms.move_block)) 1985 if (OB_TYPE_MOVE_BLOCK (ob, ms.move_block))
2008 continue; 1986 continue;
2009 1987
2010 if (ob->blocked (m, pos.x, pos.y)) 1988 if (ob->blocked (pos.m, pos.x, pos.y))
2011 continue; 1989 continue;
2012 1990
2013 altern [index++] = i; 1991 altern [index++] = i;
2014 } 1992 }
2015 1993
2083 * there is capable of. 2061 * there is capable of.
2084 */ 2062 */
2085int 2063int
2086find_dir (maptile *m, int x, int y, object *exclude) 2064find_dir (maptile *m, int x, int y, object *exclude)
2087{ 2065{
2088 int i, max = SIZEOFFREE, mflags; 2066 int max = SIZEOFFREE, mflags;
2089
2090 sint16 nx, ny;
2091 object *tmp;
2092 maptile *mp;
2093
2094 MoveType blocked, move_type; 2067 MoveType move_type;
2095 2068
2096 if (exclude && exclude->head_ () != exclude) 2069 if (exclude && exclude->head_ () != exclude)
2097 { 2070 {
2098 exclude = exclude->head; 2071 exclude = exclude->head;
2099 move_type = exclude->move_type; 2072 move_type = exclude->move_type;
2102 { 2075 {
2103 /* If we don't have anything, presume it can use all movement types. */ 2076 /* If we don't have anything, presume it can use all movement types. */
2104 move_type = MOVE_ALL; 2077 move_type = MOVE_ALL;
2105 } 2078 }
2106 2079
2107 for (i = 1; i < max; i++) 2080 for (int i = 1; i < max; i++)
2108 { 2081 {
2109 mp = m; 2082 mapxy pos (m, x, y);
2110 nx = x + freearr_x[i]; 2083 pos.move (i);
2111 ny = y + freearr_y[i];
2112 2084
2113 mflags = get_map_flags (m, &mp, nx, ny, &nx, &ny); 2085 if (!pos.normalise ())
2114
2115 if (mflags & P_OUT_OF_MAP)
2116 max = maxfree[i]; 2086 max = maxfree[i];
2117 else 2087 else
2118 { 2088 {
2119 mapspace &ms = mp->at (nx, ny); 2089 mapspace &ms = *pos;
2120 2090
2121 blocked = ms.move_block;
2122
2123 if ((move_type & blocked) == move_type) 2091 if ((move_type & ms.move_block) == move_type)
2124 max = maxfree[i]; 2092 max = maxfree [i];
2125 else if (mflags & P_IS_ALIVE) 2093 else if (ms.flags () & P_IS_ALIVE)
2126 { 2094 {
2127 for (tmp = ms.bot; tmp; tmp = tmp->above) 2095 for (object *tmp = ms.bot; tmp; tmp = tmp->above)
2128 if ((tmp->flag [FLAG_MONSTER] || tmp->type == PLAYER) 2096 if ((tmp->flag [FLAG_MONSTER] || tmp->is_player ())
2129 && (tmp != exclude || (tmp->head_ () != tmp && tmp->head_ () != exclude))) 2097 && (tmp != exclude || (tmp->head_ () != tmp && tmp->head_ () != exclude)))
2130 break;
2131
2132 if (tmp)
2133 return freedir[i]; 2098 return freedir [i];
2134 } 2099 }
2135 } 2100 }
2136 } 2101 }
2137 2102
2138 return 0; 2103 return 0;
2213 * This basically means that if direction is 15, then it could either go 2178 * This basically means that if direction is 15, then it could either go
2214 * direction 4, 14, or 16 to get back to where we are. 2179 * direction 4, 14, or 16 to get back to where we are.
2215 * Moved from spell_util.c to object.c with the other related direction 2180 * Moved from spell_util.c to object.c with the other related direction
2216 * functions. 2181 * functions.
2217 */ 2182 */
2218int reduction_dir[SIZEOFFREE][3] = { 2183const int reduction_dir[SIZEOFFREE][3] = {
2219 {0, 0, 0}, /* 0 */ 2184 {0, 0, 0}, /* 0 */
2220 {0, 0, 0}, /* 1 */ 2185 {0, 0, 0}, /* 1 */
2221 {0, 0, 0}, /* 2 */ 2186 {0, 0, 0}, /* 2 */
2222 {0, 0, 0}, /* 3 */ 2187 {0, 0, 0}, /* 3 */
2223 {0, 0, 0}, /* 4 */ 2188 {0, 0, 0}, /* 4 */
2319int 2284int
2320can_pick (const object *who, const object *item) 2285can_pick (const object *who, const object *item)
2321{ 2286{
2322 return /*QUERY_FLAG(who,FLAG_WIZ)|| */ 2287 return /*QUERY_FLAG(who,FLAG_WIZ)|| */
2323 (item->weight > 0 && !QUERY_FLAG (item, FLAG_NO_PICK) && 2288 (item->weight > 0 && !QUERY_FLAG (item, FLAG_NO_PICK) &&
2324 !QUERY_FLAG (item, FLAG_ALIVE) && !item->invisible && (who->type == PLAYER || item->weight < who->weight / 3)); 2289 !QUERY_FLAG (item, FLAG_ALIVE) && !item->invisible && (who->is_player () || item->weight < who->weight / 3));
2325} 2290}
2326 2291
2327/* 2292/*
2328 * create clone from object to another 2293 * create clone from object to another
2329 */ 2294 */
2330object * 2295object *
2331object_create_clone (object *asrc) 2296object::deep_clone ()
2332{ 2297{
2333 object *dst = 0, *tmp, *src, *prev, *item; 2298 assert (("deep_clone called on non-head object", is_head ()));
2334 2299
2335 if (!asrc) 2300 object *dst = clone ();
2336 return 0;
2337 2301
2338 src = asrc->head_ (); 2302 object *prev = dst;
2339
2340 prev = 0;
2341 for (object *part = src; part; part = part->more) 2303 for (object *part = this->more; part; part = part->more)
2342 { 2304 {
2343 tmp = part->clone (); 2305 object *tmp = part->clone ();
2344 tmp->x -= src->x;
2345 tmp->y -= src->y;
2346
2347 if (!part->head)
2348 {
2349 dst = tmp;
2350 tmp->head = 0;
2351 }
2352 else
2353 tmp->head = dst; 2306 tmp->head = dst;
2354
2355 tmp->more = 0;
2356
2357 if (prev)
2358 prev->more = tmp; 2307 prev->more = tmp;
2359
2360 prev = tmp; 2308 prev = tmp;
2361 } 2309 }
2362 2310
2363 for (item = src->inv; item; item = item->below) 2311 for (object *item = inv; item; item = item->below)
2364 insert_ob_in_ob (object_create_clone (item), dst); 2312 insert_ob_in_ob (item->deep_clone (), dst);
2365 2313
2366 return dst; 2314 return dst;
2367} 2315}
2368 2316
2369/* This returns the first object in who's inventory that 2317/* This returns the first object in who's inventory that
2378 return tmp; 2326 return tmp;
2379 2327
2380 return 0; 2328 return 0;
2381} 2329}
2382 2330
2383/* If ob has a field named key, return the link from the list, 2331shstr_tmp
2384 * otherwise return NULL. 2332object::kv_get (shstr_tmp key) const
2385 *
2386 * key must be a passed in shared string - otherwise, this won't
2387 * do the desired thing.
2388 */
2389key_value *
2390get_ob_key_link (const object *ob, const char *key)
2391{ 2333{
2392 for (key_value *link = ob->key_values; link; link = link->next) 2334 for (key_value *kv = key_values; kv; kv = kv->next)
2393 if (link->key == key) 2335 if (kv->key == key)
2394 return link;
2395
2396 return 0;
2397}
2398
2399/*
2400 * Returns the value of op has an extra_field for key, or NULL.
2401 *
2402 * The argument doesn't need to be a shared string.
2403 *
2404 * The returned string is shared.
2405 */
2406const char *
2407get_ob_key_value (const object *op, const char *const key)
2408{
2409 key_value *link;
2410 shstr_cmp canonical_key (key);
2411
2412 if (!canonical_key)
2413 {
2414 /* 1. There being a field named key on any object
2415 * implies there'd be a shared string to find.
2416 * 2. Since there isn't, no object has this field.
2417 * 3. Therefore, *this* object doesn't have this field.
2418 */
2419 return 0;
2420 }
2421
2422 /* This is copied from get_ob_key_link() above -
2423 * only 4 lines, and saves the function call overhead.
2424 */
2425 for (link = op->key_values; link; link = link->next)
2426 if (link->key == canonical_key)
2427 return link->value; 2336 return kv->value;
2428 2337
2429 return 0; 2338 return shstr ();
2430} 2339}
2431 2340
2432/* 2341void
2433 * Updates the canonical_key in op to value. 2342object::kv_set (shstr_tmp key, shstr_tmp value)
2434 *
2435 * canonical_key is a shared string (value doesn't have to be).
2436 *
2437 * Unless add_key is TRUE, it won't add fields, only change the value of existing
2438 * keys.
2439 *
2440 * Returns TRUE on success.
2441 */
2442int
2443set_ob_key_value_s (object *op, const shstr & canonical_key, const char *value, int add_key)
2444{ 2343{
2445 key_value *field = NULL, *last = NULL; 2344 for (key_value *kv = key_values; kv; kv = kv->next)
2446 2345 if (kv->key == key)
2447 for (field = op->key_values; field != NULL; field = field->next)
2448 {
2449 if (field->key != canonical_key)
2450 { 2346 {
2451 last = field; 2347 kv->value = value;
2452 continue; 2348 return;
2453 } 2349 }
2454 2350
2455 if (value) 2351 key_value *kv = new key_value;
2456 field->value = value; 2352
2457 else 2353 kv->next = key_values;
2354 kv->key = key;
2355 kv->value = value;
2356
2357 key_values = kv;
2358}
2359
2360void
2361object::kv_del (shstr_tmp key)
2362{
2363 for (key_value **kvp = &key_values; *kvp; kvp = &(*kvp)->next)
2364 if ((*kvp)->key == key)
2458 { 2365 {
2459 /* Basically, if the archetype has this key set, 2366 key_value *kv = *kvp;
2460 * we need to store the null value so when we save 2367 *kvp = (*kvp)->next;
2461 * it, we save the empty value so that when we load, 2368 delete kv;
2462 * we get this value back again. 2369 return;
2463 */
2464 if (get_ob_key_link (op->arch, canonical_key))
2465 field->value = 0;
2466 else
2467 {
2468 if (last)
2469 last->next = field->next;
2470 else
2471 op->key_values = field->next;
2472
2473 delete field;
2474 }
2475 } 2370 }
2476 return TRUE;
2477 }
2478 /* IF we get here, key doesn't exist */
2479
2480 /* No field, we'll have to add it. */
2481
2482 if (!add_key)
2483 return FALSE;
2484
2485 /* There isn't any good reason to store a null
2486 * value in the key/value list. If the archetype has
2487 * this key, then we should also have it, so shouldn't
2488 * be here. If user wants to store empty strings,
2489 * should pass in ""
2490 */
2491 if (value == NULL)
2492 return TRUE;
2493
2494 field = new key_value;
2495
2496 field->key = canonical_key;
2497 field->value = value;
2498 /* Usual prepend-addition. */
2499 field->next = op->key_values;
2500 op->key_values = field;
2501
2502 return TRUE;
2503}
2504
2505/*
2506 * Updates the key in op to value.
2507 *
2508 * If add_key is FALSE, this will only update existing keys,
2509 * and not add new ones.
2510 * In general, should be little reason FALSE is ever passed in for add_key
2511 *
2512 * Returns TRUE on success.
2513 */
2514int
2515set_ob_key_value (object *op, const char *key, const char *value, int add_key)
2516{
2517 shstr key_ (key);
2518
2519 return set_ob_key_value_s (op, key_, value, add_key);
2520} 2371}
2521 2372
2522object::depth_iterator::depth_iterator (object *container) 2373object::depth_iterator::depth_iterator (object *container)
2523: iterator_base (container) 2374: iterator_base (container)
2524{ 2375{
2574{ 2425{
2575 char flagdesc[512]; 2426 char flagdesc[512];
2576 char info2[256 * 4]; 2427 char info2[256 * 4];
2577 char *p = info; 2428 char *p = info;
2578 2429
2579 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}",
2580 count, 2431 count,
2581 uuid.c_str (), 2432 uuid.c_str (),
2582 &name, 2433 &name,
2583 title ? "\",title:\"" : "", 2434 title ? ",title:\"" : "",
2584 title ? (const char *)title : "", 2435 title ? (const char *)title : "",
2436 title ? "\"" : "",
2585 flag_desc (flagdesc, 512), type); 2437 flag_desc (flagdesc, 512), type);
2586 2438
2587 if (!flag[FLAG_REMOVED] && env) 2439 if (!flag[FLAG_REMOVED] && env)
2588 p += snprintf (p, 256, "(in %s)", env->debug_desc (info2)); 2440 p += snprintf (p, 256, "(in %s)", env->debug_desc (info2));
2589 2441
2645 container = 0; 2497 container = 0;
2646 2498
2647 // client needs item update to make it work, client bug requires this to be separate 2499 // client needs item update to make it work, client bug requires this to be separate
2648 esrv_update_item (UPD_FLAGS, this, old_container); 2500 esrv_update_item (UPD_FLAGS, this, old_container);
2649 2501
2650 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.", old_container->query_name ());
2651 play_sound (sound_find ("chest_close")); 2503 play_sound (sound_find ("chest_close"));
2652 } 2504 }
2653 2505
2654 if (new_container) 2506 if (new_container)
2655 { 2507 {
2665 closer->flag [FLAG_NO_MAP_SAVE] = 1; 2517 closer->flag [FLAG_NO_MAP_SAVE] = 1;
2666 new_container->insert (closer); 2518 new_container->insert (closer);
2667 } 2519 }
2668#endif 2520#endif
2669 2521
2670 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.", new_container->query_name ());
2671 2523
2672 // make sure the container is available, client bug requires this to be separate 2524 // make sure the container is available, client bug requires this to be separate
2673 esrv_send_item (this, new_container); 2525 esrv_send_item (this, new_container);
2674 2526
2675 new_container->flag [FLAG_APPLIED] = true; 2527 new_container->flag [FLAG_APPLIED] = true;
2683// else if (!old_container->env && contr && contr->ns) 2535// else if (!old_container->env && contr && contr->ns)
2684// contr->ns->floorbox_reset (); 2536// contr->ns->floorbox_reset ();
2685} 2537}
2686 2538
2687object * 2539object *
2688object::force_find (const shstr name) 2540object::force_find (shstr_tmp name)
2689{ 2541{
2690 /* cycle through his inventory to look for the MARK we want to 2542 /* cycle through his inventory to look for the MARK we want to
2691 * place 2543 * place
2692 */ 2544 */
2693 for (object *tmp = inv; tmp; tmp = tmp->below) 2545 for (object *tmp = inv; tmp; tmp = tmp->below)
2695 return splay (tmp); 2547 return splay (tmp);
2696 2548
2697 return 0; 2549 return 0;
2698} 2550}
2699 2551
2700void 2552object *
2701object::force_add (const shstr name, int duration) 2553object::force_add (shstr_tmp name, int duration)
2702{ 2554{
2703 if (object *force = force_find (name)) 2555 if (object *force = force_find (name))
2704 force->destroy (); 2556 force->destroy ();
2705 2557
2706 object *force = get_archetype (FORCE_NAME); 2558 object *force = get_archetype (FORCE_NAME);
2711 2563
2712 force->set_speed (duration ? 1.f / duration : 0.f); 2564 force->set_speed (duration ? 1.f / duration : 0.f);
2713 force->flag [FLAG_IS_USED_UP] = true; 2565 force->flag [FLAG_IS_USED_UP] = true;
2714 force->flag [FLAG_APPLIED] = true; 2566 force->flag [FLAG_APPLIED] = true;
2715 2567
2716 insert (force); 2568 return insert (force);
2717} 2569}
2718 2570
2719void 2571void
2720object::play_sound (faceidx sound) 2572object::play_sound (faceidx sound) const
2721{ 2573{
2722 if (!sound) 2574 if (!sound)
2723 return; 2575 return;
2724 2576
2725 if (flag [FLAG_REMOVED]) 2577 if (is_on_map ())
2578 map->play_sound (sound, x, y);
2579 else if (object *pl = in_player ())
2580 pl->contr->play_sound (sound);
2581}
2582
2583void
2584object::say_msg (const char *msg) const
2585{
2586 if (is_on_map ())
2587 map->say_msg (msg, x, y);
2588 else if (object *pl = in_player ())
2589 pl->contr->play_sound (sound);
2590}
2591
2592void
2593object::make_noise ()
2594{
2595 // we do not model noise in the map, so instead put
2596 // a temporary light into the noise source
2597 // could use the map instead, but that's less reliable for our
2598 // goal, which is to make invisibility a bit harder to exploit
2599
2600 // currently only works sensibly for players
2601 if (!is_player ())
2726 return; 2602 return;
2727 2603
2728 if (env) 2604 // find old force, or create new one
2729 { 2605 object *force = force_find (shstr_noise_force);
2730 if (object *pl = in_player ()) 2606
2731 pl->contr->play_sound (sound); 2607 if (force)
2732 } 2608 force->speed_left = -1.f; // patch old speed up
2733 else 2609 else
2734 map->play_sound (sound, x, y); 2610 {
2735} 2611 force = archetype::get (shstr_noise_force);
2736 2612
2613 force->slaying = shstr_noise_force;
2614 force->stats.food = 1;
2615 force->speed_left = -1.f;
2616
2617 force->set_speed (1.f / 4.f);
2618 force->flag [FLAG_IS_USED_UP] = true;
2619 force->flag [FLAG_APPLIED] = true;
2620
2621 insert (force);
2622 }
2623}
2624

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines