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.43 by root, Thu Sep 14 17:10:24 2006 UTC vs.
Revision 1.74 by root, Wed Dec 20 09:14:21 2006 UTC

1
2/* 1/*
3 CrossFire, A Multiplayer game for X-windows 2 CrossFire, A Multiplayer game for X-windows
4 3
5 Copyright (C) 2001 Mark Wedel & Crossfire Development Team 4 Copyright (C) 2001 Mark Wedel & Crossfire Development Team
6 Copyright (C) 1992 Frank Tore Johansen 5 Copyright (C) 1992 Frank Tore Johansen
17 16
18 You should have received a copy of the GNU General Public License 17 You should have received a copy of the GNU General Public License
19 along with this program; if not, write to the Free Software 18 along with this program; if not, write to the Free Software
20 Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. 19 Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
21 20
22 The authors can be reached via e-mail at crossfire-devel@real-time.com 21 The authors can be reached via e-mail at <crossfire@schmorp.de>
23*/ 22*/
24 23
25/* Eneq(@csd.uu.se): Added weight-modifiers in environment of objects. 24/* Eneq(@csd.uu.se): Added weight-modifiers in environment of objects.
26 sub/add_weight will transcend the environment updating the carrying 25 sub/add_weight will transcend the environment updating the carrying
27 variable. */ 26 variable. */
31#include <sys/uio.h> 30#include <sys/uio.h>
32#include <object.h> 31#include <object.h>
33#include <funcpoint.h> 32#include <funcpoint.h>
34#include <loader.h> 33#include <loader.h>
35 34
35#include <bitset>
36
36int nrofallocobjects = 0; 37int nrofallocobjects = 0;
37static UUID uuid; 38static UUID uuid;
38const uint64 UUID_SKIP = 1<<19; 39const uint64 UUID_SKIP = 1<<19;
39 40
40object *objects; /* Pointer to the list of used objects */
41object *active_objects; /* List of active objects that need to be processed */ 41object *active_objects; /* List of active objects that need to be processed */
42 42
43short freearr_x[SIZEOFFREE] = { 0, 0, 1, 1, 1, 0, -1, -1, -1, 0, 1, 2, 2, 2, 2, 2, 1, 0, -1, -2, -2, -2, -2, -2, -1, 43short freearr_x[SIZEOFFREE] = { 0, 0, 1, 1, 1, 0, -1, -1, -1, 0, 1, 2, 2, 2, 2, 2, 1, 0, -1, -2, -2, -2, -2, -2, -1,
44 0, 1, 2, 3, 3, 3, 3, 3, 3, 3, 2, 1, 0, -1, -2, -3, -3, -3, -3, -3, -3, -3, -2, -1 44 0, 1, 2, 3, 3, 3, 3, 3, 3, 3, 2, 1, 0, -1, -2, -3, -3, -3, -3, -3, -3, -3, -2, -1
45}; 45};
106 _exit (1); 106 _exit (1);
107 } 107 }
108 108
109 uuid.seq = uid; 109 uuid.seq = uid;
110 write_uuid (); 110 write_uuid ();
111 LOG (llevDebug, "read UID: %lld\n", uid); 111 LOG (llevDebug, "read UID: %" PRId64 "\n", uid);
112 fclose (fp); 112 fclose (fp);
113} 113}
114 114
115UUID 115UUID
116gen_uuid () 116gen_uuid ()
185 * 185 *
186 * Note that this function appears a lot longer than the macro it 186 * Note that this function appears a lot longer than the macro it
187 * replaces - this is mostly for clarity - a decent compiler should hopefully 187 * replaces - this is mostly for clarity - a decent compiler should hopefully
188 * reduce this to the same efficiency. 188 * reduce this to the same efficiency.
189 * 189 *
190 * Check nrof variable *before* calling CAN_MERGE() 190 * Check nrof variable *before* calling can_merge()
191 * 191 *
192 * Improvements made with merge: Better checking on potion, and also 192 * Improvements made with merge: Better checking on potion, and also
193 * check weight 193 * check weight
194 */ 194 */
195 195
196bool object::can_merge (object *ob1, object *ob2) 196bool object::can_merge_slow (object *ob1, object *ob2)
197{ 197{
198 /* A couple quicksanity checks */ 198 /* A couple quicksanity checks */
199 if ((ob1 == ob2) || (ob1->type != ob2->type)) 199 if (ob1 == ob2
200 || ob1->type != ob2->type
201 || ob1->speed != ob2->speed
202 || ob1->value != ob2->value
203 || ob1->name != ob2->name)
200 return 0; 204 return 0;
201 205
202 if (ob1->speed != ob2->speed) 206 //TODO: this ain't working well, use nicer and correct overflow check
203 return 0;
204
205 /* Do not merge objects if nrof would overflow. We use 1UL<<31 since that 207 /* Do not merge objects if nrof would overflow. We use 1UL<<31 since that
206 * value could not be stored in a sint32 (which unfortunately sometimes is 208 * value could not be stored in a sint32 (which unfortunately sometimes is
207 * used to store nrof). 209 * used to store nrof).
208 */ 210 */
209 if (ob1->nrof + ob2->nrof >= 1UL << 31) 211 if (ob1->nrof + ob2->nrof >= 1UL << 31)
219 SET_FLAG (ob1, FLAG_BEEN_APPLIED); 221 SET_FLAG (ob1, FLAG_BEEN_APPLIED);
220 222
221 if (QUERY_FLAG (ob2, FLAG_IDENTIFIED)) 223 if (QUERY_FLAG (ob2, FLAG_IDENTIFIED))
222 SET_FLAG (ob2, FLAG_BEEN_APPLIED); 224 SET_FLAG (ob2, FLAG_BEEN_APPLIED);
223 225
224 226 if ((ob1->flags ^ ob2->flags).reset (FLAG_INV_LOCKED).reset (FLAG_CLIENT_SENT).any ()
225 /* the 0x400000 on flags2 is FLAG_INV_LOCK. I don't think something 227 || ob1->arch != ob2->arch
226 * being locked in inventory should prevent merging.
227 * 0x4 in flags3 is CLIENT_SENT
228 */
229 if ((ob1->arch != ob2->arch) ||
230 (ob1->flags[0] != ob2->flags[0]) ||
231 (ob1->flags[1] != ob2->flags[1]) ||
232 ((ob1->flags[2] & ~0x400000) != (ob2->flags[2] & ~0x400000)) ||
233 ((ob1->flags[3] & ~0x4) != (ob2->flags[3] & ~0x4)) ||
234 (ob1->name != ob2->name) || 228 || ob1->name != ob2->name
235 (ob1->title != ob2->title) || 229 || ob1->title != ob2->title
236 (ob1->msg != ob2->msg) || 230 || ob1->msg != ob2->msg
237 (ob1->weight != ob2->weight) || 231 || ob1->weight != ob2->weight
238 (memcmp (&ob1->resist, &ob2->resist, sizeof (ob1->resist)) != 0) || 232 || memcmp (&ob1->resist, &ob2->resist, sizeof (ob1->resist))
239 (memcmp (&ob1->stats, &ob2->stats, sizeof (ob1->stats)) != 0) || 233 || memcmp (&ob1->stats , &ob2->stats , sizeof (ob1->stats) )
240 (ob1->attacktype != ob2->attacktype) || 234 || ob1->attacktype != ob2->attacktype
241 (ob1->magic != ob2->magic) || 235 || ob1->magic != ob2->magic
242 (ob1->slaying != ob2->slaying) || 236 || ob1->slaying != ob2->slaying
243 (ob1->skill != ob2->skill) || 237 || ob1->skill != ob2->skill
244 (ob1->value != ob2->value) || 238 || ob1->value != ob2->value
245 (ob1->animation_id != ob2->animation_id) || 239 || ob1->animation_id != ob2->animation_id
246 (ob1->client_type != ob2->client_type) || 240 || ob1->client_type != ob2->client_type
247 (ob1->materialname != ob2->materialname) || 241 || ob1->materialname != ob2->materialname
248 (ob1->lore != ob2->lore) || 242 || ob1->lore != ob2->lore
249 (ob1->subtype != ob2->subtype) || 243 || ob1->subtype != ob2->subtype
250 (ob1->move_type != ob2->move_type) || 244 || ob1->move_type != ob2->move_type
251 (ob1->move_block != ob2->move_block) || 245 || ob1->move_block != ob2->move_block
252 (ob1->move_allow != ob2->move_allow) || 246 || ob1->move_allow != ob2->move_allow
253 (ob1->move_on != ob2->move_on) || 247 || ob1->move_on != ob2->move_on
254 (ob1->move_off != ob2->move_off) || (ob1->move_slow != ob2->move_slow) || (ob1->move_slow_penalty != ob2->move_slow_penalty)) 248 || ob1->move_off != ob2->move_off
249 || ob1->move_slow != ob2->move_slow
250 || ob1->move_slow_penalty != ob2->move_slow_penalty)
255 return 0; 251 return 0;
256 252
257 /* This is really a spellbook check - really, we should 253 /* This is really a spellbook check - really, we should
258 * check all objects in the inventory. 254 * check all objects in the inventory.
259 */ 255 */
262 /* if one object has inventory but the other doesn't, not equiv */ 258 /* if one object has inventory but the other doesn't, not equiv */
263 if ((ob1->inv && !ob2->inv) || (ob2->inv && !ob1->inv)) 259 if ((ob1->inv && !ob2->inv) || (ob2->inv && !ob1->inv))
264 return 0; 260 return 0;
265 261
266 /* Now check to see if the two inventory objects could merge */ 262 /* Now check to see if the two inventory objects could merge */
267 if (!CAN_MERGE (ob1->inv, ob2->inv)) 263 if (!object::can_merge (ob1->inv, ob2->inv))
268 return 0; 264 return 0;
269 265
270 /* inventory ok - still need to check rest of this object to see 266 /* inventory ok - still need to check rest of this object to see
271 * if it is valid. 267 * if it is valid.
272 */ 268 */
356 op = op->env; 352 op = op->env;
357 return op; 353 return op;
358} 354}
359 355
360/* 356/*
361 * Eneq(@csd.uu.se): Since we can have items buried in a character we need
362 * a better check. We basically keeping traversing up until we can't
363 * or find a player.
364 */
365
366object *
367is_player_inv (object *op)
368{
369 for (; op != NULL && op->type != PLAYER; op = op->env)
370 if (op->env == op)
371 op->env = NULL;
372 return op;
373}
374
375/*
376 * Used by: Crossedit: dump. Server DM commands: dumpbelow, dump. 357 * Used by: Crossedit: dump. Server DM commands: dumpbelow, dump.
377 * Some error messages. 358 * Some error messages.
378 * The result of the dump is stored in the static global errmsg array. 359 * The result of the dump is stored in the static global errmsg array.
379 */ 360 */
380 361
381void 362char *
382dump_object2 (object *op)
383{
384 errmsg[0] = 0;
385 return;
386 //TODO//D#d#
387#if 0
388 char *cp;
389
390/* object *tmp;*/
391
392 if (op->arch != NULL)
393 {
394 strcat (errmsg, "arch ");
395 strcat (errmsg, op->arch->name ? op->arch->name : "(null)");
396 strcat (errmsg, "\n");
397 if ((cp = get_ob_diff (op, &empty_archetype->clone)) != NULL)
398 strcat (errmsg, cp);
399# if 0
400 /* Don't dump player diffs - they are too long, mostly meaningless, and
401 * will overflow the buffer.
402 * Changed so that we don't dump inventory either. This may
403 * also overflow the buffer.
404 */
405 if (op->type != PLAYER && (cp = get_ob_diff (op, &empty_archetype->clone)) != NULL)
406 strcat (errmsg, cp);
407 for (tmp = op->inv; tmp; tmp = tmp->below)
408 dump_object2 (tmp);
409# endif
410 strcat (errmsg, "end\n");
411 }
412 else
413 {
414 strcat (errmsg, "Object ");
415 if (op->name == NULL)
416 strcat (errmsg, "(null)");
417 else
418 strcat (errmsg, op->name);
419 strcat (errmsg, "\n");
420# if 0
421 if ((cp = get_ob_diff (op, &empty_archetype->clone)) != NULL)
422 strcat (errmsg, cp);
423 for (tmp = op->inv; tmp; tmp = tmp->below)
424 dump_object2 (tmp);
425# endif
426 strcat (errmsg, "end\n");
427 }
428#endif
429}
430
431/*
432 * Dumps an object. Returns output in the static global errmsg array.
433 */
434
435void
436dump_object (object *op) 363dump_object (object *op)
437{ 364{
438 if (op == NULL) 365 if (!op)
439 { 366 return strdup ("[NULLOBJ]");
440 strcpy (errmsg, "[NULL pointer]");
441 return;
442 }
443 errmsg[0] = '\0';
444 dump_object2 (op);
445}
446 367
447void 368 object_freezer freezer;
448dump_all_objects (void) 369 save_object (freezer, op, 3);
449{ 370 return freezer.as_string ();
450 object *op;
451
452 for (op = objects; op != NULL; op = op->next)
453 {
454 dump_object (op);
455 fprintf (logfile, "Object %d\n:%s\n", op->count, errmsg);
456 }
457} 371}
458 372
459/* 373/*
460 * get_nearest_part(multi-object, object 2) returns the part of the 374 * get_nearest_part(multi-object, object 2) returns the part of the
461 * multi-object 1 which is closest to the second object. 375 * multi-object 1 which is closest to the second object.
481 */ 395 */
482 396
483object * 397object *
484find_object (tag_t i) 398find_object (tag_t i)
485{ 399{
486 object *op; 400 for (object *op = object::first; op; op = op->next)
487
488 for (op = objects; op != NULL; op = op->next)
489 if (op->count == i) 401 if (op->count == i)
490 break; 402 return op;
403
491 return op; 404 return 0;
492} 405}
493 406
494/* 407/*
495 * Returns the first object which has a name equal to the argument. 408 * Returns the first object which has a name equal to the argument.
496 * Used only by the patch command, but not all that useful. 409 * Used only by the patch command, but not all that useful.
501find_object_name (const char *str) 414find_object_name (const char *str)
502{ 415{
503 shstr_cmp str_ (str); 416 shstr_cmp str_ (str);
504 object *op; 417 object *op;
505 418
506 for (op = objects; op != NULL; op = op->next) 419 for (op = object::first; op != NULL; op = op->next)
507 if (op->name == str_) 420 if (op->name == str_)
508 break; 421 break;
509 422
510 return op; 423 return op;
511} 424}
596 509
597 if (settings.casting_time) 510 if (settings.casting_time)
598 casting_time = -1; 511 casting_time = -1;
599} 512}
600 513
601void object::clone (object *destination)
602{
603 *(object_copy *)destination = *this;
604 *(object_pod *)destination = *this;
605
606 if (self || cb)
607 INVOKE_OBJECT (CLONE, this, ARG_OBJECT (destination));
608}
609
610/* 514/*
611 * copy object first frees everything allocated by the second object, 515 * copy_to first frees everything allocated by the dst object,
612 * and then copies the contends of the first object into the second 516 * and then copies the contents of itself into the second
613 * object, allocating what needs to be allocated. Basically, any 517 * object, allocating what needs to be allocated. Basically, any
614 * data that is malloc'd needs to be re-malloc/copied. Otherwise, 518 * data that is malloc'd needs to be re-malloc/copied. Otherwise,
615 * if the first object is freed, the pointers in the new object 519 * if the first object is freed, the pointers in the new object
616 * will point at garbage. 520 * will point at garbage.
617 */ 521 */
618void 522void
619copy_object (object *op2, object *op) 523object::copy_to (object *dst)
620{ 524{
621 bool is_freed = QUERY_FLAG (op, FLAG_FREED); 525 bool is_freed = QUERY_FLAG (dst, FLAG_FREED);
622 bool is_removed = QUERY_FLAG (op, FLAG_REMOVED); 526 bool is_removed = QUERY_FLAG (dst, FLAG_REMOVED);
623 527
624 op2->clone (op); 528 *(object_copy *)dst = *this;
529 *(object_pod *)dst = *this;
530
531 if (self || cb)
532 INVOKE_OBJECT (CLONE, this, ARG_OBJECT (dst));
625 533
626 if (is_freed) 534 if (is_freed)
627 SET_FLAG (op, FLAG_FREED); 535 SET_FLAG (dst, FLAG_FREED);
536
628 if (is_removed) 537 if (is_removed)
629 SET_FLAG (op, FLAG_REMOVED); 538 SET_FLAG (dst, FLAG_REMOVED);
630 539
631 if (op2->speed < 0) 540 if (speed < 0)
632 op->speed_left = op2->speed_left - RANDOM () % 200 / 100.0; 541 dst->speed_left = speed_left - RANDOM () % 200 / 100.0;
633 542
634 /* Copy over key_values, if any. */ 543 /* Copy over key_values, if any. */
635 if (op2->key_values) 544 if (key_values)
636 { 545 {
637 key_value *tail = 0; 546 key_value *tail = 0;
638 key_value *i; 547 key_value *i;
639 548
640 op->key_values = 0; 549 dst->key_values = 0;
641 550
642 for (i = op2->key_values; i; i = i->next) 551 for (i = key_values; i; i = i->next)
643 { 552 {
644 key_value *new_link = new key_value; 553 key_value *new_link = new key_value;
645 554
646 new_link->next = 0; 555 new_link->next = 0;
647 new_link->key = i->key; 556 new_link->key = i->key;
648 new_link->value = i->value; 557 new_link->value = i->value;
649 558
650 /* Try and be clever here, too. */ 559 /* Try and be clever here, too. */
651 if (!op->key_values) 560 if (!dst->key_values)
652 { 561 {
653 op->key_values = new_link; 562 dst->key_values = new_link;
654 tail = new_link; 563 tail = new_link;
655 } 564 }
656 else 565 else
657 { 566 {
658 tail->next = new_link; 567 tail->next = new_link;
659 tail = new_link; 568 tail = new_link;
660 } 569 }
661 } 570 }
662 } 571 }
663 572
664 update_ob_speed (op); 573 update_ob_speed (dst);
574}
575
576object *
577object::clone ()
578{
579 object *neu = create ();
580 copy_to (neu);
581 return neu;
665} 582}
666 583
667/* 584/*
668 * If an object with the IS_TURNABLE() flag needs to be turned due 585 * If an object with the IS_TURNABLE() flag needs to be turned due
669 * to the closest player being on the other side, this function can 586 * to the closest player being on the other side, this function can
874 update_now = 1; 791 update_now = 1;
875 792
876 if ((move_slow | op->move_slow) != move_slow) 793 if ((move_slow | op->move_slow) != move_slow)
877 update_now = 1; 794 update_now = 1;
878 } 795 }
796
879 /* if the object is being removed, we can't make intelligent 797 /* if the object is being removed, we can't make intelligent
880 * decisions, because remove_ob can't really pass the object 798 * decisions, because remove_ob can't really pass the object
881 * that is being removed. 799 * that is being removed.
882 */ 800 */
883 else if (action == UP_OBJ_CHANGE || action == UP_OBJ_REMOVE) 801 else if (action == UP_OBJ_CHANGE || action == UP_OBJ_REMOVE)
895 813
896 if (op->more != NULL) 814 if (op->more != NULL)
897 update_object (op->more, action); 815 update_object (op->more, action);
898} 816}
899 817
900static unordered_vector<object *> mortals; 818object::vector object::mortals;
819object::vector object::objects; // not yet used
820object *object::first;
901 821
902void object::free_mortals () 822void object::free_mortals ()
903{ 823{
904 for (unordered_vector<object *>::iterator i = mortals.begin (); i != mortals.end ();) 824 for (AUTODECL (i, mortals.begin ()); i != mortals.end ();)
905 if ((*i)->refcnt) 825 if ((*i)->refcnt)
906 ++i; // further delay freeing 826 ++i; // further delay freeing
907 else 827 else
908 { 828 {
909 delete *i; 829 delete *i;
910 mortals.erase (i); 830 mortals.erase (i);
911 } 831 }
912
913 static int lastmortals = 0;//D
914
915 if (mortals.size() != lastmortals)//D
916 {
917 lastmortals = mortals.size ();//D
918 LOG (llevDebug, "%d objects in mortal queue\n", lastmortals);//D
919 }
920} 832}
921 833
922object::object () 834object::object ()
923{ 835{
924 SET_FLAG (this, FLAG_REMOVED); 836 SET_FLAG (this, FLAG_REMOVED);
936{ 848{
937 count = ++ob_count; 849 count = ++ob_count;
938 uuid = gen_uuid (); 850 uuid = gen_uuid ();
939 851
940 prev = 0; 852 prev = 0;
941 next = objects; 853 next = object::first;
942 854
943 if (objects) 855 if (object::first)
944 objects->prev = this; 856 object::first->prev = this;
945 857
946 objects = this; 858 object::first = this;
947} 859}
948 860
949void object::unlink () 861void object::unlink ()
950{ 862{
951 //count = 0;//D
952 if (!prev && !next) return;//D
953
954 if (this == objects) 863 if (this == object::first)
955 objects = next; 864 object::first = next;
956 865
957 /* Remove this object from the list of used objects */ 866 /* Remove this object from the list of used objects */
958 if (prev) prev->next = next; 867 if (prev) prev->next = next;
959 if (next) next->prev = prev; 868 if (next) next->prev = prev;
960 869
974 * it from the list of used objects, and puts it on the list of 883 * it from the list of used objects, and puts it on the list of
975 * free objects. The IS_FREED() flag is set in the object. 884 * free objects. The IS_FREED() flag is set in the object.
976 * The object must have been removed by remove_ob() first for 885 * The object must have been removed by remove_ob() first for
977 * this function to succeed. 886 * this function to succeed.
978 * 887 *
979 * If free_inventory is set, free inventory as well. Else drop items in 888 * If destroy_inventory is set, free inventory as well. Else drop items in
980 * inventory to the ground. 889 * inventory to the ground.
981 */ 890 */
982void object::free (bool free_inventory) 891void object::destroy (bool destroy_inventory)
983{ 892{
984 if (QUERY_FLAG (this, FLAG_FREED)) 893 if (QUERY_FLAG (this, FLAG_FREED))
985 return; 894 return;
986 895
987 if (QUERY_FLAG (this, FLAG_FRIENDLY)) 896 if (QUERY_FLAG (this, FLAG_FRIENDLY))
988 remove_friendly_object (this); 897 remove_friendly_object (this);
989 898
990 if (!QUERY_FLAG (this, FLAG_REMOVED)) 899 if (!QUERY_FLAG (this, FLAG_REMOVED))
991 remove_ob (this); 900 remove ();
992 901
993 SET_FLAG (this, FLAG_FREED); 902 SET_FLAG (this, FLAG_FREED);
994 903
995 if (more) 904 if (more)
996 { 905 {
997 more->free (free_inventory); 906 more->destroy (destroy_inventory);
998 more = 0; 907 more = 0;
999 } 908 }
1000 909
1001 if (inv) 910 if (inv)
1002 { 911 {
1003 /* Only if the space blocks everything do we not process - 912 /* Only if the space blocks everything do we not process -
1004 * if some form of movement is allowed, let objects 913 * if some form of movement is allowed, let objects
1005 * drop on that space. 914 * drop on that space.
1006 */ 915 */
1007 if (free_inventory || !map || map->in_memory != MAP_IN_MEMORY || GET_MAP_MOVE_BLOCK (map, x, y) == MOVE_ALL) 916 if (destroy_inventory || !map || map->in_memory != MAP_IN_MEMORY || GET_MAP_MOVE_BLOCK (map, x, y) == MOVE_ALL)
1008 { 917 {
1009 object *op = inv; 918 object *op = inv;
1010 919
1011 while (op) 920 while (op)
1012 { 921 {
1013 object *tmp = op->below; 922 object *tmp = op->below;
1014 op->free (free_inventory); 923 op->destroy (destroy_inventory);
1015 op = tmp; 924 op = tmp;
1016 } 925 }
1017 } 926 }
1018 else 927 else
1019 { /* Put objects in inventory onto this space */ 928 { /* Put objects in inventory onto this space */
1021 930
1022 while (op) 931 while (op)
1023 { 932 {
1024 object *tmp = op->below; 933 object *tmp = op->below;
1025 934
1026 remove_ob (op); 935 op->remove ();
1027 936
1028 if (QUERY_FLAG (op, FLAG_STARTEQUIP) 937 if (QUERY_FLAG (op, FLAG_STARTEQUIP)
1029 || QUERY_FLAG (op, FLAG_NO_DROP) || op->type == RUNE || op->type == TRAP || QUERY_FLAG (op, FLAG_IS_A_TEMPLATE)) 938 || QUERY_FLAG (op, FLAG_NO_DROP) || op->type == RUNE || op->type == TRAP || QUERY_FLAG (op, FLAG_IS_A_TEMPLATE))
1030 free_object (op); 939 op->destroy ();
1031 else 940 else
1032 { 941 {
1033 op->x = x; 942 op->x = x;
1034 op->y = y; 943 op->y = y;
1035 insert_ob_in_map (op, map, 0, 0); /* Insert in same map as the envir */ 944 insert_ob_in_map (op, map, 0, 0); /* Insert in same map as the envir */
1038 op = tmp; 947 op = tmp;
1039 } 948 }
1040 } 949 }
1041 } 950 }
1042 951
952 // hack to ensure that freed objects still have a valid map
953 {
954 static maptile *freed_map; // freed objects are moved here to avoid crashes
955
956 if (!freed_map)
957 {
958 freed_map = new maptile;
959
960 freed_map->name = "/internal/freed_objects_map";
961 freed_map->width = 3;
962 freed_map->height = 3;
963
964 freed_map->allocate ();
965 }
966
967 map = freed_map;
968 x = 1;
969 y = 1;
970 }
971
972 // clear those pointers that likely might have circular references to us
1043 owner = 0; 973 owner = 0;
974 enemy = 0;
975 attacked_by = 0;
976
977 // only relevant for players(?), but make sure of it anyways
978 contr = 0;
1044 979
1045 /* Remove object from the active list */ 980 /* Remove object from the active list */
1046 speed = 0; 981 speed = 0;
1047 update_ob_speed (this); 982 update_ob_speed (this);
1048 983
1053 988
1054/* 989/*
1055 * sub_weight() recursively (outwards) subtracts a number from the 990 * sub_weight() recursively (outwards) subtracts a number from the
1056 * weight of an object (and what is carried by it's environment(s)). 991 * weight of an object (and what is carried by it's environment(s)).
1057 */ 992 */
1058
1059void 993void
1060sub_weight (object *op, signed long weight) 994sub_weight (object *op, signed long weight)
1061{ 995{
1062 while (op != NULL) 996 while (op != NULL)
1063 { 997 {
1064 if (op->type == CONTAINER) 998 if (op->type == CONTAINER)
1065 {
1066 weight = (signed long) (weight * (100 - op->stats.Str) / 100); 999 weight = (signed long) (weight * (100 - op->stats.Str) / 100);
1067 } 1000
1068 op->carrying -= weight; 1001 op->carrying -= weight;
1069 op = op->env; 1002 op = op->env;
1070 } 1003 }
1071} 1004}
1072 1005
1073/* remove_ob(op): 1006/* op->remove ():
1074 * This function removes the object op from the linked list of objects 1007 * This function removes the object op from the linked list of objects
1075 * which it is currently tied to. When this function is done, the 1008 * which it is currently tied to. When this function is done, the
1076 * object will have no environment. If the object previously had an 1009 * object will have no environment. If the object previously had an
1077 * environment, the x and y coordinates will be updated to 1010 * environment, the x and y coordinates will be updated to
1078 * the previous environment. 1011 * the previous environment.
1079 * Beware: This function is called from the editor as well! 1012 * Beware: This function is called from the editor as well!
1080 */ 1013 */
1081
1082void 1014void
1083remove_ob (object *op) 1015object::remove ()
1084{ 1016{
1017 object *tmp, *last = 0;
1085 object * 1018 object *otmp;
1086 tmp, *
1087 last = NULL;
1088 object *
1089 otmp;
1090 1019
1091 tag_t
1092 tag;
1093 int
1094 check_walk_off; 1020 int check_walk_off;
1095 mapstruct *
1096 m;
1097 1021
1098 sint16
1099 x,
1100 y;
1101
1102 if (QUERY_FLAG (op, FLAG_REMOVED)) 1022 if (QUERY_FLAG (this, FLAG_REMOVED))
1103 return; 1023 return;
1104 1024
1105 SET_FLAG (op, FLAG_REMOVED); 1025 SET_FLAG (this, FLAG_REMOVED);
1106 1026
1107 if (op->more != NULL) 1027 if (more)
1108 remove_ob (op->more); 1028 more->remove ();
1109 1029
1110 /* 1030 /*
1111 * In this case, the object to be removed is in someones 1031 * In this case, the object to be removed is in someones
1112 * inventory. 1032 * inventory.
1113 */ 1033 */
1114 if (op->env != NULL) 1034 if (env)
1115 { 1035 {
1116 if (op->nrof) 1036 if (nrof)
1117 sub_weight (op->env, op->weight * op->nrof); 1037 sub_weight (env, weight * nrof);
1118 else 1038 else
1119 sub_weight (op->env, op->weight + op->carrying); 1039 sub_weight (env, weight + carrying);
1120 1040
1121 /* NO_FIX_PLAYER is set when a great many changes are being 1041 /* NO_FIX_PLAYER is set when a great many changes are being
1122 * made to players inventory. If set, avoiding the call 1042 * made to players inventory. If set, avoiding the call
1123 * to save cpu time. 1043 * to save cpu time.
1124 */ 1044 */
1125 if ((otmp = is_player_inv (op->env)) != NULL && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER)) 1045 if ((otmp = in_player ()) && otmp->contr && !QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
1126 fix_player (otmp); 1046 fix_player (otmp);
1127 1047
1128 if (op->above != NULL) 1048 if (above != NULL)
1129 op->above->below = op->below; 1049 above->below = below;
1130 else 1050 else
1131 op->env->inv = op->below; 1051 env->inv = below;
1132 1052
1133 if (op->below != NULL) 1053 if (below != NULL)
1134 op->below->above = op->above; 1054 below->above = above;
1135 1055
1136 /* we set up values so that it could be inserted into 1056 /* we set up values so that it could be inserted into
1137 * the map, but we don't actually do that - it is up 1057 * the map, but we don't actually do that - it is up
1138 * to the caller to decide what we want to do. 1058 * to the caller to decide what we want to do.
1139 */ 1059 */
1140 op->x = op->env->x, op->y = op->env->y; 1060 x = env->x, y = env->y;
1141 op->map = op->env->map; 1061 map = env->map;
1142 op->above = NULL, op->below = NULL; 1062 above = 0, below = 0;
1143 op->env = NULL; 1063 env = 0;
1144 } 1064 }
1145 else if (op->map) 1065 else if (map)
1146 { 1066 {
1147 x = op->x;
1148 y = op->y;
1149 m = get_map_from_coord (op->map, &x, &y);
1150
1151 if (!m)
1152 {
1153 LOG (llevError, "remove_ob called when object was on map but appears to not be within valid coordinates? %s (%d,%d)\n",
1154 op->map->path, op->x, op->y);
1155 /* in old days, we used to set x and y to 0 and continue.
1156 * it seems if we get into this case, something is probablye
1157 * screwed up and should be fixed.
1158 */
1159 abort ();
1160 }
1161
1162 if (op->map != m)
1163 LOG (llevDebug, "remove_ob: Object not really on map it claimed to be on? %s != %s, %d,%d != %d,%d\n",
1164 op->map->path, m->path, op->x, op->y, x, y);
1165
1166 /* Re did the following section of code - it looks like it had 1067 /* Re did the following section of code - it looks like it had
1167 * lots of logic for things we no longer care about 1068 * lots of logic for things we no longer care about
1168 */ 1069 */
1169 1070
1170 /* link the object above us */ 1071 /* link the object above us */
1171 if (op->above) 1072 if (above)
1172 op->above->below = op->below; 1073 above->below = below;
1173 else 1074 else
1174 SET_MAP_TOP (m, x, y, op->below); /* we were top, set new top */ 1075 SET_MAP_TOP (map, x, y, below); /* we were top, set new top */
1175 1076
1176 /* Relink the object below us, if there is one */ 1077 /* Relink the object below us, if there is one */
1177 if (op->below) 1078 if (below)
1178 op->below->above = op->above; 1079 below->above = above;
1179 else 1080 else
1180 { 1081 {
1181 /* Nothing below, which means we need to relink map object for this space 1082 /* Nothing below, which means we need to relink map object for this space
1182 * use translated coordinates in case some oddness with map tiling is 1083 * use translated coordinates in case some oddness with map tiling is
1183 * evident 1084 * evident
1184 */ 1085 */
1185 if (GET_MAP_OB (m, x, y) != op) 1086 if (GET_MAP_OB (map, x, y) != this)
1186 { 1087 {
1187 dump_object (op); 1088 char *dump = dump_object (this);
1188 LOG (llevError, 1089 LOG (llevError,
1189 "remove_ob: GET_MAP_OB does not return object to be removed even though it appears to be on the bottom?\n%s\n", errmsg); 1090 "remove_ob: GET_MAP_OB does not return object to be removed even though it appears to be on the bottom?\n%s\n", dump);
1091 free (dump);
1190 dump_object (GET_MAP_OB (m, x, y)); 1092 dump = dump_object (GET_MAP_OB (map, x, y));
1191 LOG (llevError, "%s\n", errmsg); 1093 LOG (llevError, "%s\n", dump);
1094 free (dump);
1192 } 1095 }
1193 1096
1194 SET_MAP_OB (m, x, y, op->above); /* goes on above it. */ 1097 SET_MAP_OB (map, x, y, above); /* goes on above it. */
1195 } 1098 }
1196 1099
1197 op->above = 0; 1100 above = 0;
1198 op->below = 0; 1101 below = 0;
1199 1102
1200 if (op->map->in_memory == MAP_SAVING) 1103 if (map->in_memory == MAP_SAVING)
1201 return; 1104 return;
1202 1105
1203 tag = op->count;
1204 check_walk_off = !QUERY_FLAG (op, FLAG_NO_APPLY); 1106 check_walk_off = !QUERY_FLAG (this, FLAG_NO_APPLY);
1205 1107
1206 for (tmp = GET_MAP_OB (m, x, y); tmp != NULL; tmp = tmp->above) 1108 for (tmp = GET_MAP_OB (map, x, y); tmp; tmp = tmp->above)
1207 { 1109 {
1208 /* No point updating the players look faces if he is the object 1110 /* No point updating the players look faces if he is the object
1209 * being removed. 1111 * being removed.
1210 */ 1112 */
1211 1113
1212 if (tmp->type == PLAYER && tmp != op) 1114 if (tmp->type == PLAYER && tmp != this)
1213 { 1115 {
1214 /* If a container that the player is currently using somehow gets 1116 /* If a container that the player is currently using somehow gets
1215 * removed (most likely destroyed), update the player view 1117 * removed (most likely destroyed), update the player view
1216 * appropriately. 1118 * appropriately.
1217 */ 1119 */
1218 if (tmp->container == op) 1120 if (tmp->container == this)
1219 { 1121 {
1220 CLEAR_FLAG (op, FLAG_APPLIED); 1122 CLEAR_FLAG (this, FLAG_APPLIED);
1221 tmp->container = NULL; 1123 tmp->container = 0;
1222 } 1124 }
1223 1125
1224 tmp->contr->socket.update_look = 1; 1126 tmp->contr->socket->floorbox_update ();
1225 } 1127 }
1226 1128
1227 /* See if player moving off should effect something */ 1129 /* See if player moving off should effect something */
1228 if (check_walk_off && ((op->move_type & tmp->move_off) && (op->move_type & ~tmp->move_off & ~tmp->move_block) == 0)) 1130 if (check_walk_off
1131 && ((move_type & tmp->move_off)
1132 && (move_type & ~tmp->move_off & ~tmp->move_block) == 0))
1229 { 1133 {
1230 move_apply (tmp, op, NULL); 1134 move_apply (tmp, this, 0);
1231 1135
1232 if (was_destroyed (op, tag)) 1136 if (destroyed ())
1233 {
1234 LOG (llevError, "BUG: remove_ob(): name %s, archname %s destroyed " "leaving object\n", &tmp->name, &tmp->arch->name); 1137 LOG (llevError, "BUG: remove_ob(): name %s, destroyed leaving object\n", tmp->debug_desc ());
1235 }
1236 } 1138 }
1237 1139
1238 /* Eneq(@csd.uu.se): Fixed this to skip tmp->above=tmp */ 1140 /* Eneq(@csd.uu.se): Fixed this to skip tmp->above=tmp */
1239 1141
1240 if (tmp->above == tmp) 1142 if (tmp->above == tmp)
1241 tmp->above = NULL; 1143 tmp->above = 0;
1242 1144
1243 last = tmp; 1145 last = tmp;
1244 } 1146 }
1245 1147
1246 /* last == NULL of there are no objects on this space */ 1148 /* last == NULL of there are no objects on this space */
1247 if (last == NULL) 1149 if (!last)
1248 { 1150 {
1249 /* set P_NEED_UPDATE, otherwise update_position will complain. In theory, 1151 /* set P_NEED_UPDATE, otherwise update_position will complain. In theory,
1250 * we could preserve the flags (GET_MAP_FLAGS), but update_position figures 1152 * we could preserve the flags (GET_MAP_FLAGS), but update_position figures
1251 * those out anyways, and if there are any flags set right now, they won't 1153 * those out anyways, and if there are any flags set right now, they won't
1252 * be correct anyways. 1154 * be correct anyways.
1253 */ 1155 */
1254 SET_MAP_FLAGS (op->map, op->x, op->y, P_NEED_UPDATE); 1156 SET_MAP_FLAGS (map, x, y, P_NEED_UPDATE);
1255 update_position (op->map, op->x, op->y); 1157 update_position (map, x, y);
1256 } 1158 }
1257 else 1159 else
1258 update_object (last, UP_OBJ_REMOVE); 1160 update_object (last, UP_OBJ_REMOVE);
1259 1161
1260 if (QUERY_FLAG (op, FLAG_BLOCKSVIEW) || (op->glow_radius != 0)) 1162 if (QUERY_FLAG (this, FLAG_BLOCKSVIEW) || glow_radius)
1261 update_all_los (op->map, op->x, op->y); 1163 update_all_los (map, x, y);
1262 } 1164 }
1263} 1165}
1264 1166
1265/* 1167/*
1266 * merge_ob(op,top): 1168 * merge_ob(op,top):
1281 1183
1282 for (; top != NULL; top = top->below) 1184 for (; top != NULL; top = top->below)
1283 { 1185 {
1284 if (top == op) 1186 if (top == op)
1285 continue; 1187 continue;
1286 if (CAN_MERGE (op, top)) 1188
1189 if (object::can_merge (op, top))
1287 { 1190 {
1288 top->nrof += op->nrof; 1191 top->nrof += op->nrof;
1289 1192
1290/* CLEAR_FLAG(top,FLAG_STARTEQUIP);*/ 1193/* CLEAR_FLAG(top,FLAG_STARTEQUIP);*/
1291 op->weight = 0; /* Don't want any adjustements now */ 1194 op->weight = 0; /* Don't want any adjustements now */
1292 remove_ob (op); 1195 op->destroy ();
1293 free_object (op);
1294 return top; 1196 return top;
1295 } 1197 }
1296 } 1198 }
1297 1199
1298 return NULL; 1200 return 0;
1299} 1201}
1300 1202
1301/* 1203/*
1302 * same as insert_ob_in_map except it handle separate coordinates and do a clean 1204 * same as insert_ob_in_map except it handle separate coordinates and do a clean
1303 * job preparing multi-part monsters 1205 * job preparing multi-part monsters
1304 */ 1206 */
1305object * 1207object *
1306insert_ob_in_map_at (object *op, mapstruct *m, object *originator, int flag, int x, int y) 1208insert_ob_in_map_at (object *op, maptile *m, object *originator, int flag, int x, int y)
1307{ 1209{
1308 object *tmp; 1210 object *tmp;
1309 1211
1310 if (op->head) 1212 if (op->head)
1311 op = op->head; 1213 op = op->head;
1339 * NULL if 'op' was destroyed 1241 * NULL if 'op' was destroyed
1340 * just 'op' otherwise 1242 * just 'op' otherwise
1341 */ 1243 */
1342 1244
1343object * 1245object *
1344insert_ob_in_map (object *op, mapstruct *m, object *originator, int flag) 1246insert_ob_in_map (object *op, maptile *m, object *originator, int flag)
1345{ 1247{
1346 object *tmp, *top, *floor = NULL; 1248 object *tmp, *top, *floor = NULL;
1347 sint16 x, y; 1249 sint16 x, y;
1348 1250
1349 if (QUERY_FLAG (op, FLAG_FREED)) 1251 if (QUERY_FLAG (op, FLAG_FREED))
1352 return NULL; 1254 return NULL;
1353 } 1255 }
1354 1256
1355 if (m == NULL) 1257 if (m == NULL)
1356 { 1258 {
1357 dump_object (op); 1259 char *dump = dump_object (op);
1358 LOG (llevError, "Trying to insert in null-map!\n%s\n", errmsg); 1260 LOG (llevError, "Trying to insert in null-map!\n%s\n", dump);
1261 free (dump);
1359 return op; 1262 return op;
1360 } 1263 }
1361 1264
1362 if (out_of_map (m, op->x, op->y)) 1265 if (out_of_map (m, op->x, op->y))
1363 { 1266 {
1364 dump_object (op); 1267 char *dump = dump_object (op);
1365 LOG (llevError, "Trying to insert object outside the map.\n%s\n", errmsg); 1268 LOG (llevError, "Trying to insert object outside the map.\n%s\n", dump);
1366#ifdef MANY_CORES 1269#ifdef MANY_CORES
1367 /* Better to catch this here, as otherwise the next use of this object 1270 /* Better to catch this here, as otherwise the next use of this object
1368 * is likely to cause a crash. Better to find out where it is getting 1271 * is likely to cause a crash. Better to find out where it is getting
1369 * improperly inserted. 1272 * improperly inserted.
1370 */ 1273 */
1371 abort (); 1274 abort ();
1372#endif 1275#endif
1276 free (dump);
1373 return op; 1277 return op;
1374 } 1278 }
1375 1279
1376 if (!QUERY_FLAG (op, FLAG_REMOVED)) 1280 if (!QUERY_FLAG (op, FLAG_REMOVED))
1377 { 1281 {
1378 dump_object (op); 1282 char *dump = dump_object (op);
1379 LOG (llevError, "Trying to insert (map) inserted object.\n%s\n", errmsg); 1283 LOG (llevError, "Trying to insert (map) inserted object.\n%s\n", dump);
1284 free (dump);
1380 return op; 1285 return op;
1381 } 1286 }
1382 1287
1383 if (op->more != NULL) 1288 if (op->more != NULL)
1384 { 1289 {
1422 1327
1423 /* this has to be done after we translate the coordinates. 1328 /* this has to be done after we translate the coordinates.
1424 */ 1329 */
1425 if (op->nrof && !(flag & INS_NO_MERGE)) 1330 if (op->nrof && !(flag & INS_NO_MERGE))
1426 for (tmp = GET_MAP_OB (op->map, x, y); tmp != NULL; tmp = tmp->above) 1331 for (tmp = GET_MAP_OB (op->map, x, y); tmp != NULL; tmp = tmp->above)
1427 if (CAN_MERGE (op, tmp)) 1332 if (object::can_merge (op, tmp))
1428 { 1333 {
1429 op->nrof += tmp->nrof; 1334 op->nrof += tmp->nrof;
1430 remove_ob (tmp); 1335 tmp->destroy ();
1431 free_object (tmp);
1432 } 1336 }
1433 1337
1434 CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */ 1338 CLEAR_FLAG (op, FLAG_APPLIED); /* hack for fixing F_APPLIED in items of dead people */
1435 CLEAR_FLAG (op, FLAG_INV_LOCKED); 1339 CLEAR_FLAG (op, FLAG_INV_LOCKED);
1436 1340
1564 * it, so save a few ticks and start from there. 1468 * it, so save a few ticks and start from there.
1565 */ 1469 */
1566 if (!(flag & INS_MAP_LOAD)) 1470 if (!(flag & INS_MAP_LOAD))
1567 for (tmp = floor ? floor : GET_MAP_OB (op->map, op->x, op->y); tmp != NULL; tmp = tmp->above) 1471 for (tmp = floor ? floor : GET_MAP_OB (op->map, op->x, op->y); tmp != NULL; tmp = tmp->above)
1568 if (tmp->type == PLAYER) 1472 if (tmp->type == PLAYER)
1569 tmp->contr->socket.update_look = 1; 1473 tmp->contr->socket->floorbox_update ();
1570 1474
1571 /* If this object glows, it may affect lighting conditions that are 1475 /* If this object glows, it may affect lighting conditions that are
1572 * visible to others on this map. But update_all_los is really 1476 * visible to others on this map. But update_all_los is really
1573 * an inefficient way to do this, as it means los for all players 1477 * an inefficient way to do this, as it means los for all players
1574 * on the map will get recalculated. The players could very well 1478 * on the map will get recalculated. The players could very well
1582 1486
1583 /* updates flags (blocked, alive, no magic, etc) for this map space */ 1487 /* updates flags (blocked, alive, no magic, etc) for this map space */
1584 update_object (op, UP_OBJ_INSERT); 1488 update_object (op, UP_OBJ_INSERT);
1585 1489
1586 /* Don't know if moving this to the end will break anything. However, 1490 /* Don't know if moving this to the end will break anything. However,
1587 * we want to have update_look set above before calling this. 1491 * we want to have floorbox_update called before calling this.
1588 * 1492 *
1589 * check_move_on() must be after this because code called from 1493 * check_move_on() must be after this because code called from
1590 * check_move_on() depends on correct map flags (so functions like 1494 * check_move_on() depends on correct map flags (so functions like
1591 * blocked() and wall() work properly), and these flags are updated by 1495 * blocked() and wall() work properly), and these flags are updated by
1592 * update_object(). 1496 * update_object().
1622 tmp1; 1526 tmp1;
1623 1527
1624 /* first search for itself and remove any old instances */ 1528 /* first search for itself and remove any old instances */
1625 1529
1626 for (tmp = GET_MAP_OB (op->map, op->x, op->y); tmp != NULL; tmp = tmp->above) 1530 for (tmp = GET_MAP_OB (op->map, op->x, op->y); tmp != NULL; tmp = tmp->above)
1627 {
1628 if (!strcmp (tmp->arch->name, arch_string)) /* same archetype */ 1531 if (!strcmp (tmp->arch->name, arch_string)) /* same archetype */
1629 { 1532 tmp->destroy ();
1630 remove_ob (tmp);
1631 free_object (tmp);
1632 }
1633 }
1634 1533
1635 tmp1 = arch_to_object (find_archetype (arch_string)); 1534 tmp1 = arch_to_object (archetype::find (arch_string));
1636 1535
1637 tmp1->x = op->x; 1536 tmp1->x = op->x;
1638 tmp1->y = op->y; 1537 tmp1->y = op->y;
1639 insert_ob_in_map (tmp1, op->map, op, 0); 1538 insert_ob_in_map (tmp1, op->map, op, 0);
1640} 1539}
1648 */ 1547 */
1649 1548
1650object * 1549object *
1651get_split_ob (object *orig_ob, uint32 nr) 1550get_split_ob (object *orig_ob, uint32 nr)
1652{ 1551{
1653 object * 1552 object *newob;
1654 newob;
1655 int
1656 is_removed = (QUERY_FLAG (orig_ob, FLAG_REMOVED) != 0); 1553 int is_removed = (QUERY_FLAG (orig_ob, FLAG_REMOVED) != 0);
1657 1554
1658 if (orig_ob->nrof < nr) 1555 if (orig_ob->nrof < nr)
1659 { 1556 {
1660 sprintf (errmsg, "There are only %d %ss.", orig_ob->nrof ? orig_ob->nrof : 1, &orig_ob->name); 1557 sprintf (errmsg, "There are only %d %ss.", orig_ob->nrof ? orig_ob->nrof : 1, &orig_ob->name);
1661 return NULL; 1558 return NULL;
1662 } 1559 }
1663 1560
1664 newob = object_create_clone (orig_ob); 1561 newob = object_create_clone (orig_ob);
1665 1562
1666 if ((orig_ob->nrof -= nr) < 1) 1563 if ((orig_ob->nrof -= nr) < 1)
1667 { 1564 orig_ob->destroy (1);
1668 if (!is_removed)
1669 remove_ob (orig_ob);
1670 free_object2 (orig_ob, 1);
1671 }
1672 else if (!is_removed) 1565 else if (!is_removed)
1673 { 1566 {
1674 if (orig_ob->env != NULL) 1567 if (orig_ob->env != NULL)
1675 sub_weight (orig_ob->env, orig_ob->weight * nr); 1568 sub_weight (orig_ob->env, orig_ob->weight * nr);
1676 if (orig_ob->env == NULL && orig_ob->map->in_memory != MAP_IN_MEMORY) 1569 if (orig_ob->env == NULL && orig_ob->map->in_memory != MAP_IN_MEMORY)
1706 if (i > op->nrof) 1599 if (i > op->nrof)
1707 i = op->nrof; 1600 i = op->nrof;
1708 1601
1709 if (QUERY_FLAG (op, FLAG_REMOVED)) 1602 if (QUERY_FLAG (op, FLAG_REMOVED))
1710 op->nrof -= i; 1603 op->nrof -= i;
1711 else if (op->env != NULL) 1604 else if (op->env)
1712 { 1605 {
1713 /* is this object in the players inventory, or sub container 1606 /* is this object in the players inventory, or sub container
1714 * therein? 1607 * therein?
1715 */ 1608 */
1716 tmp = is_player_inv (op->env); 1609 tmp = op->in_player ();
1717 /* nope. Is this a container the player has opened? 1610 /* nope. Is this a container the player has opened?
1718 * If so, set tmp to that player. 1611 * If so, set tmp to that player.
1719 * IMO, searching through all the players will mostly 1612 * IMO, searching through all the players will mostly
1720 * likely be quicker than following op->env to the map, 1613 * likely be quicker than following op->env to the map,
1721 * and then searching the map for a player. 1614 * and then searching the map for a player.
1722 */ 1615 */
1723 if (!tmp) 1616 if (!tmp)
1724 { 1617 {
1725 for (pl = first_player; pl; pl = pl->next) 1618 for (pl = first_player; pl; pl = pl->next)
1726 if (pl->ob->container == op->env) 1619 if (pl->ob->container == op->env)
1620 {
1621 tmp = pl->ob;
1727 break; 1622 break;
1728 if (pl) 1623 }
1729 tmp = pl->ob;
1730 else
1731 tmp = NULL;
1732 } 1624 }
1733 1625
1734 if (i < op->nrof) 1626 if (i < op->nrof)
1735 { 1627 {
1736 sub_weight (op->env, op->weight * i); 1628 sub_weight (op->env, op->weight * i);
1737 op->nrof -= i; 1629 op->nrof -= i;
1738 if (tmp) 1630 if (tmp)
1739 {
1740 esrv_send_item (tmp, op); 1631 esrv_send_item (tmp, op);
1741 }
1742 } 1632 }
1743 else 1633 else
1744 { 1634 {
1745 remove_ob (op); 1635 op->remove ();
1746 op->nrof = 0; 1636 op->nrof = 0;
1747 if (tmp) 1637 if (tmp)
1748 {
1749 esrv_del_item (tmp->contr, op->count); 1638 esrv_del_item (tmp->contr, op->count);
1750 }
1751 } 1639 }
1752 } 1640 }
1753 else 1641 else
1754 { 1642 {
1755 object *above = op->above; 1643 object *above = op->above;
1756 1644
1757 if (i < op->nrof) 1645 if (i < op->nrof)
1758 op->nrof -= i; 1646 op->nrof -= i;
1759 else 1647 else
1760 { 1648 {
1761 remove_ob (op); 1649 op->remove ();
1762 op->nrof = 0; 1650 op->nrof = 0;
1763 } 1651 }
1764 1652
1765 /* Since we just removed op, op->above is null */ 1653 /* Since we just removed op, op->above is null */
1766 for (tmp = above; tmp != NULL; tmp = tmp->above) 1654 for (tmp = above; tmp; tmp = tmp->above)
1767 if (tmp->type == PLAYER) 1655 if (tmp->type == PLAYER)
1768 { 1656 {
1769 if (op->nrof) 1657 if (op->nrof)
1770 esrv_send_item (tmp, op); 1658 esrv_send_item (tmp, op);
1771 else 1659 else
1775 1663
1776 if (op->nrof) 1664 if (op->nrof)
1777 return op; 1665 return op;
1778 else 1666 else
1779 { 1667 {
1780 free_object (op); 1668 op->destroy ();
1781 return NULL; 1669 return 0;
1782 } 1670 }
1783} 1671}
1784 1672
1785/* 1673/*
1786 * add_weight(object, weight) adds the specified weight to an object, 1674 * add_weight(object, weight) adds the specified weight to an object,
1798 op->carrying += weight; 1686 op->carrying += weight;
1799 op = op->env; 1687 op = op->env;
1800 } 1688 }
1801} 1689}
1802 1690
1691object *
1692insert_ob_in_ob (object *op, object *where)
1693{
1694 if (!where)
1695 {
1696 char *dump = dump_object (op);
1697 LOG (llevError, "Trying to put object in NULL.\n%s\n", dump);
1698 free (dump);
1699 return op;
1700 }
1701
1702 if (where->head)
1703 {
1704 LOG (llevDebug, "Warning: Tried to insert object into wrong part of multipart object.\n");
1705 where = where->head;
1706 }
1707
1708 return where->insert (op);
1709}
1710
1803/* 1711/*
1804 * insert_ob_in_ob(op,environment): 1712 * env->insert (op)
1805 * This function inserts the object op in the linked list 1713 * This function inserts the object op in the linked list
1806 * inside the object environment. 1714 * inside the object environment.
1807 * 1715 *
1808 * Eneq(@csd.uu.se): Altered insert_ob_in_ob to make things picked up enter
1809 * the inventory at the last position or next to other objects of the same
1810 * type.
1811 * Frank: Now sorted by type, archetype and magic!
1812 *
1813 * The function returns now pointer to inserted item, and return value can 1716 * The function returns now pointer to inserted item, and return value can
1814 * be != op, if items are merged. -Tero 1717 * be != op, if items are merged. -Tero
1815 */ 1718 */
1816 1719
1817object * 1720object *
1818insert_ob_in_ob (object *op, object *where) 1721object::insert (object *op)
1819{ 1722{
1820 object * 1723 object *tmp, *otmp;
1821 tmp, *
1822 otmp;
1823 1724
1824 if (!QUERY_FLAG (op, FLAG_REMOVED)) 1725 if (!QUERY_FLAG (op, FLAG_REMOVED))
1825 { 1726 op->remove ();
1826 dump_object (op);
1827 LOG (llevError, "Trying to insert (ob) inserted object.\n%s\n", errmsg);
1828 return op;
1829 }
1830
1831 if (where == NULL)
1832 {
1833 dump_object (op);
1834 LOG (llevError, "Trying to put object in NULL.\n%s\n", errmsg);
1835 return op;
1836 }
1837
1838 if (where->head)
1839 {
1840 LOG (llevDebug, "Warning: Tried to insert object wrong part of multipart object.\n");
1841 where = where->head;
1842 }
1843 1727
1844 if (op->more) 1728 if (op->more)
1845 { 1729 {
1846 LOG (llevError, "Tried to insert multipart object %s (%d)\n", &op->name, op->count); 1730 LOG (llevError, "Tried to insert multipart object %s (%d)\n", &op->name, op->count);
1847 return op; 1731 return op;
1849 1733
1850 CLEAR_FLAG (op, FLAG_OBJ_ORIGINAL); 1734 CLEAR_FLAG (op, FLAG_OBJ_ORIGINAL);
1851 CLEAR_FLAG (op, FLAG_REMOVED); 1735 CLEAR_FLAG (op, FLAG_REMOVED);
1852 if (op->nrof) 1736 if (op->nrof)
1853 { 1737 {
1854 for (tmp = where->inv; tmp != NULL; tmp = tmp->below) 1738 for (tmp = inv; tmp != NULL; tmp = tmp->below)
1855 if (CAN_MERGE (tmp, op)) 1739 if (object::can_merge (tmp, op))
1856 { 1740 {
1857 /* return the original object and remove inserted object 1741 /* return the original object and remove inserted object
1858 (client needs the original object) */ 1742 (client needs the original object) */
1859 tmp->nrof += op->nrof; 1743 tmp->nrof += op->nrof;
1860 /* Weight handling gets pretty funky. Since we are adding to 1744 /* Weight handling gets pretty funky. Since we are adding to
1861 * tmp->nrof, we need to increase the weight. 1745 * tmp->nrof, we need to increase the weight.
1862 */ 1746 */
1863 add_weight (where, op->weight * op->nrof); 1747 add_weight (this, op->weight * op->nrof);
1864 SET_FLAG (op, FLAG_REMOVED); 1748 SET_FLAG (op, FLAG_REMOVED);
1865 free_object (op); /* free the inserted object */ 1749 op->destroy (); /* free the inserted object */
1866 op = tmp; 1750 op = tmp;
1867 remove_ob (op); /* and fix old object's links */ 1751 op->remove (); /* and fix old object's links */
1868 CLEAR_FLAG (op, FLAG_REMOVED); 1752 CLEAR_FLAG (op, FLAG_REMOVED);
1869 break; 1753 break;
1870 } 1754 }
1871 1755
1872 /* I assume combined objects have no inventory 1756 /* I assume combined objects have no inventory
1873 * We add the weight - this object could have just been removed 1757 * We add the weight - this object could have just been removed
1874 * (if it was possible to merge). calling remove_ob will subtract 1758 * (if it was possible to merge). calling remove_ob will subtract
1875 * the weight, so we need to add it in again, since we actually do 1759 * the weight, so we need to add it in again, since we actually do
1876 * the linking below 1760 * the linking below
1877 */ 1761 */
1878 add_weight (where, op->weight * op->nrof); 1762 add_weight (this, op->weight * op->nrof);
1879 } 1763 }
1880 else 1764 else
1881 add_weight (where, (op->weight + op->carrying)); 1765 add_weight (this, (op->weight + op->carrying));
1882 1766
1883 otmp = is_player_inv (where); 1767 otmp = this->in_player ();
1884 if (otmp && otmp->contr != NULL) 1768 if (otmp && otmp->contr)
1885 {
1886 if (!QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER)) 1769 if (!QUERY_FLAG (otmp, FLAG_NO_FIX_PLAYER))
1887 fix_player (otmp); 1770 fix_player (otmp);
1888 }
1889 1771
1890 op->map = NULL; 1772 op->map = 0;
1891 op->env = where; 1773 op->env = this;
1892 op->above = NULL; 1774 op->above = 0;
1893 op->below = NULL; 1775 op->below = 0;
1894 op->x = 0, op->y = 0; 1776 op->x = 0, op->y = 0;
1895 1777
1896 /* reset the light list and los of the players on the map */ 1778 /* reset the light list and los of the players on the map */
1897 if ((op->glow_radius != 0) && where->map) 1779 if ((op->glow_radius != 0) && map)
1898 { 1780 {
1899#ifdef DEBUG_LIGHTS 1781#ifdef DEBUG_LIGHTS
1900 LOG (llevDebug, " insert_ob_in_ob(): got %s to insert in map/op\n", op->name); 1782 LOG (llevDebug, " insert_ob_in_ob(): got %s to insert in map/op\n", op->name);
1901#endif /* DEBUG_LIGHTS */ 1783#endif /* DEBUG_LIGHTS */
1902 if (MAP_DARKNESS (where->map)) 1784 if (MAP_DARKNESS (map))
1903 update_all_los (where->map, where->x, where->y); 1785 update_all_los (map, x, y);
1904 } 1786 }
1905 1787
1906 /* Client has no idea of ordering so lets not bother ordering it here. 1788 /* Client has no idea of ordering so lets not bother ordering it here.
1907 * It sure simplifies this function... 1789 * It sure simplifies this function...
1908 */ 1790 */
1909 if (where->inv == NULL) 1791 if (!inv)
1910 where->inv = op; 1792 inv = op;
1911 else 1793 else
1912 { 1794 {
1913 op->below = where->inv; 1795 op->below = inv;
1914 op->below->above = op; 1796 op->below->above = op;
1915 where->inv = op; 1797 inv = op;
1916 } 1798 }
1799
1917 return op; 1800 return op;
1918} 1801}
1919 1802
1920/* 1803/*
1921 * Checks if any objects has a move_type that matches objects 1804 * Checks if any objects has a move_type that matches objects
1939 */ 1822 */
1940 1823
1941int 1824int
1942check_move_on (object *op, object *originator) 1825check_move_on (object *op, object *originator)
1943{ 1826{
1944 object * 1827 object *tmp;
1945 tmp; 1828 maptile *m = op->map;
1946 tag_t
1947 tag;
1948 mapstruct *
1949 m = op->map;
1950 int
1951 x = op->x, y = op->y; 1829 int x = op->x, y = op->y;
1952 1830
1953 MoveType 1831 MoveType move_on, move_slow, move_block;
1954 move_on,
1955 move_slow,
1956 move_block;
1957 1832
1958 if (QUERY_FLAG (op, FLAG_NO_APPLY)) 1833 if (QUERY_FLAG (op, FLAG_NO_APPLY))
1959 return 0; 1834 return 0;
1960
1961 tag = op->count;
1962 1835
1963 move_on = GET_MAP_MOVE_ON (op->map, op->x, op->y); 1836 move_on = GET_MAP_MOVE_ON (op->map, op->x, op->y);
1964 move_slow = GET_MAP_MOVE_SLOW (op->map, op->x, op->y); 1837 move_slow = GET_MAP_MOVE_SLOW (op->map, op->x, op->y);
1965 move_block = GET_MAP_MOVE_BLOCK (op->map, op->x, op->y); 1838 move_block = GET_MAP_MOVE_BLOCK (op->map, op->x, op->y);
1966 1839
2027 if ((!op->move_type && tmp->move_on & MOVE_WALK) || 1900 if ((!op->move_type && tmp->move_on & MOVE_WALK) ||
2028 ((op->move_type & tmp->move_on) && (op->move_type & ~tmp->move_on & ~tmp->move_block) == 0)) 1901 ((op->move_type & tmp->move_on) && (op->move_type & ~tmp->move_on & ~tmp->move_block) == 0))
2029 { 1902 {
2030 move_apply (tmp, op, originator); 1903 move_apply (tmp, op, originator);
2031 1904
2032 if (was_destroyed (op, tag)) 1905 if (op->destroyed ())
2033 return 1; 1906 return 1;
2034 1907
2035 /* what the person/creature stepped onto has moved the object 1908 /* what the person/creature stepped onto has moved the object
2036 * someplace new. Don't process any further - if we did, 1909 * someplace new. Don't process any further - if we did,
2037 * have a feeling strange problems would result. 1910 * have a feeling strange problems would result.
2049 * a matching archetype at the given map and coordinates. 1922 * a matching archetype at the given map and coordinates.
2050 * The first matching object is returned, or NULL if none. 1923 * The first matching object is returned, or NULL if none.
2051 */ 1924 */
2052 1925
2053object * 1926object *
2054present_arch (const archetype *at, mapstruct *m, int x, int y) 1927present_arch (const archetype *at, maptile *m, int x, int y)
2055{ 1928{
2056 object * 1929 object *
2057 tmp; 1930 tmp;
2058 1931
2059 if (m == NULL || out_of_map (m, x, y)) 1932 if (m == NULL || out_of_map (m, x, y))
2072 * a matching type variable at the given map and coordinates. 1945 * a matching type variable at the given map and coordinates.
2073 * The first matching object is returned, or NULL if none. 1946 * The first matching object is returned, or NULL if none.
2074 */ 1947 */
2075 1948
2076object * 1949object *
2077present (unsigned char type, mapstruct *m, int x, int y) 1950present (unsigned char type, maptile *m, int x, int y)
2078{ 1951{
2079 object * 1952 object *
2080 tmp; 1953 tmp;
2081 1954
2082 if (out_of_map (m, x, y)) 1955 if (out_of_map (m, x, y))
2222 * the archetype because that isn't correct if the monster has been 2095 * the archetype because that isn't correct if the monster has been
2223 * customized, changed states, etc. 2096 * customized, changed states, etc.
2224 */ 2097 */
2225 2098
2226int 2099int
2227find_free_spot (const object *ob, mapstruct *m, int x, int y, int start, int stop) 2100find_free_spot (const object *ob, maptile *m, int x, int y, int start, int stop)
2228{ 2101{
2229 int 2102 int
2230 i, 2103 i,
2231 index = 0, flag; 2104 index = 0, flag;
2232 static int 2105 static int
2244 * if the space being examined only has a wall to the north and empty 2117 * if the space being examined only has a wall to the north and empty
2245 * spaces in all the other directions, this will reduce the search space 2118 * spaces in all the other directions, this will reduce the search space
2246 * to only the spaces immediately surrounding the target area, and 2119 * to only the spaces immediately surrounding the target area, and
2247 * won't look 2 spaces south of the target space. 2120 * won't look 2 spaces south of the target space.
2248 */ 2121 */
2249 else if ((flag & AB_NO_PASS) && maxfree[i] < stop) 2122 else if ((flag & P_NO_PASS) && maxfree[i] < stop)
2250 stop = maxfree[i]; 2123 stop = maxfree[i];
2251 } 2124 }
2125
2252 if (!index) 2126 if (!index)
2253 return -1; 2127 return -1;
2128
2254 return altern[RANDOM () % index]; 2129 return altern[RANDOM () % index];
2255} 2130}
2256 2131
2257/* 2132/*
2258 * find_first_free_spot(archetype, mapstruct, x, y) works like 2133 * find_first_free_spot(archetype, maptile, x, y) works like
2259 * find_free_spot(), but it will search max number of squares. 2134 * find_free_spot(), but it will search max number of squares.
2260 * But it will return the first available spot, not a random choice. 2135 * But it will return the first available spot, not a random choice.
2261 * Changed 0.93.2: Have it return -1 if there is no free spot available. 2136 * Changed 0.93.2: Have it return -1 if there is no free spot available.
2262 */ 2137 */
2263 2138
2264int 2139int
2265find_first_free_spot (const object *ob, mapstruct *m, int x, int y) 2140find_first_free_spot (const object *ob, maptile *m, int x, int y)
2266{ 2141{
2267 int 2142 int
2268 i; 2143 i;
2269 2144
2270 for (i = 0; i < SIZEOFFREE; i++) 2145 for (i = 0; i < SIZEOFFREE; i++)
2334 * because we have to know what movement the thing looking to move 2209 * because we have to know what movement the thing looking to move
2335 * there is capable of. 2210 * there is capable of.
2336 */ 2211 */
2337 2212
2338int 2213int
2339find_dir (mapstruct *m, int x, int y, object *exclude) 2214find_dir (maptile *m, int x, int y, object *exclude)
2340{ 2215{
2341 int 2216 int
2342 i, 2217 i,
2343 max = SIZEOFFREE, mflags; 2218 max = SIZEOFFREE, mflags;
2344 2219
2345 sint16 nx, ny; 2220 sint16 nx, ny;
2346 object * 2221 object *
2347 tmp; 2222 tmp;
2348 mapstruct * 2223 maptile *
2349 mp; 2224 mp;
2350 2225
2351 MoveType blocked, move_type; 2226 MoveType blocked, move_type;
2352 2227
2353 if (exclude && exclude->head) 2228 if (exclude && exclude->head)
2561 * Modified to be map tile aware -.MSW 2436 * Modified to be map tile aware -.MSW
2562 */ 2437 */
2563 2438
2564 2439
2565int 2440int
2566can_see_monsterP (mapstruct *m, int x, int y, int dir) 2441can_see_monsterP (maptile *m, int x, int y, int dir)
2567{ 2442{
2568 sint16 dx, dy; 2443 sint16 dx, dy;
2569 int 2444 int
2570 mflags; 2445 mflags;
2571 2446
2620 * create clone from object to another 2495 * create clone from object to another
2621 */ 2496 */
2622object * 2497object *
2623object_create_clone (object *asrc) 2498object_create_clone (object *asrc)
2624{ 2499{
2625 object *
2626 dst = NULL, *tmp, *src, *part, *prev, *item; 2500 object *dst = 0, *tmp, *src, *part, *prev, *item;
2627 2501
2628 if (!asrc) 2502 if (!asrc)
2629 return NULL; 2503 return 0;
2504
2630 src = asrc; 2505 src = asrc;
2631 if (src->head) 2506 if (src->head)
2632 src = src->head; 2507 src = src->head;
2633 2508
2634 prev = NULL; 2509 prev = 0;
2635 for (part = src; part; part = part->more) 2510 for (part = src; part; part = part->more)
2636 { 2511 {
2637 tmp = get_object (); 2512 tmp = part->clone ();
2638 copy_object (part, tmp);
2639 tmp->x -= src->x; 2513 tmp->x -= src->x;
2640 tmp->y -= src->y; 2514 tmp->y -= src->y;
2515
2641 if (!part->head) 2516 if (!part->head)
2642 { 2517 {
2643 dst = tmp; 2518 dst = tmp;
2644 tmp->head = NULL; 2519 tmp->head = 0;
2645 } 2520 }
2646 else 2521 else
2647 { 2522 {
2648 tmp->head = dst; 2523 tmp->head = dst;
2649 } 2524 }
2525
2650 tmp->more = NULL; 2526 tmp->more = 0;
2527
2651 if (prev) 2528 if (prev)
2652 prev->more = tmp; 2529 prev->more = tmp;
2530
2653 prev = tmp; 2531 prev = tmp;
2654 } 2532 }
2655 2533
2656 /*** copy inventory ***/
2657 for (item = src->inv; item; item = item->below) 2534 for (item = src->inv; item; item = item->below)
2658 {
2659 (void) insert_ob_in_ob (object_create_clone (item), dst); 2535 insert_ob_in_ob (object_create_clone (item), dst);
2660 }
2661 2536
2662 return dst; 2537 return dst;
2663} 2538}
2664 2539
2665/* return true if the object was destroyed, 0 otherwise */
2666int
2667was_destroyed (const object *op, tag_t old_tag)
2668{
2669 /* checking for FLAG_FREED isn't necessary, but makes this function more
2670 * robust */
2671 return op->count != old_tag || QUERY_FLAG (op, FLAG_FREED);
2672}
2673
2674/* GROS - Creates an object using a string representing its content. */ 2540/* GROS - Creates an object using a string representing its content. */
2675
2676/* Basically, we save the content of the string to a temp file, then call */ 2541/* Basically, we save the content of the string to a temp file, then call */
2677
2678/* load_object on it. I admit it is a highly inefficient way to make things, */ 2542/* load_object on it. I admit it is a highly inefficient way to make things, */
2679
2680/* but it was simple to make and allows reusing the load_object function. */ 2543/* but it was simple to make and allows reusing the load_object function. */
2681
2682/* Remember not to use load_object_str in a time-critical situation. */ 2544/* Remember not to use load_object_str in a time-critical situation. */
2683
2684/* Also remember that multiparts objects are not supported for now. */ 2545/* Also remember that multiparts objects are not supported for now. */
2685 2546
2686object * 2547object *
2687load_object_str (const char *obstr) 2548load_object_str (const char *obstr)
2688{ 2549{
2689 object * 2550 object *op;
2690 op;
2691 char
2692 filename[MAX_BUF]; 2551 char filename[MAX_BUF];
2693 2552
2694 sprintf (filename, "%s/cfloadobstr2044", settings.tmpdir); 2553 sprintf (filename, "%s/cfloadobstr2044", settings.tmpdir);
2695 2554
2696 FILE *
2697 tempfile = fopen (filename, "w"); 2555 FILE *tempfile = fopen (filename, "w");
2698 2556
2699 if (tempfile == NULL) 2557 if (tempfile == NULL)
2700 { 2558 {
2701 LOG (llevError, "Error - Unable to access load object temp file\n"); 2559 LOG (llevError, "Error - Unable to access load object temp file\n");
2702 return NULL; 2560 return NULL;
2703 } 2561 }
2704 2562
2705 fprintf (tempfile, obstr); 2563 fprintf (tempfile, obstr);
2706 fclose (tempfile); 2564 fclose (tempfile);
2707 2565
2708 op = get_object (); 2566 op = object::create ();
2709 2567
2710 object_thawer thawer (filename); 2568 object_thawer thawer (filename);
2711 2569
2712 if (thawer) 2570 if (thawer)
2713 load_object (thawer, op, 0); 2571 load_object (thawer, op, 0);
2723 * returns NULL if no match. 2581 * returns NULL if no match.
2724 */ 2582 */
2725object * 2583object *
2726find_obj_by_type_subtype (const object *who, int type, int subtype) 2584find_obj_by_type_subtype (const object *who, int type, int subtype)
2727{ 2585{
2728 object * 2586 object *tmp;
2729 tmp;
2730 2587
2731 for (tmp = who->inv; tmp; tmp = tmp->below) 2588 for (tmp = who->inv; tmp; tmp = tmp->below)
2732 if (tmp->type == type && tmp->subtype == subtype) 2589 if (tmp->type == type && tmp->subtype == subtype)
2733 return tmp; 2590 return tmp;
2734 2591
2742 * do the desired thing. 2599 * do the desired thing.
2743 */ 2600 */
2744key_value * 2601key_value *
2745get_ob_key_link (const object *ob, const char *key) 2602get_ob_key_link (const object *ob, const char *key)
2746{ 2603{
2747 key_value * 2604 key_value *link;
2748 link;
2749 2605
2750 for (link = ob->key_values; link != NULL; link = link->next) 2606 for (link = ob->key_values; link != NULL; link = link->next)
2751 {
2752 if (link->key == key) 2607 if (link->key == key)
2753 {
2754 return link; 2608 return link;
2755 }
2756 }
2757 2609
2758 return NULL; 2610 return NULL;
2759} 2611}
2760 2612
2761/* 2613/*

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines