--- deliantra/server/common/loader.C 2007/01/15 21:06:18 1.46 +++ deliantra/server/common/loader.C 2007/04/17 18:24:29 1.69 @@ -26,13 +26,11 @@ sub/add_weight will transcend the environment updating the carrying variable. */ - #include #include #include -/* Maps the MOVE_* values to names */ -static const char *const move_name[] = { "walk", "fly_low", "fly_high", "swim", "boat", NULL }; +///////////////////////////////////////////////////////////////////////////// /* This table is only necessary to convert objects that existed before the * spell object conversion to the new object. It was not practical @@ -254,7 +252,7 @@ NULL }; -extern int arch_init; +extern bool loading_arch; /* This function checks the object after it has been loaded (when we * get the 'end' in the input stream). This function can be used to @@ -264,8 +262,6 @@ static void check_loaded_object (object *op) { - int ip; - /* We do some specialised handling to handle legacy cases of name_pl. * If the object doesn't have a name_pl, we just use the object name - * this isn't perfect (things won't be properly pluralised), but works to @@ -276,7 +272,7 @@ * information (appear as just 'hearts' and not 'goblins heart') */ if (op->arch && op->name != op->arch->clone.name && op->name_pl == op->arch->clone.name_pl) - op->name_pl = NULL; + op->name_pl = 0; if (!op->name_pl) op->name_pl = op->name; @@ -289,33 +285,31 @@ else set_materialname (op, 5, NULL); } + /* only do these when program is first run - a bit * excessive to do this at every run - most of this is * really just to catch any errors - program will still run, but * not in the ideal fashion. */ - if ((op->type == WEAPON || op->type == BOW) && arch_init) + if ((op->type == WEAPON || op->type == BOW) && loading_arch) { if (!op->skill) - { - LOG (llevError, "Weapon %s lacks a skill.\n", op->debug_desc ()); - } + LOG (llevError, "Weapon %s lacks a skill.\n", op->debug_desc ()); else if ((!strcmp (op->skill, "one handed weapons") && op->body_info[1] != -1) || (!strcmp (op->skill, "two handed weapons") && op->body_info[1] != -2)) - { - LOG (llevError, "weapon %s arm usage does not match skill: %d, %s\n", op->debug_desc (), op->body_info[1], &op->skill); - } + LOG (llevError, "weapon %s arm usage does not match skill: %d, %s\n", op->debug_desc (), op->body_info[1], &op->skill); } /* We changed last_heal to gen_sp_armour, which is what it * really does for many objects. Need to catch any in maps * that may have an old value. */ - if ((op->type == WEAPON) || - (op->type == ARMOUR) || (op->type == HELMET) || - (op->type == SHIELD) || (op->type == RING) || - (op->type == BOOTS) || (op->type == GLOVES) || - (op->type == AMULET) || (op->type == GIRDLE) || (op->type == BRACERS) || (op->type == CLOAK)) + if (op->type == WEAPON + || op->type == ARMOUR || op->type == HELMET + || op->type == SHIELD || op->type == RING + || op->type == BOOTS || op->type == GLOVES + || op->type == AMULET || op->type == GIRDLE + || op->type == BRACERS || op->type == CLOAK) { if (op->last_heal) { @@ -324,16 +318,17 @@ op->last_heal = 0; } - ip = calc_item_power (op, 0); + int ip = calc_item_power (op, 0); + /* Legacy objects from before item power was in the game */ if (!op->item_power && ip) { if (ip > 3) - { - LOG (llevDebug, "Object %s had no item power, using %d\n", op->debug_desc (), ip); - } + LOG (llevDebug, "Object %s had no item power, using %d\n", op->debug_desc (), ip); + op->item_power = ip; } + /* Check for possibly bogus values. Has to meet both these criteria - * something that has item_power 1 is probably just fine if our calculated * value is 1 or 2 - these values are small enough that hard to be precise. @@ -341,32 +336,26 @@ * but not the second one. */ if (ip > 2 * op->item_power && ip > (op->item_power + 3)) - { - LOG (llevDebug, "Object %s seems to have too low item power? %d > %d\n", op->debug_desc (), ip, op->item_power); - } - + LOG (llevDebug, "Object %s seems to have too low item power? %d > %d\n", op->debug_desc (), ip, op->item_power); } + /* Old spellcasting object - need to load in the appropiate object */ if ((op->type == ROD || op->type == WAND || op->type == SCROLL || op->type == HORN || op->type == FIREWALL || /* POTIONS and ALTARS don't always cast spells, but if they do, update them */ - ((op->type == POTION || op->type == ALTAR) && op->stats.sp)) && !op->inv && !arch_init) + ((op->type == POTION || op->type == ALTAR) && op->stats.sp)) && !op->inv && !loading_arch) { - object *tmp; - /* Fireall is bizarre in that spell type was stored in dam. Rest are 'normal' * in that spell was stored in sp. */ - tmp = get_archetype (spell_mapping[op->type == FIREWALL ? op->stats.dam : op->stats.sp]); + object *tmp = get_archetype (spell_mapping[op->type == FIREWALL ? op->stats.dam : op->stats.sp]); insert_ob_in_ob (tmp, op); op->randomitems = NULL; /* So another spell isn't created for this object */ } - /* spellbooks & runes use slaying. But not to arch name, but to spell name */ - if ((op->type == SPELLBOOK || op->type == RUNE) && op->slaying && !op->inv && !arch_init) + /* spellbooks & runes use slaying. But not to arch name, but to spell name */ + if ((op->type == SPELLBOOK || op->type == RUNE) && op->slaying && !op->inv && !loading_arch) { - object *tmp; - - tmp = get_archetype_by_object_name (op->slaying); + object *tmp = get_archetype_by_object_name (op->slaying); insert_ob_in_ob (tmp, op); op->randomitems = NULL; /* So another spell isn't created for this object */ /* without this, value is all screwed up */ @@ -376,7 +365,10 @@ if (QUERY_FLAG (op, FLAG_MONSTER)) { if (op->stats.hp > op->stats.maxhp) - LOG (llevDebug, "Monster %s has hp set higher than maxhp (%d>%d)\n", op->debug_desc (), op->stats.hp, op->stats.maxhp); + { + LOG (llevDebug, "Monster %s has hp set higher than maxhp (%d>%d)\n", op->debug_desc (), op->stats.hp, op->stats.maxhp); + op->stats.maxhp = op->stats.hp; + } /* The archs just need to be updated for this */ if (op->move_type == 0) @@ -409,17 +401,28 @@ op->stats.maxhp = 0; } else - { - op->move_type = MOVE_WALK; - } + op->move_type = MOVE_WALK; } } } static void -set_move (MoveType & mt, const char *str) +set_move (MoveType &mt, const char *str) { + static const struct flagstr { + const char *name; + MoveType flags; + } move_flags[] = { + { "walk" , MOVE_WALK }, + { "flying" , MOVE_FLY_LOW | MOVE_FLY_HIGH }, + { "fly_low" , MOVE_FLY_LOW }, + { "fly_high", MOVE_FLY_HIGH }, + { "swim" , MOVE_SWIM }, + { "boat" , MOVE_BOAT }, + { "all" , MOVE_ALL }, + }; + if (!str) { mt = 0; @@ -436,298 +439,230 @@ for (str = strtok ((char *) str, " "); str; str = strtok (0, " ")) { - if (!strcasecmp (str, "all")) - mt |= MOVE_ALL; - else - { - int i, negate = 0; + bool negate = 0; - if (*str == '-') - { - negate = 1; - str++; - } - - for (i = 0; move_name[i]; i++) - { - if (!strcasecmp (move_name[i], str)) - { - if (negate) - mt &= ~(1 << i); - else - mt |= (1 << i); - - break; - } - } + if (*str == '-') + { + negate = 1; + str++; + } - if (!move_name[i]) + for (const flagstr *f = move_flags; f < move_flags + sizeof (move_flags) / sizeof (move_flags [0]); ++f) + { + if (!strcmp (f->name, str)) { - /* fly is a special case - covers both fly_low and - * fly_high - since it doesn't match to a specific - * single bit, have to special case it. - */ - if (!strcasecmp (str, "flying")) - { - if (negate) - mt &= ~MOVE_FLYING; - else - mt |= MOVE_FLYING; - } + if (negate) + mt &= ~f->flags; else - LOG (llevDebug, "common/loader.l: set_move - unknown move string '%s'\n", str); + mt |= f->flags; + + goto next; } } + + LOG (llevDebug, "common/loader.C: set_move - unknown move string '%s'\n", str); + +next: ; } } #define GET_FLAG(op,flag) \ - if (thawer.get_sint32 ()) \ + if (f.get_sint32 ()) \ SET_FLAG (op, flag); \ else \ CLEAR_FLAG (op, flag) \ -int -parse_object (object *op, object_thawer &thawer, int map_flags) +bool +object::parse_kv (object_thawer &f) { - bool ismore = 0; - object *op_inv = op->inv; - - for (;;) - { - keyword kw = thawer.get_kv (); + assert (f.kw == KW_object || f.kw == KW_arch); - switch (kw) - { - case KW_Object: // uppercase alias - case KW_object: - thawer.get (op->name); + if (f.kw == KW_object) + f.get (name); // preset name from object name - if (op->arch) - op->arch->name = op->name; + f.next (); - break; + object *op_inv = inv; + for (;;) + { + switch (f.kw) + { case KW_uuid: - if (const char *s = thawer.get_str ()) + if (const char *s = f.get_str ()) { unsigned int version; unsigned long long seq; if (2 == sscanf (s, "<%d.%llx>", &version, &seq) && version == 1) { - op->uuid.seq = seq; + uuid.seq = seq; break; } } - op->uuid = gen_uuid (); + uuid = gen_uuid (); break; case KW_oid: - thawer.get (op, thawer.get_sint32 ()); + f.get (this, f.get_sint32 ()); break; - case KW_name: thawer.get (op->name); break; - case KW_name_pl: thawer.get (op->name_pl); break; - case KW_title: thawer.get (op->title); break; - case KW_custom_name: thawer.get (op->custom_name); break; - - case KW_attach: thawer.get_ornull (op->attach); break; - case KW_skill: thawer.get_ornull (op->skill); break; - case KW_race: thawer.get_ornull (op->race); break; - case KW_slaying: thawer.get_ornull (op->slaying); break; + case KW_name: f.get (name); break; + case KW_name_pl: f.get (name_pl); break; + case KW_title: f.get (title); break; + case KW_custom_name: f.get (custom_name); break; + + case KW_attach: f.get_ornull (attach); break; + case KW_skill: f.get_ornull (skill); break; + case KW_race: f.get_ornull (race); break; + case KW_slaying: f.get_ornull (slaying); break; case KW_arch: { - const char *str = thawer.get_str (); + object *tmp = object::read (f); + tmp->deactivate (); - if (op->arch) - { - /* If op->arch has been set, then this new object - * must be part of the inventory. So process - * appropriately. - */ - archetype *arch = archetype::find (str); - - object *tmp; - - if (arch) - tmp = arch_to_object (arch); - else - { - tmp = object::create (); - /* record the name of the broken object */ - tmp->name = str; - } - - // decativate the object we, as we are still going - // to read the speed value. Objects should be activated - // when the map, object or player is completly loaded. - tmp->deactivate (); - - parse_object (tmp, thawer, map_flags); - - if (tmp->arch) - { - // was: insert_ob_in_ob (tmp, op); - // but manually adding it can improve map loading times a lot - // also, appending instead of prepending keeps the - // save ordering the same between repeated load/saves. - CLEAR_FLAG (tmp, FLAG_OBJ_ORIGINAL); - CLEAR_FLAG (tmp, FLAG_REMOVED); - - if (!op_inv) - { - op->inv = tmp; - tmp->above = 0; - } - else - { - while (op_inv->below) - op_inv = op_inv->below; - - op_inv->below = tmp; - tmp->above = op_inv; - } - - tmp->below = 0; - tmp->env = op; - op_inv = tmp; - } - else - { - LOG (llevDebug, "Discarding object without arch: %s\n", tmp->name ? (const char *) tmp->name : "(null)"); - tmp->destroy (); - } - } - else - { - /* This is the actual archetype definition then */ - op->arch = archetype::find (str); - - if (op->arch) - { - op->arch->clone.copy_to (op); - // copy_to activates, this should be fixed properly - op->deactivate (); - } - else if (!arch_init) - /* record the name of the broken object */ - op->name = str; - } + { + // was: insert_ob_in_ob (tmp, op); + // but manually adding it can improve map loading times a lot + // also, appending instead of prepending keeps the + // save ordering the same between repeated load/saves. + CLEAR_FLAG (tmp, FLAG_OBJ_ORIGINAL); + CLEAR_FLAG (tmp, FLAG_REMOVED); + + if (!op_inv) + { + inv = tmp; + tmp->above = 0; + } + else + { + while (op_inv->below) + op_inv = op_inv->below; + + op_inv->below = tmp; + tmp->above = op_inv; + } + + tmp->below = 0; + tmp->env = this; + op_inv = tmp; + } } - break; + continue; case KW_other_arch: - op->other_arch = archetype::find (thawer.get_str ()); + other_arch = + loading_arch + ? archetype::get (f.get_str ()) + : archetype::find (f.get_str ()); + + if (!other_arch) + LOG (llevError, "%s uses unknown other_arch '%s'.\n", debug_desc (), f.get_str ()); break; case KW_animation: { - const char *str = thawer.get_str (); + CLEAR_FLAG (this, FLAG_ANIMATE); + animation_id = 0; - if (!str) - { - op->animation_id = 0; - CLEAR_FLAG (op, FLAG_ANIMATE); - } - else if ((op->animation_id = find_animation (str))) - SET_FLAG (op, FLAG_ANIMATE); + const char *str = f.get_str (); + if (str && (animation_id = find_animation (str))) + SET_FLAG (this, FLAG_ANIMATE); } - break; - case KW_last_heal: thawer.get (op->last_heal); break; - case KW_last_sp: thawer.get (op->last_sp); break; - case KW_last_grace: thawer.get (op->last_grace); break; - case KW_last_eat: thawer.get (op->last_eat); break; - case KW_speed_left: thawer.get (op->speed_left); break; + case KW_last_heal: f.get (last_heal); break; + case KW_last_sp: f.get (last_sp); break; + case KW_last_grace: f.get (last_grace); break; + case KW_last_eat: f.get (last_eat); break; + case KW_speed_left: f.get (speed_left); break; case KW_speed: - thawer.get (op->speed); + f.get (speed); //TODO: maybe do in check_object // removed check for style maps - if (op->speed < 0) - op->speed_left = op->speed_left - RANDOM () % 100 / 100.0; + if (speed < 0) + speed_left = speed_left - rndm (); break; case KW_slow_move: - op->move_slow |= MOVE_WALK; - thawer.get (op->move_slow_penalty); + move_slow |= MOVE_WALK; + f.get (move_slow_penalty); break; case KW_face: - op->face = &new_faces[FindFace (thawer.get_str (), 0)]; + face = face_find (f.get_str ()); break; - case KW_x: thawer.get (op->x); break; - case KW_y: thawer.get (op->y); break; + case KW_x: f.get (x); break; + case KW_y: f.get (y); break; case KW_Str: // uppercase alias - case KW_str: thawer.get (op->stats.Str); break; + case KW_str: f.get (stats.Str); break; case KW_Dex: // uppercase alias - case KW_dex: thawer.get (op->stats.Dex); break; + case KW_dex: f.get (stats.Dex); break; case KW_Con: // uppercase alias - case KW_con: thawer.get (op->stats.Con); break; + case KW_con: f.get (stats.Con); break; case KW_Wis: // uppercase alias - case KW_wis: thawer.get (op->stats.Wis); break; + case KW_wis: f.get (stats.Wis); break; case KW_Cha: // uppercase alias - case KW_cha: thawer.get (op->stats.Cha); break; + case KW_cha: f.get (stats.Cha); break; case KW_Int: // uppercase alias - case KW_int: thawer.get (op->stats.Int); break; + case KW_int: f.get (stats.Int); break; case KW_Pow: // uppercase alias - case KW_pow: thawer.get (op->stats.Pow); break; + case KW_pow: f.get (stats.Pow); break; - case KW_hp: thawer.get (op->stats.hp); break; - case KW_maxhp: thawer.get (op->stats.maxhp); break; - case KW_sp: thawer.get (op->stats.sp); break; - case KW_maxsp: thawer.get (op->stats.maxsp); break; - case KW_grace: thawer.get (op->stats.grace); break; - case KW_maxgrace: thawer.get (op->stats.maxgrace); break; - - case KW_exp: thawer.get (op->stats.exp); break; - case KW_perm_exp: thawer.get (op->perm_exp); break; - case KW_food: thawer.get (op->stats.food); break; - case KW_dam: thawer.get (op->stats.dam); break; - case KW_wc: thawer.get (op->stats.wc); break; - case KW_ac: thawer.get (op->stats.ac); break; - case KW_nrof: thawer.get (op->nrof); break; - case KW_level: thawer.get (op->level); break; - case KW_direction: thawer.get (op->direction); break; - case KW_type: thawer.get (op->type); break; - case KW_subtype: thawer.get (op->subtype); break; - case KW_value: thawer.get (op->value); break; - case KW_weight: thawer.get (op->weight); break; - case KW_carrying: thawer.get (op->carrying); break; - case KW_attacktype: thawer.get (op->attacktype); break; - case KW_path_attuned: thawer.get (op->path_attuned); break; - case KW_path_repelled: thawer.get (op->path_repelled); break; - case KW_path_denied: thawer.get (op->path_denied); break; - case KW_invisible: thawer.get (op->invisible); break; - case KW_magic: thawer.get (op->magic); break; - case KW_state: thawer.get (op->state); break; - case KW_move_slow_penalty: thawer.get (op->move_slow_penalty); break; - case KW_material: thawer.get (op->material); break; - case KW_materialname: thawer.get (op->materialname); break; + case KW_hp: f.get (stats.hp); break; + case KW_maxhp: f.get (stats.maxhp); break; + case KW_sp: f.get (stats.sp); break; + case KW_maxsp: f.get (stats.maxsp); break; + case KW_grace: f.get (stats.grace); break; + case KW_maxgrace: f.get (stats.maxgrace); break; + + case KW_exp: f.get (stats.exp); break; + case KW_perm_exp: f.get (perm_exp); break; + case KW_food: f.get (stats.food); break; + case KW_dam: f.get (stats.dam); break; + case KW_wc: f.get (stats.wc); break; + case KW_ac: f.get (stats.ac); break; + case KW_nrof: f.get (nrof); break; + case KW_level: f.get (level); break; + case KW_direction: f.get (direction); break; + case KW_type: f.get (type); break; + case KW_subtype: f.get (subtype); break; + case KW_value: f.get (value); break; + case KW_weight: f.get (weight); break; + case KW_carrying: f.get (carrying); break; + case KW_attacktype: f.get (attacktype); break; + case KW_path_attuned: f.get (path_attuned); break; + case KW_path_repelled: f.get (path_repelled); break; + case KW_path_denied: f.get (path_denied); break; + case KW_invisible: f.get (invisible); break; + case KW_magic: f.get (magic); break; + case KW_state: f.get (state); break; + case KW_move_slow_penalty: f.get (move_slow_penalty); break; + case KW_material: f.get (materials); break; //TODO: nuke + case KW_materialname: f.get (materialname); break; /* These are the new values */ - case KW_move_block: set_move (op->move_block, thawer.get_str ()); break; - case KW_move_allow: set_move (op->move_allow, thawer.get_str ()); break; - case KW_move_type: set_move (op->move_type, thawer.get_str ()); break; - case KW_move_on: set_move (op->move_on, thawer.get_str ()); break; - case KW_move_off: set_move (op->move_off, thawer.get_str ()); break; - case KW_move_slow: set_move (op->move_slow, thawer.get_str ()); break; + case KW_move_block: set_move (move_block, f.get_str ()); break; + case KW_move_allow: set_move (move_allow, f.get_str ()); break; + case KW_move_type: set_move (move_type, f.get_str ()); break; + case KW_move_on: set_move (move_on, f.get_str ()); break; + case KW_move_off: set_move (move_off, f.get_str ()); break; + case KW_move_slow: set_move (move_slow, f.get_str ()); break; /* These are all legacy - any new objects should use the move_ .. values */ case KW_no_pass: { - if (thawer.get_sint32 ()) - op->move_block = MOVE_ALL; + if (f.get_sint32 ()) + move_block = MOVE_ALL; else - op->move_block = 0; + move_block = 0; } break; @@ -735,10 +670,10 @@ /* These are all legacy - any new objects should use the move_ .. values */ case KW_walk_on: { - if (thawer.get_sint32 ()) - op->move_on |= MOVE_WALK; + if (f.get_sint32 ()) + move_on |= MOVE_WALK; else - op->move_on &= ~MOVE_WALK; + move_on &= ~MOVE_WALK; } break; @@ -746,320 +681,331 @@ /* These are all legacy - any new objects should use the move_ .. values */ case KW_walk_off: { - if (thawer.get_sint32 ()) - op->move_off |= MOVE_WALK; + if (f.get_sint32 ()) + move_off |= MOVE_WALK; else - op->move_off &= ~MOVE_WALK; + move_off &= ~MOVE_WALK; } break; /* These are all legacy - any new objects should use the move_ .. values */ case KW_fly_on: { - if (thawer.get_sint32 ()) - op->move_on |= MOVE_FLY_LOW; + if (f.get_sint32 ()) + move_on |= MOVE_FLY_LOW; else - op->move_on &= ~MOVE_FLY_LOW; + move_on &= ~MOVE_FLY_LOW; } break; /* These are all legacy - any new objects should use the move_ .. values */ case KW_fly_off: { - if (thawer.get_sint32 ()) - op->move_off |= MOVE_FLY_LOW; + if (f.get_sint32 ()) + move_off |= MOVE_FLY_LOW; else - op->move_off &= ~MOVE_FLY_LOW; + move_off &= ~MOVE_FLY_LOW; } break; //TODO: remove these after converting archetypes case KW_can_use_wand: - GET_FLAG (op, FLAG_USE_RANGE); + GET_FLAG (this, FLAG_USE_RANGE); break; /* These are all legacy - any new objects should use the move_ .. values */ case KW_flying: { - if (thawer.get_sint32 ()) - op->move_type |= MOVE_FLY_LOW; + if (f.get_sint32 ()) + move_type |= MOVE_FLY_LOW; else - op->move_type &= ~MOVE_FLY_LOW; + move_type &= ~MOVE_FLY_LOW; } break; case KW_identified: - GET_FLAG (op, FLAG_IDENTIFIED); + GET_FLAG (this, FLAG_IDENTIFIED); //TODO: move to check_object or so - if (QUERY_FLAG (op, FLAG_IDENTIFIED)) - CLEAR_FLAG (op, FLAG_KNOWN_MAGICAL); + if (QUERY_FLAG (this, FLAG_IDENTIFIED)) + CLEAR_FLAG (this, FLAG_KNOWN_MAGICAL); break; case KW_wiz: - GET_FLAG (op, FLAG_WIZ); + GET_FLAG (this, FLAG_WIZ); //TODO: move to check_object - if (QUERY_FLAG (op, FLAG_WIZ)) + if (QUERY_FLAG (this, FLAG_WIZ)) { - SET_FLAG (op, FLAG_WAS_WIZ); - SET_FLAG (op, FLAG_WIZPASS); - SET_FLAG (op, FLAG_WIZCAST); + SET_FLAG (this, FLAG_WAS_WIZ); + SET_FLAG (this, FLAG_WIZPASS); + SET_FLAG (this, FLAG_WIZCAST); } else { - CLEAR_FLAG (op, FLAG_WIZPASS); - CLEAR_FLAG (op, FLAG_WIZCAST); + CLEAR_FLAG (this, FLAG_WIZPASS); + CLEAR_FLAG (this, FLAG_WIZCAST); } break; case KW_friendly: - if (thawer.get_sint32 ()) - if (op->type != PLAYER) - add_friendly_object (op); - - break; - - case KW_monster: GET_FLAG (op, FLAG_MONSTER); break; - case KW_neutral: GET_FLAG (op, FLAG_NEUTRAL); break; - case KW_no_attack: GET_FLAG (op, FLAG_NO_ATTACK); break; - case KW_no_damage: GET_FLAG (op, FLAG_NO_DAMAGE); break; - case KW_obj_original: GET_FLAG (op, FLAG_OBJ_ORIGINAL); break; - case KW_generator: GET_FLAG (op, FLAG_GENERATOR); break; - case KW_use_content_on_gen: GET_FLAG (op, FLAG_CONTENT_ON_GEN); break; - case KW_is_thrown: GET_FLAG (op, FLAG_IS_THROWN); break; - case KW_auto_apply: GET_FLAG (op, FLAG_AUTO_APPLY); break; - case KW_see_invisible: GET_FLAG (op, FLAG_SEE_INVISIBLE); break; - case KW_can_roll: GET_FLAG (op, FLAG_CAN_ROLL); break; - case KW_overlay_floor: GET_FLAG (op, FLAG_OVERLAY_FLOOR); break; - case KW_is_turnable: GET_FLAG (op, FLAG_IS_TURNABLE); break; - case KW_is_used_up: GET_FLAG (op, FLAG_IS_USED_UP); break; - case KW_alive: GET_FLAG (op, FLAG_ALIVE); break; - case KW_applied: GET_FLAG (op, FLAG_APPLIED); break; - case KW_unpaid: GET_FLAG (op, FLAG_UNPAID); break; - case KW_is_animated: GET_FLAG (op, FLAG_ANIMATE); break; - case KW_no_pick: GET_FLAG (op, FLAG_NO_PICK); break; - case KW_reflecting: GET_FLAG (op, FLAG_REFLECTING); break; - case KW_changing: GET_FLAG (op, FLAG_CHANGING); break; - case KW_splitting: GET_FLAG (op, FLAG_SPLITTING); break; - case KW_hitback: GET_FLAG (op, FLAG_HITBACK); break; - case KW_startequip: GET_FLAG (op, FLAG_STARTEQUIP); break; - case KW_blocksview: GET_FLAG (op, FLAG_BLOCKSVIEW); break; - case KW_undead: GET_FLAG (op, FLAG_UNDEAD); break; - case KW_scared: GET_FLAG (op, FLAG_SCARED); break; - case KW_unaggressive: GET_FLAG (op, FLAG_UNAGGRESSIVE); break; - case KW_reflect_missile: GET_FLAG (op, FLAG_REFL_MISSILE); break; - case KW_reflect_spell: GET_FLAG (op, FLAG_REFL_SPELL); break; - case KW_no_magic: GET_FLAG (op, FLAG_NO_MAGIC); break; - case KW_no_drop: GET_FLAG (op, FLAG_NO_DROP); break; - case KW_random_movement: GET_FLAG (op, FLAG_RANDOM_MOVE); break; - case KW_was_wiz: GET_FLAG (op, FLAG_WAS_WIZ); break; - case KW_no_fix_player: GET_FLAG (op, FLAG_NO_FIX_PLAYER); break; - case KW_is_lightable: GET_FLAG (op, FLAG_IS_LIGHTABLE); break; - case KW_tear_down: GET_FLAG (op, FLAG_TEAR_DOWN); break; - case KW_can_use_shield: GET_FLAG (op, FLAG_USE_SHIELD); break; - case KW_can_cast_spell: GET_FLAG (op, FLAG_CAST_SPELL); break; - case KW_can_use_scroll: GET_FLAG (op, FLAG_USE_SCROLL); break; - case KW_can_use_range: GET_FLAG (op, FLAG_USE_RANGE); break; - case KW_can_use_bow: GET_FLAG (op, FLAG_USE_BOW); break; - case KW_can_use_armour: GET_FLAG (op, FLAG_USE_ARMOUR); break; - case KW_can_use_weapon: GET_FLAG (op, FLAG_USE_WEAPON); break; - case KW_can_use_ring: GET_FLAG (op, FLAG_USE_RING); break; - case KW_has_ready_bow: GET_FLAG (op, FLAG_READY_BOW); break; - case KW_has_ready_range: GET_FLAG (op, FLAG_READY_RANGE); break; - case KW_xrays: GET_FLAG (op, FLAG_XRAYS); break; - case KW_is_floor: GET_FLAG (op, FLAG_IS_FLOOR); break; - case KW_lifesave: GET_FLAG (op, FLAG_LIFESAVE); break; - case KW_no_strength: GET_FLAG (op, FLAG_NO_STRENGTH); break; - case KW_sleep: GET_FLAG (op, FLAG_SLEEP); break; - case KW_stand_still: GET_FLAG (op, FLAG_STAND_STILL); break; - case KW_random_move: GET_FLAG (op, FLAG_RANDOM_MOVE); break; - case KW_only_attack: GET_FLAG (op, FLAG_ONLY_ATTACK); break; - case KW_activate_on_push: GET_FLAG (op, FLAG_ACTIVATE_ON_PUSH); break; - case KW_activate_on_release: GET_FLAG (op, FLAG_ACTIVATE_ON_RELEASE); break; - case KW_confused: GET_FLAG (op, FLAG_CONFUSED); break; - case KW_stealth: GET_FLAG (op, FLAG_STEALTH); break; - case KW_cursed: GET_FLAG (op, FLAG_CURSED); break; - case KW_damned: GET_FLAG (op, FLAG_DAMNED); break; - case KW_see_anywhere: GET_FLAG (op, FLAG_SEE_ANYWHERE); break; - case KW_known_magical: GET_FLAG (op, FLAG_KNOWN_MAGICAL); break; - case KW_known_cursed: GET_FLAG (op, FLAG_KNOWN_CURSED); break; - case KW_can_use_skill: GET_FLAG (op, FLAG_CAN_USE_SKILL); break; - case KW_been_applied: GET_FLAG (op, FLAG_BEEN_APPLIED); break; - case KW_has_ready_scroll: GET_FLAG (op, FLAG_READY_SCROLL); break; - case KW_can_use_rod: GET_FLAG (op, FLAG_USE_ROD); break; - case KW_can_use_horn: GET_FLAG (op, FLAG_USE_HORN); break; - case KW_unique: GET_FLAG (op, FLAG_UNIQUE); break; - case KW_make_invisible: GET_FLAG (op, FLAG_MAKE_INVIS); break; - case KW_inv_locked: GET_FLAG (op, FLAG_INV_LOCKED); break; - case KW_is_wooded: GET_FLAG (op, FLAG_IS_WOODED); break; - case KW_is_hilly: GET_FLAG (op, FLAG_IS_HILLY); break; - case KW_is_water: GET_FLAG (op, FLAG_IS_WATER); break; - case KW_has_ready_skill: GET_FLAG (op, FLAG_READY_SKILL); break; - case KW_has_ready_weapon: GET_FLAG (op, FLAG_READY_WEAPON); break; - case KW_no_skill_ident: GET_FLAG (op, FLAG_NO_SKILL_IDENT); break; - case KW_is_blind: GET_FLAG (op, FLAG_BLIND); break; - case KW_can_see_in_dark: GET_FLAG (op, FLAG_SEE_IN_DARK); break; - case KW_is_cauldron: GET_FLAG (op, FLAG_IS_CAULDRON); break; - case KW_no_steal: GET_FLAG (op, FLAG_NO_STEAL); break; - case KW_one_hit: GET_FLAG (op, FLAG_ONE_HIT); break; - case KW_berserk: GET_FLAG (op, FLAG_BERSERK); break; - case KW_is_buildable: GET_FLAG (op, FLAG_IS_BUILDABLE); break; - case KW_destroy_on_death: GET_FLAG (op, FLAG_DESTROY_ON_DEATH); break; - - case KW_armour: thawer.get (op->resist[ATNR_PHYSICAL]); break; - case KW_resist_physical: thawer.get (op->resist[ATNR_PHYSICAL]); break; - case KW_resist_magic: thawer.get (op->resist[ATNR_MAGIC]); break; - case KW_resist_fire: thawer.get (op->resist[ATNR_FIRE]); break; - case KW_resist_electricity: thawer.get (op->resist[ATNR_ELECTRICITY]); break; - case KW_resist_cold: thawer.get (op->resist[ATNR_COLD]); break; - case KW_resist_confusion: thawer.get (op->resist[ATNR_CONFUSION]); break; - case KW_resist_acid: thawer.get (op->resist[ATNR_ACID]); break; - case KW_resist_drain: thawer.get (op->resist[ATNR_DRAIN]); break; - case KW_resist_weaponmagic: thawer.get (op->resist[ATNR_WEAPONMAGIC]); break; - case KW_resist_ghosthit: thawer.get (op->resist[ATNR_GHOSTHIT]); break; - case KW_resist_poison: thawer.get (op->resist[ATNR_POISON]); break; - case KW_resist_slow: thawer.get (op->resist[ATNR_SLOW]); break; - case KW_resist_paralyze: thawer.get (op->resist[ATNR_PARALYZE]); break; - case KW_resist_turn_undead: thawer.get (op->resist[ATNR_TURN_UNDEAD]); break; - case KW_resist_fear: thawer.get (op->resist[ATNR_FEAR]); break; - case KW_resist_cancellation: thawer.get (op->resist[ATNR_CANCELLATION]); break; - case KW_resist_deplete: thawer.get (op->resist[ATNR_DEPLETE]); break; - case KW_resist_death: thawer.get (op->resist[ATNR_DEATH]); break; - case KW_resist_chaos: thawer.get (op->resist[ATNR_CHAOS]); break; - case KW_resist_counterspell: thawer.get (op->resist[ATNR_COUNTERSPELL]); break; - case KW_resist_godpower: thawer.get (op->resist[ATNR_GODPOWER]); break; - case KW_resist_holyword: thawer.get (op->resist[ATNR_HOLYWORD]); break; - case KW_resist_blind: thawer.get (op->resist[ATNR_BLIND]); break; - case KW_resist_internal: thawer.get (op->resist[ATNR_INTERNAL]); break; - case KW_resist_life_stealing: thawer.get (op->resist[ATNR_LIFE_STEALING]); break; - case KW_resist_disease: thawer.get (op->resist[ATNR_DISEASE]); break; - - case KW_luck: thawer.get (op->stats.luck); break; - case KW_run_away: thawer.get (op->run_away); break; - case KW_pick_up: thawer.get (op->pick_up); break; - case KW_item_power: thawer.get (op->item_power); break; - case KW_gen_sp_armour: thawer.get (op->gen_sp_armour); break; - case KW_anim_speed: thawer.get (op->anim_speed); break; - case KW_container: thawer.get (op->weight_limit); break; - case KW_will_apply: thawer.get (op->will_apply); break; - case KW_attack_movement: thawer.get (op->attack_movement); break; - case KW_move_state: thawer.get (op->move_status); break; - case KW_expmul: thawer.get (op->expmul); break; - case KW_glow_radius: thawer.get (op->glow_radius); break; - case KW_weapontype: thawer.get (op->weapontype); break; - case KW_tooltype: thawer.get (op->tooltype); break; - case KW_casting_time: thawer.get (op->casting_time); break; - case KW_elevation: thawer.get (op->elevation); break; - case KW_smoothlevel: thawer.get (op->smoothlevel); break; - case KW_client_type: thawer.get (op->client_type); break; - case KW_duration: thawer.get (op->duration); break; - case KW_range: thawer.get (op->range); break; - case KW_range_modifier: thawer.get (op->range_modifier); break; - case KW_dam_modifier: thawer.get (op->dam_modifier); break; - case KW_duration_modifier: thawer.get (op->duration_modifier); break; + if (f.get_sint32 ()) + if (type != PLAYER) + add_friendly_object (this); + + break; + + case KW_monster: GET_FLAG (this, FLAG_MONSTER); break; + case KW_neutral: GET_FLAG (this, FLAG_NEUTRAL); break; + case KW_no_attack: GET_FLAG (this, FLAG_NO_ATTACK); break; + case KW_no_damage: GET_FLAG (this, FLAG_NO_DAMAGE); break; + case KW_obj_original: GET_FLAG (this, FLAG_OBJ_ORIGINAL); break; + case KW_generator: GET_FLAG (this, FLAG_GENERATOR); break; + case KW_use_content_on_gen: GET_FLAG (this, FLAG_CONTENT_ON_GEN); break; + case KW_is_thrown: GET_FLAG (this, FLAG_IS_THROWN); break; + case KW_auto_apply: GET_FLAG (this, FLAG_AUTO_APPLY); break; + case KW_see_invisible: GET_FLAG (this, FLAG_SEE_INVISIBLE); break; + case KW_can_roll: GET_FLAG (this, FLAG_CAN_ROLL); break; + case KW_overlay_floor: GET_FLAG (this, FLAG_OVERLAY_FLOOR); break; + case KW_is_turnable: GET_FLAG (this, FLAG_IS_TURNABLE); break; + case KW_is_used_up: GET_FLAG (this, FLAG_IS_USED_UP); break; + case KW_alive: GET_FLAG (this, FLAG_ALIVE); break; + case KW_applied: GET_FLAG (this, FLAG_APPLIED); break; + case KW_unpaid: GET_FLAG (this, FLAG_UNPAID); break; + case KW_is_animated: GET_FLAG (this, FLAG_ANIMATE); break; + case KW_no_pick: GET_FLAG (this, FLAG_NO_PICK); break; + case KW_reflecting: GET_FLAG (this, FLAG_REFLECTING); break; + case KW_changing: GET_FLAG (this, FLAG_CHANGING); break; + case KW_splitting: GET_FLAG (this, FLAG_SPLITTING); break; + case KW_hitback: GET_FLAG (this, FLAG_HITBACK); break; + case KW_startequip: GET_FLAG (this, FLAG_STARTEQUIP); break; + case KW_blocksview: GET_FLAG (this, FLAG_BLOCKSVIEW); break; + case KW_undead: GET_FLAG (this, FLAG_UNDEAD); break; + case KW_scared: GET_FLAG (this, FLAG_SCARED); break; + case KW_unaggressive: GET_FLAG (this, FLAG_UNAGGRESSIVE); break; + case KW_reflect_missile: GET_FLAG (this, FLAG_REFL_MISSILE); break; + case KW_reflect_spell: GET_FLAG (this, FLAG_REFL_SPELL); break; + case KW_no_magic: GET_FLAG (this, FLAG_NO_MAGIC); break; + case KW_no_drop: GET_FLAG (this, FLAG_NO_DROP); break; + case KW_random_movement: GET_FLAG (this, FLAG_RANDOM_MOVE); break; + case KW_was_wiz: GET_FLAG (this, FLAG_WAS_WIZ); break; + case KW_no_fix_player: GET_FLAG (this, FLAG_NO_FIX_PLAYER); break; + case KW_is_lightable: GET_FLAG (this, FLAG_IS_LIGHTABLE); break; + case KW_tear_down: GET_FLAG (this, FLAG_TEAR_DOWN); break; + case KW_can_use_shield: GET_FLAG (this, FLAG_USE_SHIELD); break; + case KW_can_cast_spell: GET_FLAG (this, FLAG_CAST_SPELL); break; + case KW_can_use_scroll: GET_FLAG (this, FLAG_USE_SCROLL); break; + case KW_can_use_range: GET_FLAG (this, FLAG_USE_RANGE); break; + case KW_can_use_bow: GET_FLAG (this, FLAG_USE_BOW); break; + case KW_can_use_armour: GET_FLAG (this, FLAG_USE_ARMOUR); break; + case KW_can_use_weapon: GET_FLAG (this, FLAG_USE_WEAPON); break; + case KW_can_use_ring: GET_FLAG (this, FLAG_USE_RING); break; + case KW_has_ready_bow: GET_FLAG (this, FLAG_READY_BOW); break; + case KW_has_ready_range: GET_FLAG (this, FLAG_READY_RANGE); break; + case KW_xrays: GET_FLAG (this, FLAG_XRAYS); break; + case KW_is_floor: GET_FLAG (this, FLAG_IS_FLOOR); break; + case KW_lifesave: GET_FLAG (this, FLAG_LIFESAVE); break; + case KW_no_strength: GET_FLAG (this, FLAG_NO_STRENGTH); break; + case KW_sleep: GET_FLAG (this, FLAG_SLEEP); break; + case KW_stand_still: GET_FLAG (this, FLAG_STAND_STILL); break; + case KW_random_move: GET_FLAG (this, FLAG_RANDOM_MOVE); break; + case KW_only_attack: GET_FLAG (this, FLAG_ONLY_ATTACK); break; + case KW_activate_on_push: GET_FLAG (this, FLAG_ACTIVATE_ON_PUSH); break; + case KW_activate_on_release: GET_FLAG (this, FLAG_ACTIVATE_ON_RELEASE); break; + case KW_confused: GET_FLAG (this, FLAG_CONFUSED); break; + case KW_stealth: GET_FLAG (this, FLAG_STEALTH); break; + case KW_cursed: GET_FLAG (this, FLAG_CURSED); break; + case KW_damned: GET_FLAG (this, FLAG_DAMNED); break; + case KW_see_anywhere: GET_FLAG (this, FLAG_SEE_ANYWHERE); break; + case KW_known_magical: GET_FLAG (this, FLAG_KNOWN_MAGICAL); break; + case KW_known_cursed: GET_FLAG (this, FLAG_KNOWN_CURSED); break; + case KW_can_use_skill: GET_FLAG (this, FLAG_CAN_USE_SKILL); break; + case KW_been_applied: GET_FLAG (this, FLAG_BEEN_APPLIED); break; + case KW_has_ready_scroll: GET_FLAG (this, FLAG_READY_SCROLL); break; + case KW_can_use_rod: GET_FLAG (this, FLAG_USE_ROD); break; + case KW_can_use_horn: GET_FLAG (this, FLAG_USE_HORN); break; + case KW_unique: GET_FLAG (this, FLAG_UNIQUE); break; + case KW_make_invisible: GET_FLAG (this, FLAG_MAKE_INVIS); break; + case KW_inv_locked: GET_FLAG (this, FLAG_INV_LOCKED); break; + case KW_is_wooded: GET_FLAG (this, FLAG_IS_WOODED); break; + case KW_is_hilly: GET_FLAG (this, FLAG_IS_HILLY); break; + case KW_is_water: GET_FLAG (this, FLAG_IS_WATER); break; + case KW_has_ready_skill: GET_FLAG (this, FLAG_READY_SKILL); break; + case KW_has_ready_weapon: GET_FLAG (this, FLAG_READY_WEAPON); break; + case KW_no_skill_ident: GET_FLAG (this, FLAG_NO_SKILL_IDENT); break; + case KW_is_blind: GET_FLAG (this, FLAG_BLIND); break; + case KW_can_see_in_dark: GET_FLAG (this, FLAG_SEE_IN_DARK); break; + case KW_is_cauldron: GET_FLAG (this, FLAG_IS_CAULDRON); break; + case KW_no_steal: GET_FLAG (this, FLAG_NO_STEAL); break; + case KW_one_hit: GET_FLAG (this, FLAG_ONE_HIT); break; + case KW_berserk: GET_FLAG (this, FLAG_BERSERK); break; + case KW_is_buildable: GET_FLAG (this, FLAG_IS_BUILDABLE); break; + case KW_destroy_on_death: GET_FLAG (this, FLAG_DESTROY_ON_DEATH); break; + + case KW_armour: f.get (resist[ATNR_PHYSICAL]); break; + case KW_resist_physical: f.get (resist[ATNR_PHYSICAL]); break; + case KW_resist_magic: f.get (resist[ATNR_MAGIC]); break; + case KW_resist_fire: f.get (resist[ATNR_FIRE]); break; + case KW_resist_electricity: f.get (resist[ATNR_ELECTRICITY]); break; + case KW_resist_cold: f.get (resist[ATNR_COLD]); break; + case KW_resist_confusion: f.get (resist[ATNR_CONFUSION]); break; + case KW_resist_acid: f.get (resist[ATNR_ACID]); break; + case KW_resist_drain: f.get (resist[ATNR_DRAIN]); break; + case KW_resist_weaponmagic: f.get (resist[ATNR_WEAPONMAGIC]); break; + case KW_resist_ghosthit: f.get (resist[ATNR_GHOSTHIT]); break; + case KW_resist_poison: f.get (resist[ATNR_POISON]); break; + case KW_resist_slow: f.get (resist[ATNR_SLOW]); break; + case KW_resist_paralyze: f.get (resist[ATNR_PARALYZE]); break; + case KW_resist_turn_undead: f.get (resist[ATNR_TURN_UNDEAD]); break; + case KW_resist_fear: f.get (resist[ATNR_FEAR]); break; + case KW_resist_cancellation: f.get (resist[ATNR_CANCELLATION]); break; + case KW_resist_deplete: f.get (resist[ATNR_DEPLETE]); break; + case KW_resist_death: f.get (resist[ATNR_DEATH]); break; + case KW_resist_chaos: f.get (resist[ATNR_CHAOS]); break; + case KW_resist_counterspell: f.get (resist[ATNR_COUNTERSPELL]); break; + case KW_resist_godpower: f.get (resist[ATNR_GODPOWER]); break; + case KW_resist_holyword: f.get (resist[ATNR_HOLYWORD]); break; + case KW_resist_blind: f.get (resist[ATNR_BLIND]); break; + case KW_resist_internal: f.get (resist[ATNR_INTERNAL]); break; + case KW_resist_life_stealing: f.get (resist[ATNR_LIFE_STEALING]); break; + case KW_resist_disease: f.get (resist[ATNR_DISEASE]); break; + + case KW_luck: f.get (stats.luck); break; + case KW_run_away: f.get (run_away); break; + case KW_pick_up: f.get (pick_up); break; + case KW_item_power: f.get (item_power); break; + case KW_gen_sp_armour: f.get (gen_sp_armour); break; + case KW_anim_speed: f.get (anim_speed); break; + case KW_container: f.get (weight_limit); break; + case KW_will_apply: f.get (will_apply); break; + case KW_attack_movement: f.get (attack_movement); break; + case KW_move_state: f.get (move_status); break; + case KW_expmul: f.get (expmul); break; + case KW_glow_radius: f.get (glow_radius); break; + case KW_weapontype: f.get (weapontype); break; + case KW_tooltype: f.get (tooltype); break; + case KW_casting_time: f.get (casting_time); break; + case KW_elevation: f.get (elevation); break; + case KW_smoothlevel: f.get (smoothlevel); smoothlevel = clamp (smoothlevel, 0, 255); break; + case KW_client_type: f.get (client_type); break; + case KW_duration: f.get (duration); break; + case KW_range: f.get (range); break; + case KW_range_modifier: f.get (range_modifier); break; + case KW_dam_modifier: f.get (dam_modifier); break; + case KW_duration_modifier: f.get (duration_modifier); break; //TODO: mechanism to ensure that KW_xxx is consecutive needed from include/preprocess //TODO: parse from other include files - case KW_body_range: thawer.get (op->body_info[0]); break; - case KW_body_arm: thawer.get (op->body_info[1]); break; - case KW_body_torso: thawer.get (op->body_info[2]); break; - case KW_body_head: thawer.get (op->body_info[3]); break; - case KW_body_neck: thawer.get (op->body_info[4]); break; - case KW_body_skill: thawer.get (op->body_info[5]); break; - case KW_body_finger: thawer.get (op->body_info[6]); break; - case KW_body_shoulder: thawer.get (op->body_info[7]); break; - case KW_body_foot: thawer.get (op->body_info[8]); break; - case KW_body_hand: thawer.get (op->body_info[9]); break; - case KW_body_wrist: thawer.get (op->body_info[10]); break; - case KW_body_waist: thawer.get (op->body_info[11]); break; + case KW_body_range: f.get (body_info[0]); break; + case KW_body_arm: f.get (body_info[1]); break; + case KW_body_torso: f.get (body_info[2]); break; + case KW_body_head: f.get (body_info[3]); break; + case KW_body_neck: f.get (body_info[4]); break; + case KW_body_skill: f.get (body_info[5]); break; + case KW_body_finger: f.get (body_info[6]); break; + case KW_body_shoulder: f.get (body_info[7]); break; + case KW_body_foot: f.get (body_info[8]); break; + case KW_body_hand: f.get (body_info[9]); break; + case KW_body_wrist: f.get (body_info[10]); break; + case KW_body_waist: f.get (body_info[11]); break; case KW_can_apply: break; case KW_connected: - add_button_link (op, op->map, thawer.get_sint32 ()); + add_button_link (this, map, f.get_sint32 ()); break; case KW_randomitems: - op->randomitems = find_treasurelist (thawer.get_str ()); + if (f.get_str ()) + { + randomitems = + loading_arch + ? treasurelist::get (f.get_str ()) + : treasurelist::find (f.get_str ()); + + if (!randomitems) + LOG (llevError, "%s uses unknown randomitems '%s'.\n", debug_desc (), f.get_str ()); + } break; case KW_msg: - thawer.get_ml (KW_endmsg, op->msg); + f.get_ml (KW_endmsg, msg); //TODO: allow longer messages - if (strlen (op->msg) >= HUGE_BUF) + if (strlen (msg) >= HUGE_BUF) { - LOG (llevDebug, "\tError message length >= %d: %d\n>%.80s<\n", HUGE_BUF, strlen (op->msg), &op->msg); - op->msg = "ERROR, please report: string too long, winged.\n"; + LOG (llevDebug, "\tError message length >= %d: %d\n>%.80s<\n", HUGE_BUF, strlen (msg), &msg); + msg = "ERROR, please report: string too long, winged.\n"; } break; case KW_lore: - thawer.get_ml (KW_endlore, op->lore); + f.get_ml (KW_endlore, lore); //TODO: allow longer messages /* Just print a warning so we can be reasonably safe * about not overflowing the buffer. */ - if (strlen (op->lore) > (HUGE_BUF / 2)) + if (strlen (lore) > (HUGE_BUF / 2)) LOG (llevDebug, "\tWarning lore length > %d (max allowed=%d): %d\n>%.80s<\n", - HUGE_BUF / 2, HUGE_BUF, strlen (op->lore), &op->lore); + HUGE_BUF / 2, HUGE_BUF, strlen (lore), &lore); break; case KW_editable: case KW_editor_folder: break; - case KW_More: // uppercase alias - case KW_more: - /* We need to record that this is a multipart object, - * so the calling function can glue things back together - */ - ismore = 1; - break; - case KW_end: - check_loaded_object (op); - - if (!arch_init) - op->instantiate (); + check_loaded_object (this); - return ismore ? LL_MORE : LL_NORMAL; + if (!loading_arch) + instantiate (); - case KW_EOF: - return LL_EOF; + f.next (); + return true; case KW_ERROR: - set_ob_key_value (op, thawer.last_keyword, thawer.last_value, true); - //fprintf (stderr, "addkv(%s,%s)\n", thawer.last_keyword, thawer.last_value);//D + set_ob_key_value (this, f.kw_str, f.value, true); + //fprintf (stderr, "addkv(%s,%s)\n", f.kw_str, f.value);//D break; default: - LOG (llevError, "UNSUPPORTED KEYWORD IN MAP: \"%s\", bug in normaliser. skipping.\n", keyword_str[kw]); + if (!f.parse_error ("object", name)) + return false; break; } + + f.next (); } } -/* - * Loads an object from the given file-pointer. - * Variables will be read and parsed and patched into the object - * until the string "end" is reached, or the end of the file. - * - * bufstat is used to determine various file attributes: - * LO_REPATE (0): We are reading from the same buffer as the last call. - * LO_LINEMODE (1): file that is being read from is multi purpose (ie, other functions - * will also be reading from this (treasure file, artifacts.) - * LO_NEWFILE (2): This is the first read from a particular file, so the buffers should - * be reset. - * LO_NOREAD (3): Reset the buffers, but don't read from it. (op can be null) - * - */ -int -load_object (object_thawer &fp, object *op, int map_flags) +object * +object::read (object_thawer &f, maptile *map) { - coroapi::cede_every (1000); // cede once in a while + assert (f.kw == KW_arch); + + archetype *arch = archetype::find (f.get_str ()); - return parse_object (op, fp, map_flags); + if (!arch) + { + LOG (llevError, "object refering to nonexistant archetype '%s'.\n", f.get_str ()); + arch = archetype::find ("earthwall"); + } + + assert (arch); //D maybe use exception handling of sorts? + + object *op = object::create (); + + op->map = map; + arch->clone.copy_to (op); + // copy_to activates, this should be fixed properly + op->deactivate (); + + if (!op->parse_kv (f)) + { + op->destroy (true); + return 0; + } + + return op; } /* This takes a buffer, scans it for variables, and sets those variables @@ -1071,19 +1017,10 @@ int set_variable (object *op, char *buf) { - object_thawer thawer (buf, (AV *)0); - - return parse_object (op, thawer, 0); -} - -/* - * Initialises the array of variable-names. Needed before any - * objects can be loaded. Called by init_library(). - */ + object_thawer f (buf, (AV *)0); -void -init_vars () -{ + f.kw = KW_arch; // special hack so that parse_kv skips + return op->parse_kv (f); } /* This returns a string of the integer movement type */ @@ -1138,8 +1075,8 @@ #endif // compare *op against *tmp and output differences -void -put (object_freezer &f, object *op, object *tmp) +static void +write_diff (object_freezer &f, object *op, object *tmp) { static const keyword resist_save[NROFATTACKS] = { # define def(uc, lc, name, plus, change) KW_resist_ ## lc, @@ -1320,7 +1257,8 @@ f.put (KW_lore, KW_endlore, op->lore); CMP_OUT (other_arch); - CMP_OUT (face); + + if (op->face != tmp->face) f.put (KW_face, op->face ? &faces [op->face] : 0); if (op->animation_id != tmp->animation_id) if (op->animation_id) @@ -1378,7 +1316,7 @@ CMP_OUT (path_attuned); CMP_OUT (path_repelled); CMP_OUT (path_denied); - CMP_OUT (material); + CMP_OUT2 (material, materials);//TODO: nuke CMP_OUT (materialname); CMP_OUT (value); CMP_OUT (carrying); @@ -1438,30 +1376,61 @@ * If bit 0 of flag is set, unpaid objects will be saved. As of now, * the only place this is not set is when saving the player. */ -void -save_object (object_freezer &fp, object *op, int flag) +bool +object::write (object_freezer &f) { /* Even if the object does have an owner, it would seem that we should * still save it. */ - if (op->owner) - return; + if (owner) + return true; - /* If it is unpaid and we don't want to save those, just return. */ - if (!(flag & 1) && op->flag [FLAG_UNPAID]) - return; + archetype *at = arch ? (archetype *)arch : empty_archetype; - archetype *at = op->arch ? (archetype *)op->arch : empty_archetype; + f.put (KW_arch, at->name); + write_diff (f, this, &at->clone); - fp.put (KW_arch, at->name); - put (fp, op, &at->clone); + for (object *tmp = inv; tmp; tmp = tmp->below) + tmp->write (f); - /* Eneq(@csd.uu.se): Added this to allow containers being saved with contents */ + f.put (this); + f.put (KW_end); - for (object *tmp = op->inv; tmp; tmp = tmp->below) - save_object (fp, tmp, flag); + return true; +} + +///////////////////////////////////////////////////////////////////////////// + +// generic resource file load, +// currently supports: region, treasures +bool load_resource_file (const char *filename) +{ + object_thawer f (filename); - fp.put (op); - fp.put (KW_end); + f.next (); + + for (;;) + { + switch (f.kw) + { + case KW_region: + if (!region::read (f)) + return false; + break; + + case KW_treasure: + case KW_treasureone: + if (!treasurelist::read (f)) + return false; + break; + + case KW_EOF: + return true; + + default: + if (!f.parse_error ("resource file")) + return false; + } + } }