--- deliantra/server/common/object.C 2006/12/25 17:11:16 1.85 +++ deliantra/server/common/object.C 2006/12/26 10:51:52 1.91 @@ -143,7 +143,7 @@ */ /* For each field in wants, */ - for (wants_field = wants->key_values; wants_field != NULL; wants_field = wants_field->next) + for (wants_field = wants->key_values; wants_field; wants_field = wants_field->next) { key_value *has_field; @@ -524,7 +524,7 @@ } } - update_ob_speed (dst); + dst->set_speed (dst->speed); } object * @@ -556,65 +556,62 @@ * This function needs to be called whenever the speed of an object changes. */ void -update_ob_speed (object *op) +object::set_speed (float speed) { extern int arch_init; /* No reason putting the archetypes objects on the speed list, * since they never really need to be updated. */ - - if (QUERY_FLAG (op, FLAG_FREED) && op->speed) + if (flag [FLAG_FREED] && speed) { - LOG (llevError, "Object %s is freed but has speed.\n", &op->name); -#ifdef MANY_CORES - abort (); -#else - op->speed = 0; -#endif + LOG (llevError, "Object %s is freed but has speed.\n", &name); + speed = 0; } + this->speed = speed; + if (arch_init) return; - if (FABS (op->speed) > MIN_ACTIVE_SPEED) + if (FABS (speed) > MIN_ACTIVE_SPEED) { /* If already on active list, don't do anything */ - if (op->active_next || op->active_prev || op == active_objects) + if (active_next || active_prev || this == active_objects) return; /* process_events() expects us to insert the object at the beginning * of the list. */ - op->active_next = active_objects; + active_next = active_objects; - if (op->active_next != NULL) - op->active_next->active_prev = op; + if (active_next) + active_next->active_prev = this; - active_objects = op; + active_objects = this; } else { /* If not on the active list, nothing needs to be done */ - if (!op->active_next && !op->active_prev && op != active_objects) + if (!active_next && !active_prev && this != active_objects) return; - if (op->active_prev == NULL) + if (!active_prev) { - active_objects = op->active_next; + active_objects = active_next; - if (op->active_next != NULL) - op->active_next->active_prev = NULL; + if (active_next) + active_next->active_prev = 0; } else { - op->active_prev->active_next = op->active_next; + active_prev->active_next = active_next; - if (op->active_next) - op->active_next->active_prev = op->active_prev; + if (active_next) + active_next->active_prev = active_prev; } - op->active_next = NULL; - op->active_prev = NULL; + active_next = 0; + active_prev = 0; } } @@ -783,6 +780,45 @@ next = 0; } +/* + * Remove and free all objects in the inventory of the given object. + * object.c ? + */ +void +object::destroy_inv (bool drop_to_ground) +{ + /* Only if the space blocks everything do we not process - + * if some form of movement is allowed, let objects + * drop on that space. + */ + if (!drop_to_ground || !map || map->in_memory != MAP_IN_MEMORY || GET_MAP_MOVE_BLOCK (map, x, y) == MOVE_ALL) + { + while (inv) + inv->destroy (); + } + else + { /* Put objects in inventory onto this space */ + while (inv) + { + object *op = inv; + + if (op->flag [FLAG_STARTEQUIP] + || op->flag [FLAG_NO_DROP] + || op->type == RUNE + || op->type == TRAP + || op->flag [FLAG_IS_A_TEMPLATE]) + op->destroy (); + else + { + op->remove (); + op->x = x; + op->y = y; + insert_ob_in_map (op, map, 0, 0); /* Insert in same map as the envir */ + } + } + } +} + object *object::create () { object *op = new object; @@ -790,21 +826,9 @@ return op; } -/* - * free_object() frees everything allocated by an object, removes - * it from the list of used objects, and puts it on the list of - * free objects. The IS_FREED() flag is set in the object. - * The object must have been removed by remove_ob() first for - * this function to succeed. - * - * If destroy_inventory is set, free inventory as well. Else drop items in - * inventory to the ground. - */ void object::do_destroy () { - attachable::do_destroy (); - if (flag [FLAG_IS_LINKED]) remove_button_link (this); @@ -819,6 +843,12 @@ flag [FLAG_FREED] = 1; + attachable::do_destroy (); + + destroy_inv (true); + set_speed (0); + unlink (); + // hack to ensure that freed objects still have a valid map { static maptile *freed_map; // freed objects are moved here to avoid crashes @@ -839,9 +869,13 @@ y = 1; } - more = 0; head = 0; - inv = 0; + + if (more) + { + more->destroy (); + more = 0; + } // clear those pointers that likely might have circular references to us owner = 0; @@ -850,57 +884,6 @@ // only relevant for players(?), but make sure of it anyways contr = 0; - - /* Remove object from the active list */ - speed = 0; - update_ob_speed (this); - - unlink (); -} - -/* - * Remove and free all objects in the inventory of the given object. - * object.c ? - */ -void -object::destroy_inv (bool drop_to_ground) -{ - if (!inv) - return; - - /* Only if the space blocks everything do we not process - - * if some form of movement is allowed, let objects - * drop on that space. - */ - if (!drop_to_ground || !map || map->in_memory != MAP_IN_MEMORY || GET_MAP_MOVE_BLOCK (map, x, y) == MOVE_ALL) - { - while (inv) - { - inv->destroy_inv (drop_to_ground); - inv->destroy (); - } - } - else - { /* Put objects in inventory onto this space */ - while (inv) - { - object *op = inv; - - if (op->flag [FLAG_STARTEQUIP] - || op->flag [FLAG_NO_DROP] - || op->type == RUNE - || op->type == TRAP - || op->flag [FLAG_IS_A_TEMPLATE]) - op->destroy (); - else - { - op->remove (); - op->x = x; - op->y = y; - insert_ob_in_map (op, map, 0, 0); /* Insert in same map as the envir */ - } - } - } } void @@ -909,15 +892,8 @@ if (destroyed ()) return; - if (more) - { - //TODO: non-head objects must not have inventory - more->destroy (destroy_inventory); - more = 0; - } - if (destroy_inventory) - destroy_inv (true); + destroy_inv (false); attachable::destroy (); } @@ -1040,7 +1016,7 @@ int check_walk_off = !flag [FLAG_NO_APPLY]; - for (tmp = GET_MAP_OB (map, x, y); tmp; tmp = tmp->above) + for (tmp = map->at (x, y).bot; tmp; tmp = tmp->above) { /* No point updating the players look faces if he is the object * being removed.