[ViewVC] Diff of: cvs/deliantra/server/common/loader.C

Comparing deliantra/server/common/loader.C (file contents):
Revision 1.162 by root, Fri May 21 00:31:13 2010 UTC vs.
Revision 1.173 by root, Fri Jan 27 22:00:39 2012 UTC

 /*
  * This file is part of Deliantra, the Roguelike Realtime MMORPG.
  *
- * Copyright (©) 2005,2006,2007,2008,2009,2010 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
+ * Copyright (©) 2005,2006,2007,2008,2009,2010,2011,2012 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
  * Copyright (©) 2002 Mark Wedel & Crossfire Development Team
  * Copyright (©) 1992 Frank Tore Johansen
  *
  * Deliantra is free software: you can redistribute it and/or modify it under
  * the terms of the Affero GNU General Public License as published by the
       // TODO: fix firewall object on map
       /* Firewall is bizarre in that spell type was stored in dam. Rest are 'normal'
        * in that spell was stored in sp.
        */
       //LOG (llevError, "old spellcasting object found: %s", debug_desc ());
-      object *tmp = get_archetype (spell_mapping[type == FIREWALL ? stats.dam : stats.sp]);
+      object *tmp = archetype::get (spell_mapping[type == FIREWALL ? stats.dam : stats.sp]);
       insert_ob_in_ob (tmp, this);
       randomitems = 0;
     }
   /* spellbooks & runes use slaying.  But not to arch name, but to spell name */
 bool
 object::parse_kv (object_thawer &f)
 {
   object *op_inv = inv;
-  key_value *last_kv = key_values;
   for (;;)
     {
       switch (f.kw)
         {
           case KW_move_type:            set_move (move_type , f); break;
           case KW_move_on:              set_move (move_on   , f); break;
           case KW_move_off:             set_move (move_off  , f); break;
           case KW_move_slow:            set_move (move_slow , f); break;
-            /* These are all legacy - any new objects should use the move_ .. values */
-          case KW_no_pass:
-            {
-              if (f.get_sint32 ())
-                move_block = MOVE_ALL;
-              else
-                move_block = 0;
-            }
-            break;
-            /* These are all legacy - any new objects should use the move_ .. values */
-          case KW_walk_on:
-            {
-              if (f.get_sint32 ())
-                move_on |= MOVE_WALK;
-              else
-                move_on &= ~MOVE_WALK;
-            }
-            break;
-            /* These are all legacy - any new objects should use the move_ .. values */
-          case KW_walk_off:
-            {
-              if (f.get_sint32 ())
-                move_off |= MOVE_WALK;
-              else
-                move_off &= ~MOVE_WALK;
-            }
-            break;
-            /* These are all legacy - any new objects should use the move_ .. values */
-          case KW_fly_on:
-            {
-              if (f.get_sint32 ())
-                move_on |= MOVE_FLY_LOW;
-              else
-                move_on &= ~MOVE_FLY_LOW;
-            }
-            break;
-            /* These are all legacy - any new objects should use the move_ .. values */
-          case KW_fly_off:
-            {
-              if (f.get_sint32 ())
-                move_off |= MOVE_FLY_LOW;
-              else
-                move_off &= ~MOVE_FLY_LOW;
-            }
-            break;
             //TODO: remove these after converting archetypes
           case KW_can_use_wand:
             GET_FLAG (this, FLAG_USE_RANGE);
             break;
-            /* These are all legacy - any new objects should use the move_ .. values */
-          case KW_flying:
-            {
-              if (f.get_sint32 ())
-                move_type |= MOVE_FLY_LOW;
-              else
-                move_type &= ~MOVE_FLY_LOW;
-            }
-            break;
           case KW_identified:
             GET_FLAG (this, FLAG_IDENTIFIED);
             //TODO: move to check_object or so
             if (this->flag [FLAG_IDENTIFIED])
           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_is_transparent_floor: GET_FLAG (this, FLAG_IS_TRANSPARENT_FLOOR); 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_is_buildable:         GET_FLAG (this, FLAG_IS_BUILDABLE); break;
           case KW_destroy_on_death:     GET_FLAG (this, FLAG_DESTROY_ON_DEATH); break;
           case KW_treasure_env:         GET_FLAG (this, FLAG_TREASURE_ENV); break;
           case KW_precious:             GET_FLAG (this, FLAG_PRECIOUS); break;
           case KW_random_speed:         GET_FLAG (this, FLAG_RANDOM_SPEED); break;
+          case KW_is_quad:              GET_FLAG (this, FLAG_IS_QUAD); 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;
             f.next ();
             return true;
           case KW_ERROR:
-            // append as key value pair (do not use kv_set as it prepends)
-            // we also do not even try to find old values, duplicate keys stay duplicate
+            // we do not even try to find old values, duplicate keys stay duplicate
-            {
+            // the list gets reversed after loading
-              key_value *kv = new key_value;
+            kv.add (shstr (f.kw_str), shstr (f.value));
-              kv->next  = 0;
-              kv->key   = shstr (f.kw_str);
-              kv->value = shstr (f.value);
-              if (!last_kv)
-                key_values = last_kv = kv;
-              else
-                {
-                  while (last_kv->next)
-                    last_kv = last_kv->next;
-                  last_kv->next = kv;
-                  last_kv       = kv;
-                }
-            }
-            //fprintf (stderr, "addkv(%s,%s)\n", f.kw_str, f.value);//D
             break;
           default:
             if (!f.parse_error ("object", name))
               return false;
   archetype *arch = archetype::find (f.get_str ());
   if (!arch)
     {
-      LOG (llevError, "object refering to nonexistant archetype '%s'.\n", f.get_str ());
+      LOG (llevError, "object referring to nonexistent archetype '%s'.\n", f.get_str ());
       arch = archetype::find (shstr_earthwall);
     }
   assert (arch); //D maybe use exception handling of sorts?
   if (!op->parse_kv (f))
     {
       op->destroy ();
       return  0;
     }
+  // the loader reverses the ordering of kv-pairs, so we reverse it again after loading
+  // that greatly simplifies the loading code.
+  op->kv.reverse ();
   op->post_load_check ();
   return op;
 }
     KW_NULL,
     KW_applied,
     KW_unpaid,
     KW_can_use_shield,
     KW_no_pick,
-    KW_NULL, // walk_on
+    KW_is_transparent_floor,
     /* 10 */
     KW_NULL, // no_pass
     KW_is_animated,
     KW_NULL, // slow_move
     KW_NULL, // flying
   } flagmask;
   /* This saves the key/value lists.  We do it first so that any
    * keys that match field names will be overwritten by the loader.
    */
-  for (key_value *kv = op->key_values; kv; kv = kv->next)
+  for (key_value *kv = op->kv.first; kv; kv = kv->next)
-    if (!arch->key_values || arch->kv (kv->key) != kv->value)
+    if (arch->kv.empty () || arch->kv [kv->key] != kv->value)
       f.put (kv->key, kv->value);
   if (op->uuid)
     {
       // highly optimised - this is often 25% of all data written
 #endif
   // save body locations. gcc's memcmp does an abysmal job when used
   for (int i = 0; i < NUM_BODY_LOCATIONS; i++)
     if (expect_false (op->slot[i].info != arch->slot[i].info))
-      f.put (body_locations[i].save_name, op->slot[i].info);
+      f.put (body_locations[i].kw, op->slot[i].info);
 }
 /*
  * Dumps all variables in an object to a file.
  * If bit 0 of flag is set, unpaid objects will be saved.  As of now,

Diff Legend

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

Comparing deliantra/server/common/loader.C (file contents): Revision 1.162 by root, Fri May 21 00:31:13 2010 UTC vs. Revision 1.173 by root, Fri Jan 27 22:00:39 2012 UTC

Diff Legend

Comparing deliantra/server/common/loader.C (file contents):
Revision 1.162 by root, Fri May 21 00:31:13 2010 UTC vs.
Revision 1.173 by root, Fri Jan 27 22:00:39 2012 UTC