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

Comparing deliantra/server/common/loader.C (file contents):
Revision 1.73 by root, Sat Apr 28 17:51:57 2007 UTC vs.
Revision 1.100 by root, Tue Sep 4 05:43:20 2007 UTC

1/* 1/*
2 * CrossFire, A Multiplayer game for X-windows 2 * This file is part of Crossfire TRT, the Roguelike Realtime MORPG.
3 * 3 *
4 * Copyright (C) 2005, 2006, 2007 Marc Lehmann & Crossfire+ Development Team 4 * Copyright (©) 2005,2006,2007 Marc Alexander Lehmann / Robin Redeker / the Crossfire TRT team
5 * Copyright (C) 2002 Mark Wedel & Crossfire Development Team 5 * Copyright (©) 2002,2007 Mark Wedel & Crossfire Development Team
6 * Copyright (C) 1992 Frank Tore Johansen 6 * Copyright (©) 1992,2007 Frank Tore Johansen
7 * 7 *
8 * This program is free software; you can redistribute it and/or modify 8 * Crossfire TRT is free software: you can redistribute it and/or modify
9 * it under the terms of the GNU General Public License as published by 9 * it under the terms of the GNU General Public License as published by
10 * the Free Software Foundation; either version 2 of the License, or 10 * the Free Software Foundation, either version 3 of the License, or
11 * (at your option) any later version. 11 * (at your option) any later version.
12 * 12 *
13 * This program is distributed in the hope that it will be useful, 13 * This program is distributed in the hope that it will be useful,
14 * but WITHOUT ANY WARRANTY; without even the implied warranty of 14 * but WITHOUT ANY WARRANTY; without even the implied warranty of
15 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 15 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16 * GNU General Public License for more details. 16 * GNU General Public License for more details.
17 * 17 *
18 * You should have received a copy of the GNU General Public License 18 * You should have received a copy of the GNU General Public License
19 * along with this program; if not, write to the Free Software 19 * along with this program. If not, see <http://www.gnu.org/licenses/>.
20 * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
21 * 20 *
22 * The authors can be reached via e-mail at <crossfire@schmorp.de> 21 * The authors can be reached via e-mail to <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. */
… …
257/* This function checks the object after it has been loaded (when we 256/* This function checks the object after it has been loaded (when we
258 * get the 'end' in the input stream). This function can be used to 257 * get the 'end' in the input stream). This function can be used to
259 * deal with legacy objects where fields may have changed. It can also be used 258 * deal with legacy objects where fields may have changed. It can also be used
260 * to check for objects to make sure there are no common errors. 259 * to check for objects to make sure there are no common errors.
261 */ 260 */
262static void 261void
263check_loaded_object (object *op) 262object::post_load_check ()
264{ 263{
264 if (type >= NUM_TYPES)
265 {
266 LOG (llevError, "%s: type out of range, resetting to 0.\n", debug_desc ());
267 type = 0;
268 }
269
270 switch (type)
271 {
272 case BOW:
273 case WAND:
274 case ROD:
275 case HORN:
276 if (slot [body_range].info != -1)
277 {
278 LOG (llevError, "%s: body_range %d != -1\n", debug_desc (), slot [body_range].info);
279 slot [body_range].info = -1;
280 }
281 break;
282
283 case WEAPON:
284 if (slot [body_combat].info != -1)
285 {
286 LOG (llevError, "%s: body_combat %d != -1\n", debug_desc (), slot [body_combat].info);
287 slot [body_combat].info = -1;
288 }
289 break;
290
291 case SHIELD:
292 if (slot [body_shield].info != -1)
293 {
294 LOG (llevError, "%s: body_shield %d != -1\n", debug_desc (), slot [body_shield].info);
295 slot [body_shield].info = -1;
296 }
297 break;
298
299 case PLAYER:
300 if (slot [body_shield].info != 1)
301 {
302 LOG (llevError, "%s: body_shield %d != 1\n", debug_desc (), slot [body_shield].info);
303 slot [body_shield].info = 1;
304 }
305
306 if (slot [body_combat].info != 1)
307 {
308 LOG (llevError, "%s: body_combat %d != 1\n", debug_desc (), slot [body_combat].info);
309 slot [body_combat].info = 1;
310 }
311
312 if (slot [body_range].info != 1)
313 {
314 LOG (llevError, "%s: body_range %d != 1\n", debug_desc (), slot [body_range].info);
315 slot [body_range].info = 1;
316 }
317 break;
318 }
319
265 /* We do some specialised handling to handle legacy cases of name_pl. 320 /* We do some specialised handling to handle legacy cases of name_pl.
266 * If the object doesn't have a name_pl, we just use the object name - 321 * If the object doesn't have a name_pl, we just use the object name -
267 * this isn't perfect (things won't be properly pluralised), but works to 322 * this isn't perfect (things won't be properly pluralised), but works to
268 * that degree (5 heart is still quite understandable). But the case we 323 * that degree (5 heart is still quite understandable). But the case we
269 * also have to catch is if this object is not using the normal name for 324 * also have to catch is if this object is not using the normal name for
270 * the object. In that case, we also want to use the loaded name. 325 * the object. In that case, we also want to use the loaded name.
271 * Otherwise, what happens is that the the plural name will lose 326 * Otherwise, what happens is that the the plural name will lose
272 * information (appear as just 'hearts' and not 'goblins heart') 327 * information (appear as just 'hearts' and not 'goblins heart')
273 */ 328 */
274 if (op->arch && op->name != op->arch->clone.name && op->name_pl == op->arch->clone.name_pl) 329 if (arch && name != arch->object::name && name_pl == arch->object::name_pl)
275 op->name_pl = 0; 330 name_pl = 0;
276 331
277 if (!op->name_pl) 332 if (!name_pl)
278 op->name_pl = op->name; 333 name_pl = name;
279 334
280 /* objects now have a materialname. try to patch it in */ 335 /* objects now have a materialname. try to patch it in */
281 if (!(op->is_weapon () && op->level > 0)) 336 if (!(is_weapon () && level > 0))
282 { 337 set_materialname (this, map ? map->difficulty : 5, 0);
283 if (op->map != NULL)
284 set_materialname (op, op->map->difficulty, NULL);
285 else
286 set_materialname (op, 5, NULL);
287 }
288 338
289 /* only do these when program is first run - a bit 339 /* only do these when program is first run - a bit
290 * excessive to do this at every run - most of this is 340 * excessive to do this at every run - most of this is
291 * really just to catch any errors - program will still run, but 341 * really just to catch any errors - program will still run, but
292 * not in the ideal fashion. 342 * not in the ideal fashion.
293 */ 343 */
294 if ((op->type == WEAPON || op->type == BOW) && loading_arch) 344 if (loading_arch && (type == WEAPON || type == BOW || type == ROD || type == HORN || type == WAND))
295 { 345 {
296 if (!op->skill) 346 if (!skill)
297 LOG (llevError, "Weapon %s lacks a skill.\n", op->debug_desc ()); 347 LOG (llevError, "Weapon %s lacks a skill.\n", debug_desc ());
298 else if ((!strcmp (op->skill, "one handed weapons") && op->body_info[1] != -1) || 348 else if ((!strcmp (skill, "one handed weapons") && slot[body_arm].info != -1) ||
299 (!strcmp (op->skill, "two handed weapons") && op->body_info[1] != -2)) 349 (!strcmp (skill, "two handed weapons") && slot[body_arm].info != -2))
300 LOG (llevError, "weapon %s arm usage does not match skill: %d, %s\n", op->debug_desc (), op->body_info[1], &op->skill); 350 LOG (llevError, "weapon %s arm usage does not match skill: %d, %s\n", debug_desc (), slot[body_arm].info, &skill);
301 } 351 }
302 352
303 /* We changed last_heal to gen_sp_armour, which is what it 353 /* We changed last_heal to gen_sp_armour, which is what it
304 * really does for many objects. Need to catch any in maps 354 * really does for many objects. Need to catch any in maps
305 * that may have an old value. 355 * that may have an old value.
306 */ 356 */
307 if (op->type == WEAPON 357 if (type == WEAPON
308 || op->type == ARMOUR || op->type == HELMET 358 || type == ARMOUR || type == HELMET
309 || op->type == SHIELD || op->type == RING 359 || type == SHIELD || type == RING
310 || op->type == BOOTS || op->type == GLOVES 360 || type == BOOTS || type == GLOVES
311 || op->type == AMULET || op->type == GIRDLE 361 || type == AMULET || type == GIRDLE
312 || op->type == BRACERS || op->type == CLOAK) 362 || type == BRACERS || type == CLOAK)
313 { 363 {
314 if (op->last_heal) 364 if (last_heal)
315 { 365 {
316 LOG (llevDebug, "Object %s still has last_heal set, not gen_sp_armour\n", op->debug_desc ()); 366 LOG (llevDebug, "Object %s still has last_heal set, not gen_sp_armour\n", debug_desc ());
317 op->gen_sp_armour = op->last_heal; 367 gen_sp_armour = last_heal;
318 op->last_heal = 0; 368 last_heal = 0;
319 } 369 }
320 370
321 int ip = calc_item_power (op, 0); 371 int ip = calc_item_power (this, 0);
322 372
323 /* Legacy objects from before item power was in the game */ 373 /* Legacy objects from before item power was in the game */
324 if (!op->item_power && ip) 374 if (!item_power && ip)
325 { 375 {
326 if (ip > 3) 376 if (ip > 3)
327 LOG (llevDebug, "Object %s had no item power, using %d\n", op->debug_desc (), ip); 377 LOG (llevDebug, "Object %s had no item power, using %d\n", debug_desc (), ip);
328 378
329 op->item_power = ip; 379 item_power = ip;
330 } 380 }
331 381
332 /* Check for possibly bogus values. Has to meet both these criteria - 382 /* Check for possibly bogus values. Has to meet both these criteria -
333 * something that has item_power 1 is probably just fine if our calculated 383 * something that has item_power 1 is probably just fine if our calculated
334 * value is 1 or 2 - these values are small enough that hard to be precise. 384 * value is 1 or 2 - these values are small enough that hard to be precise.
335 * similarly, it item_power is 0, the first check will always pass, 385 * similarly, it item_power is 0, the first check will always pass,
336 * but not the second one. 386 * but not the second one.
337 */ 387 */
388#if 0 //TODO
338 if (ip > 2 * op->item_power && ip > (op->item_power + 3)) 389 if (ip > 2 * item_power && ip > (item_power + 3))
339 LOG (llevDebug, "Object %s seems to have too low item power? %d > %d\n", op->debug_desc (), ip, op->item_power); 390 LOG (llevDebug, "Object %s seems to have too low item power? %d > %d\n", debug_desc (), ip, item_power);
391#endif
340 } 392 }
341 393
342 /* Old spellcasting object - need to load in the appropiate object */ 394 /* Old spellcasting object - need to load in the appropiate object */
343 if ((op->type == ROD || op->type == WAND || op->type == SCROLL || op->type == HORN || op->type == FIREWALL || 395 if ((type == ROD || type == WAND || type == SCROLL || type == HORN || type == FIREWALL ||
344 /* POTIONS and ALTARS don't always cast spells, but if they do, update them */ 396 /* POTIONS and ALTARS don't always cast spells, but if they do, update them */
345 ((op->type == POTION || op->type == ALTAR) && op->stats.sp)) && !op->inv && !loading_arch) 397 ((type == POTION || type == ALTAR) && stats.sp)) && !inv && !loading_arch)
346 { 398 {
347 /* Fireall is bizarre in that spell type was stored in dam. Rest are 'normal' 399 /* Fireall is bizarre in that spell type was stored in dam. Rest are 'normal'
348 * in that spell was stored in sp. 400 * in that spell was stored in sp.
349 */ 401 */
350 object *tmp = get_archetype (spell_mapping[op->type == FIREWALL ? op->stats.dam : op->stats.sp]); 402 object *tmp = get_archetype (spell_mapping[type == FIREWALL ? stats.dam : stats.sp]);
351 insert_ob_in_ob (tmp, op); 403 insert_ob_in_ob (tmp, this);
352 op->randomitems = NULL; /* So another spell isn't created for this object */ 404 randomitems = NULL; /* So another spell isn't created for this object */
353 } 405 }
354 406
355 /* spellbooks & runes use slaying. But not to arch name, but to spell name */ 407 /* spellbooks & runes use slaying. But not to arch name, but to spell name */
356 if ((op->type == SPELLBOOK || op->type == RUNE) && op->slaying && !op->inv && !loading_arch) 408 if ((type == SPELLBOOK || type == RUNE) && slaying && !inv && !loading_arch)
357 { 409 {
358 object *tmp = get_archetype_by_object_name (op->slaying); 410 object *tmp = get_archetype_by_object_name (slaying);
359 insert_ob_in_ob (tmp, op); 411 insert_ob_in_ob (tmp, this);
360 op->randomitems = NULL; /* So another spell isn't created for this object */ 412 randomitems = NULL; /* So another spell isn't created for this object */
361 /* without this, value is all screwed up */ 413 /* without this, value is all screwed up */
362 op->value = op->arch->clone.value * op->inv->value; 414 value = arch->value * inv->value;
363 } 415 }
364 416
365 if (QUERY_FLAG (op, FLAG_MONSTER)) 417 if (QUERY_FLAG (this, FLAG_MONSTER))
366 { 418 {
367 if (op->stats.hp > op->stats.maxhp) 419 if (stats.hp > stats.maxhp)
368 { 420 {
369 LOG (llevDebug, "Monster %s has hp set higher than maxhp (%d>%d)\n", op->debug_desc (), op->stats.hp, op->stats.maxhp); 421 LOG (llevDebug, "Monster %s has hp set higher than maxhp (%d>%d)\n", debug_desc (), stats.hp, stats.maxhp);
370 op->stats.maxhp = op->stats.hp; 422 stats.maxhp = stats.hp;
371 } 423 }
372 424
373 /* The archs just need to be updated for this */ 425 /* The archs just need to be updated for this */
374 if (op->move_type == 0) 426 if (move_type == 0)
375 op->move_type = MOVE_WALK; 427 move_type = MOVE_WALK;
376 } 428 }
377 429
378 if ((QUERY_FLAG (op, FLAG_GENERATOR) && QUERY_FLAG (op, FLAG_CONTENT_ON_GEN)) || op->type == CREATOR || op->type == CONVERTER) 430 if ((QUERY_FLAG (this, FLAG_GENERATOR) && QUERY_FLAG (this, FLAG_CONTENT_ON_GEN)) || type == CREATOR || type == CONVERTER)
379 { 431 {
380 /* Object will duplicate it's content as part of the 432 /* Object will duplicate it's content as part of the
381 * generation process. To do this, we must flag inventory 433 * generation process. To do this, we must flag inventory
382 * so it remains unevaluated concerning the randomitems and 434 * so it remains unevaluated concerning the randomitems and
383 * the living (a demonlord shouldn't cast from inside generator!) 435 * the living (a demonlord shouldn't cast from inside generator!)
384 */ 436 */
385 flag_inv (op, FLAG_IS_A_TEMPLATE); 437 flag_inv (this, FLAG_IS_A_TEMPLATE);
386 } 438 }
387 439
388 /* Handle player movers. We use move_type for player movers 440 /* Handle player movers. We use move_type for player movers
389 * because they operate on their own time (move_on 441 * because they operate on their own time (move_on
390 * would potentially cause them to be triggered when someone steps 442 * would potentially cause them to be triggered when someone steps
391 * on them). If move_type is set, presume person knows what they 443 * on them). If move_type is set, presume person knows what they
392 * are doing, otherwise, set move_type based on maxhp value. 444 * are doing, otherwise, set move_type based on maxhp value.
393 */ 445 */
394 if (op->type == PLAYERMOVER) 446 if (type == PLAYERMOVER)
395 { 447 {
396 if (!op->move_type) 448 if (!move_type)
397 { 449 {
398 if (op->stats.maxhp) 450 if (stats.maxhp)
399 { 451 {
400 op->move_type = MOVE_ALL; 452 move_type = MOVE_ALL;
401 op->stats.maxhp = 0; 453 stats.maxhp = 0;
402 } 454 }
403 else 455 else
404 op->move_type = MOVE_WALK; 456 move_type = MOVE_WALK;
405 } 457 }
406 } 458 }
407
408} 459}
409 460
410static void 461static void
411set_move (MoveType &mt, const char *str) 462set_move (MoveType &mt, const char *str)
412{ 463{
… …
465 516
466next: ; 517next: ;
467 } 518 }
468} 519}
469 520
470#define GET_FLAG(op,flag) \ 521#define GET_FLAG(op,flg) op->flag [flg] = f.get_bool ()
471 if (f.get_sint32 ()) \
472 SET_FLAG (op, flag); \
473 else \
474 CLEAR_FLAG (op, flag) \
475 522
476bool 523bool
477object::parse_kv (object_thawer &f) 524object::parse_kv (object_thawer &f)
478{ 525{
479 object *op_inv = inv; 526 object *op_inv = inv;
… …
554 601
555 if (!other_arch) 602 if (!other_arch)
556 LOG (llevError, "%s uses unknown other_arch '%s'.\n", debug_desc (), f.get_str ()); 603 LOG (llevError, "%s uses unknown other_arch '%s'.\n", debug_desc (), f.get_str ());
557 break; 604 break;
558 605
606 case KW_owner:
607 f.delayed_deref (this, owner, f.get_str ());
608 break;
609
559 case KW_animation: 610 case KW_animation:
560 { 611 {
561 CLEAR_FLAG (this, FLAG_ANIMATE); 612 CLEAR_FLAG (this, FLAG_ANIMATE);
562 animation_id = 0; 613 animation_id = 0;
563 614
… …
588 f.get (move_slow_penalty); 639 f.get (move_slow_penalty);
589 break; 640 break;
590 641
591 case KW_face: 642 case KW_face:
592 face = face_find (f.get_str ()); 643 face = face_find (f.get_str ());
644 break;
645
646 case KW_sound:
647 sound = sound_find (f.get_str ());
648 if (!sound)
649 f.parse_warn ("sound not found");
650 break;
651
652 case KW_sound_destroy:
653 sound_destroy = sound_find (f.get_str ());
654 if (!sound_destroy)
655 f.parse_warn ("sound not found");
593 break; 656 break;
594 657
595 case KW_x: f.get (x); break; 658 case KW_x: f.get (x); break;
596 case KW_y: f.get (y); break; 659 case KW_y: f.get (y); break;
597 660
… …
724 if (QUERY_FLAG (this, FLAG_IDENTIFIED)) 787 if (QUERY_FLAG (this, FLAG_IDENTIFIED))
725 CLEAR_FLAG (this, FLAG_KNOWN_MAGICAL); 788 CLEAR_FLAG (this, FLAG_KNOWN_MAGICAL);
726 789
727 break; 790 break;
728 791
729 case KW_wiz:
730 GET_FLAG (this, FLAG_WIZ);
731 //TODO: move to check_object
732 if (QUERY_FLAG (this, FLAG_WIZ))
733 {
734 SET_FLAG (this, FLAG_WAS_WIZ);
735 SET_FLAG (this, FLAG_WIZPASS);
736 SET_FLAG (this, FLAG_WIZCAST);
737 }
738 else
739 {
740 CLEAR_FLAG (this, FLAG_WIZPASS);
741 CLEAR_FLAG (this, FLAG_WIZCAST);
742 }
743 break;
744
745 case KW_friendly: 792 case KW_friendly:
746 if (f.get_sint32 ()) 793 if (f.get_bool ())
747 if (type != PLAYER) 794 if (type != PLAYER)
748 add_friendly_object (this); 795 add_friendly_object (this);
749 796
750 break; 797 break;
751 798
… …
780 case KW_reflect_missile: GET_FLAG (this, FLAG_REFL_MISSILE); break; 827 case KW_reflect_missile: GET_FLAG (this, FLAG_REFL_MISSILE); break;
781 case KW_reflect_spell: GET_FLAG (this, FLAG_REFL_SPELL); break; 828 case KW_reflect_spell: GET_FLAG (this, FLAG_REFL_SPELL); break;
782 case KW_no_magic: GET_FLAG (this, FLAG_NO_MAGIC); break; 829 case KW_no_magic: GET_FLAG (this, FLAG_NO_MAGIC); break;
783 case KW_no_drop: GET_FLAG (this, FLAG_NO_DROP); break; 830 case KW_no_drop: GET_FLAG (this, FLAG_NO_DROP); break;
784 case KW_random_movement: GET_FLAG (this, FLAG_RANDOM_MOVE); break; 831 case KW_random_movement: GET_FLAG (this, FLAG_RANDOM_MOVE); break;
785 case KW_was_wiz: GET_FLAG (this, FLAG_WAS_WIZ); break;
786 case KW_no_fix_player: GET_FLAG (this, FLAG_NO_FIX_PLAYER); break; 832 case KW_no_fix_player: GET_FLAG (this, FLAG_NO_FIX_PLAYER); break;
787 case KW_is_lightable: GET_FLAG (this, FLAG_IS_LIGHTABLE); break; 833 case KW_is_lightable: GET_FLAG (this, FLAG_IS_LIGHTABLE); break;
788 case KW_tear_down: GET_FLAG (this, FLAG_TEAR_DOWN); break; 834 case KW_tear_down: GET_FLAG (this, FLAG_TEAR_DOWN); break;
789 case KW_can_use_shield: GET_FLAG (this, FLAG_USE_SHIELD); break; 835 case KW_can_use_shield: GET_FLAG (this, FLAG_USE_SHIELD); break;
790 case KW_can_cast_spell: GET_FLAG (this, FLAG_CAST_SPELL); break; 836 case KW_can_cast_spell: GET_FLAG (this, FLAG_CAST_SPELL); break;
… …
890 case KW_dam_modifier: f.get (dam_modifier); break; 936 case KW_dam_modifier: f.get (dam_modifier); break;
891 case KW_duration_modifier: f.get (duration_modifier); break; 937 case KW_duration_modifier: f.get (duration_modifier); break;
892 938
893 //TODO: mechanism to ensure that KW_xxx is consecutive needed from include/preprocess 939 //TODO: mechanism to ensure that KW_xxx is consecutive needed from include/preprocess
894 //TODO: parse from other include files 940 //TODO: parse from other include files
895 case KW_body_range: f.get (body_info[0]); break; 941 case KW_body_range: slot[body_range] .info = f.get_sint32 (); break;
896 case KW_body_arm: f.get (body_info[1]); break; 942 case KW_body_shield: slot[body_shield] .info = f.get_sint32 (); break;
897 case KW_body_torso: f.get (body_info[2]); break; 943 case KW_body_combat: slot[body_combat] .info = f.get_sint32 (); break;
898 case KW_body_head: f.get (body_info[3]); break; 944 case KW_body_arm: slot[body_arm] .info = f.get_sint32 (); break;
899 case KW_body_neck: f.get (body_info[4]); break; 945 case KW_body_torso: slot[body_torso] .info = f.get_sint32 (); break;
900 case KW_body_skill: f.get (body_info[5]); break; 946 case KW_body_head: slot[body_head] .info = f.get_sint32 (); break;
901 case KW_body_finger: f.get (body_info[6]); break; 947 case KW_body_neck: slot[body_neck] .info = f.get_sint32 (); break;
902 case KW_body_shoulder: f.get (body_info[7]); break; 948 case KW_body_skill: slot[body_skill] .info = f.get_sint32 (); break;
903 case KW_body_foot: f.get (body_info[8]); break; 949 case KW_body_finger: slot[body_finger] .info = f.get_sint32 (); break;
904 case KW_body_hand: f.get (body_info[9]); break; 950 case KW_body_shoulder: slot[body_shoulder].info = f.get_sint32 (); break;
905 case KW_body_wrist: f.get (body_info[10]); break; 951 case KW_body_foot: slot[body_foot] .info = f.get_sint32 (); break;
906 case KW_body_waist: f.get (body_info[11]); break; 952 case KW_body_hand: slot[body_hand] .info = f.get_sint32 (); break;
953 case KW_body_wrist: slot[body_wrist] .info = f.get_sint32 (); break;
954 case KW_body_waist: slot[body_waist] .info = f.get_sint32 (); break;
907 case KW_can_apply: 955 case KW_can_apply:
908 break; 956 break;
909 957
910 case KW_connected: 958 case KW_connected:
911 add_button_link (this, map, f.get_sint32 ()); 959 add_button_link (this, map, f.get_sint32 ());
… …
920 : treasurelist::find (f.get_str ()); 968 : treasurelist::find (f.get_str ());
921 969
922 if (!randomitems) 970 if (!randomitems)
923 LOG (llevError, "%s uses unknown randomitems '%s'.\n", debug_desc (), f.get_str ()); 971 LOG (llevError, "%s uses unknown randomitems '%s'.\n", debug_desc (), f.get_str ());
924 } 972 }
973 else
974 randomitems = 0;
925 break; 975 break;
926 976
927 case KW_msg: 977 case KW_msg:
928 f.get_ml (KW_endmsg, msg); 978 f.get_ml (KW_endmsg, msg);
929 //TODO: allow longer messages
930 if (strlen (msg) >= HUGE_BUF)
931 {
932 LOG (llevDebug, "\tError message length >= %d: %d\n>%.80s<\n", HUGE_BUF, strlen (msg), &msg);
933 msg = "ERROR, please report: string too long, winged.\n";
934 }
935 break; 979 break;
936 980
937 case KW_lore: 981 case KW_lore:
938 f.get_ml (KW_endlore, lore); 982 f.get_ml (KW_endlore, lore);
939 //TODO: allow longer messages
940 /* Just print a warning so we can be reasonably safe
941 * about not overflowing the buffer.
942 */
943 if (strlen (lore) > (HUGE_BUF / 2))
944 LOG (llevDebug, "\tWarning lore length > %d (max allowed=%d): %d\n>%.80s<\n",
945 HUGE_BUF / 2, HUGE_BUF, strlen (lore), &lore);
946 break; 983 break;
947 984
948 case KW_editable: 985 case KW_editable:
949 case KW_editor_folder: 986 case KW_editor_folder:
950 break; 987 break;
951 988
952 case KW_end: 989 case KW_end:
953 check_loaded_object (this);
954
955 if (!loading_arch) 990 if (!loading_arch)
956 instantiate (); 991 instantiate ();
957 992
958 f.next (); 993 f.next ();
959 return true; 994 return true;
… …
991 f.next (); 1026 f.next ();
992 1027
993 object *op = object::create (); 1028 object *op = object::create ();
994 1029
995 op->map = map; 1030 op->map = map;
996 arch->clone.copy_to (op); 1031 arch->copy_to (op);
997 // copy_to activates, this should be fixed properly 1032 // copy_to activates, this should be fixed properly
998 op->deactivate (); 1033 op->deactivate ();
999 1034
1000 if (!op->parse_kv (f)) 1035 if (!op->parse_kv (f))
1001 { 1036 {
1002 op->destroy (true); 1037 op->destroy (true);
1003 return 0; 1038 return 0;
1004 } 1039 }
1005 1040
1041 op->post_load_check ();
1006 return op; 1042 return op;
1007} 1043}
1008 1044
1009/* This takes a buffer, scans it for variables, and sets those variables 1045/* This takes a buffer, scans it for variables, and sets those variables
1010 * as appropriate in op. 1046 * as appropriate in op.
… …
1015int 1051int
1016set_variable (object *op, char *buf) 1052set_variable (object *op, char *buf)
1017{ 1053{
1018 object_thawer f (buf, (AV *)0); 1054 object_thawer f (buf, (AV *)0);
1019 1055
1020 f.next ();
1021 return op->parse_kv (f); 1056 return op->parse_kv (f);
1022} 1057}
1023 1058
1024/* This returns a string of the integer movement type */ 1059/* This returns a string of the integer movement type */
1025#if 0 1060#if 0
… …
1087 * the 15'th element of this array should match that name. 1122 * the 15'th element of this array should match that name.
1088 * If an entry is NULL, that is a flag not to loaded/saved. 1123 * If an entry is NULL, that is a flag not to loaded/saved.
1089 */ 1124 */
1090 static const keyword flag_names [NUM_FLAGS] = { 1125 static const keyword flag_names [NUM_FLAGS] = {
1091 KW_alive, 1126 KW_alive,
1092 KW_wiz,
1093 KW_NULL, 1127 KW_NULL,
1094 KW_NULL, 1128 KW_NULL,
1095 KW_was_wiz, 1129 KW_NULL,
1130 KW_NULL,
1096 KW_applied, 1131 KW_applied,
1097 KW_unpaid, 1132 KW_unpaid,
1098 KW_can_use_shield, 1133 KW_can_use_shield,
1099 KW_no_pick, 1134 KW_no_pick,
1100 KW_NULL, // walk_on 1135 KW_NULL, // walk_on
… …
1237 char uids[64]; 1272 char uids[64];
1238 snprintf (uids, sizeof (uids), "<1.%llx>", (unsigned long long)op->uuid.seq); 1273 snprintf (uids, sizeof (uids), "<1.%llx>", (unsigned long long)op->uuid.seq);
1239 f.put (KW_uuid, (const char *)uids); 1274 f.put (KW_uuid, (const char *)uids);
1240 } 1275 }
1241 1276
1242#define CMP_OUT(v) if (op->v != tmp->v) f.put (KW_ ## v, op->v) 1277#define CMP_OUT(v) if (expect_false (op->v != tmp->v)) f.put (KW_ ## v, op->v)
1243#define CMP_OUT2(k,v) if (op->v != tmp->v) f.put (KW_ ## k, op->v) 1278#define CMP_OUT2(k,v) if (expect_false (op->v != tmp->v)) f.put (KW_ ## k, op->v)
1279
1280 if (object *owner = op->owner)
1281 f.put (KW_owner, static_cast<const char *>(owner->ref ()));
1244 1282
1245 CMP_OUT (name); 1283 CMP_OUT (name);
1246 CMP_OUT (name_pl); 1284 CMP_OUT (name_pl);
1247 CMP_OUT (custom_name); 1285 CMP_OUT (custom_name);
1248 CMP_OUT (title); 1286 CMP_OUT (title);
1249 CMP_OUT (race); 1287 CMP_OUT (race);
1250 CMP_OUT (slaying); 1288 CMP_OUT (slaying);
1251
1252 if (op->msg != tmp->msg)
1253 f.put (KW_msg, KW_endmsg, op->msg);
1254 if (op->lore != tmp->lore)
1255 f.put (KW_lore, KW_endlore, op->lore);
1256
1257 CMP_OUT (other_arch); 1289 CMP_OUT (other_arch);
1258 1290
1291 if (op->msg != tmp->msg ) f.put (KW_msg , KW_endmsg , op->msg );
1292 if (op->lore != tmp->lore) f.put (KW_lore, KW_endlore, op->lore);
1293
1259 if (op->face != tmp->face) f.put (KW_face, op->face ? &faces [op->face] : 0); 1294 if (op->face != tmp->face ) f.put (KW_face , op->face ? &faces [op->face ] : 0);
1295 if (op->sound != tmp->sound ) f.put (KW_sound , op->sound ? &faces [op->sound ] : 0);
1296 if (op->sound_destroy != tmp->sound_destroy) f.put (KW_sound_destroy, op->sound_destroy ? &faces [op->sound_destroy] : 0);
1260 1297
1261 if (op->animation_id != tmp->animation_id) 1298 if (op->animation_id != tmp->animation_id)
1262 if (op->animation_id) 1299 if (op->animation_id)
1263 { 1300 {
1264 f.put (KW_animation, animations[GET_ANIM_ID (op)].name); 1301 f.put (KW_animation, animations[GET_ANIM_ID (op)].name);
1265 1302
1266 if (!QUERY_FLAG (op, FLAG_ANIMATE)) 1303 if (!QUERY_FLAG (op, FLAG_ANIMATE))
1267 f.put (KW_is_animated, (sint32) 0); 1304 f.put (KW_is_animated, (sint32)0);
1268 } 1305 }
1269 else 1306 else
1270 f.put (KW_animation, (const char *) 0); 1307 f.put (KW_animation, (const char *)0);
1271 1308
1272 CMP_OUT2 (str, stats.Str); 1309 CMP_OUT2 (str, stats.Str);
1273 CMP_OUT2 (dex, stats.Dex); 1310 CMP_OUT2 (dex, stats.Dex);
1274 CMP_OUT2 (con, stats.Con); 1311 CMP_OUT2 (con, stats.Con);
1275 CMP_OUT2 (wis, stats.Wis); 1312 CMP_OUT2 (wis, stats.Wis);
… …
1306 CMP_OUT (type); 1343 CMP_OUT (type);
1307 CMP_OUT (subtype); 1344 CMP_OUT (subtype);
1308 CMP_OUT (attacktype); 1345 CMP_OUT (attacktype);
1309 1346
1310 for (i = 0; i < NROFATTACKS; i++) 1347 for (i = 0; i < NROFATTACKS; i++)
1311 if (op->resist[i] != tmp->resist[i]) 1348 if (expect_false (op->resist[i] != tmp->resist[i]))
1312 f.put (resist_save[i], op->resist[i]); 1349 f.put (resist_save[i], op->resist[i]);
1313 1350
1314 CMP_OUT (path_attuned); 1351 CMP_OUT (path_attuned);
1315 CMP_OUT (path_repelled); 1352 CMP_OUT (path_repelled);
1316 CMP_OUT (path_denied); 1353 CMP_OUT (path_denied);
… …
1358 CMP_OUT (move_slow); 1395 CMP_OUT (move_slow);
1359 CMP_OUT (move_slow_penalty); 1396 CMP_OUT (move_slow_penalty);
1360 1397
1361 if (op->flag != tmp->flag) 1398 if (op->flag != tmp->flag)
1362 for (i = 0; i <= NUM_FLAGS; i++) 1399 for (i = 0; i <= NUM_FLAGS; i++)
1363 if (flag_names [i] && op->flag [i] != tmp->flag [i]) 1400 if (expect_false (flag_names [i] && op->flag [i] != tmp->flag [i]))
1364 f.put (flag_names [i], op->flag [i] ? "1" : "0"); 1401 f.put (flag_names [i], op->flag [i] ? "1" : "0");
1365 1402
1366 /* Save body locations */ 1403 // save body locations
1367 for (i = 0; i < NUM_BODY_LOCATIONS; i++) 1404 for (i = 0; i < NUM_BODY_LOCATIONS; i++)
1368 if (op->body_info[i] != tmp->body_info[i]) 1405 if (expect_false (op->slot[i].info != tmp->slot[i].info))
1369 f.put (body_locations[i].save_name, op->body_info[i]); 1406 f.put (body_locations[i].save_name, op->slot[i].info);
1370} 1407}
1371 1408
1372/* 1409/*
1373 * Dumps all variables in an object to a file. 1410 * Dumps all variables in an object to a file.
1374 * If bit 0 of flag is set, unpaid objects will be saved. As of now, 1411 * If bit 0 of flag is set, unpaid objects will be saved. As of now,
1375 * the only place this is not set is when saving the player. 1412 * the only place this is not set is when saving the player.
1376 */ 1413 */
1377bool 1414bool
1378object::write (object_freezer &f) 1415object::write (object_freezer &f)
1379{ 1416{
1380 /* Even if the object does have an owner, it would seem that we should
1381 * still save it.
1382 */
1383 if (owner)
1384 return true;
1385
1386 archetype *at = arch ? (archetype *)arch : empty_archetype; 1417 archetype *at = arch ? (archetype *)arch : empty_archetype;
1387 1418
1388 f.put (KW_arch, at->name); 1419 f.put (KW_arch, at->archname);
1389 write_diff (f, this, &at->clone); 1420 write_diff (f, this, at);
1390 1421
1391 for (object *tmp = inv; tmp; tmp = tmp->below) 1422 for (object *tmp = inv; tmp; tmp = tmp->below)
1392 tmp->write (f); 1423 tmp->write (f);
1393 1424
1394 f.put (this); 1425 f.put (this);
… …
1406 object_thawer f (filename); 1437 object_thawer f (filename);
1407 1438
1408 bool success = false; 1439 bool success = false;
1409 bool seen_arch = false; 1440 bool seen_arch = false;
1410 1441
1411 f.next ();
1412
1413 for (;;) 1442 for (;;)
1414 { 1443 {
1415 switch (f.kw) 1444 switch (f.kw)
1416 { 1445 {
1417 case KW_region: 1446 case KW_region:
… …
1436 goto finish; 1465 goto finish;
1437 1466
1438 default: 1467 default:
1439 if (!f.parse_error ("resource file")) 1468 if (!f.parse_error ("resource file"))
1440 goto finish; 1469 goto finish;
1470
1471 f.next ();
1472 break;
1441 } 1473 }
1442 } 1474 }
1443 1475
1444finish: 1476finish:
1445 if (seen_arch) 1477 if (seen_arch)

Diff Legend

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