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

Comparing deliantra/server/common/loader.C (file contents):
Revision 1.177 by root, Wed Nov 16 23:41:59 2016 UTC vs.
Revision 1.186 by root, Sun Jan 27 11:04:33 2019 UTC

 /*
  * This file is part of Deliantra, the Roguelike Realtime MMORPG.
  *
+ * Copyright (©) 2018 Marc Alexander Lehmann / the Deliantra team
  * Copyright (©) 2005,2006,2007,2008,2009,2010,2011,2012,2013,2014,2015,2016 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
   "spell_glyph",                /* 205 */
   NULL
 };
 /* 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
  * deal with legacy objects where fields may have changed.  It can also be used
  * to check for objects to make sure there are no common errors.
  */
 void
 object::post_load_check ()
             LOG (llevError, "ITEMBUG: %s: body_shield %d != -1\n", debug_desc (), slot [body_shield].info);
             slot [body_shield].info = -1;
           }
         break;
+      case CONTAINER:
+        if (nrof)
+          {
+            LOG (llevError, "ITEMBUG: %s: stackable container (nrof %d)\n", debug_desc (), (int)nrof);
+            nrof = 0;
+          }
+        break;
       case PLAYER:
         if (slot [body_shield].info != 1)
           {
             LOG (llevError, "ITEMBUG: %s: body_shield %d != 1\n", debug_desc (), slot [body_shield].info);
             slot [body_shield].info = 1;
    * this isn't perfect (things won't be properly pluralised), but works to
    * that degree (5 heart is still quite understandable).  But the case we
    * also have to catch is if this object is not using the normal name for
    * the object.  In that case, we also want to use the loaded name.
    * Otherwise, what happens is that the the plural name will lose
    * information (appear as just 'hearts' and not 'goblins heart')
    */
   if (arch && name != arch->object::name && name_pl == arch->object::name_pl)
     name_pl = 0;
   if (!name_pl)
         {
           negate = 1;
           str++;
         }
-      for (const flagstr *f = move_flags; f < move_flags + array_length (move_flags); ++f)
+      for (const flagstr *f = move_flags; f < move_flags + ecb_array_length (move_flags); ++f)
         {
           if (!strcmp (f->name, str))
             {
               if (negate)
                 mt &= ~f->flags;
     {
       op->destroy ();
       return  0;
     }
-  // the loader reverses the ordering of kv-pairs, so we reverse it again after loading
+  // The loader reverses the ordering of kv-pairs, so we reverse it again after loading.
-  // that greatly simplifies the loading code.
+  // This greatly simplifies the loading code.
   op->kv.reverse ();
   op->post_load_check ();
   return op;
 }
       *ptr++ = '\n';
       f.alloc (ptr - cur);
     }
-#define CMP_OUT(v)    if (expect_false (op->v != arch->v)) f.put (KW (v), op->v)
+#define CMP_OUT(v)    if (ecb_expect_false (op->v != arch->v)) f.put (KW (v), op->v)
-#define CMP_OUT2(k,v) if (expect_false (op->v != arch->v)) f.put (KW (k), op->v)
+#define CMP_OUT2(k,v) if (ecb_expect_false (op->v != arch->v)) f.put (KW (k), op->v)
   CMP_OUT (x);
   CMP_OUT (y);
   CMP_OUT (type);
       static int    last_speed_left_len = sizeof ("sl 0\n") - 1;
       if (last_speed_left != op->speed_left)
         {
           last_speed_left = op->speed_left;
+          // .7g loses precision even for float, but gives nice round numbers and smaller files
+          // maybe hex format or a raw binary dump of the float is good enough, more exact, and much faster?
+          // (printf is typically very slow)
           last_speed_left_len = sizeof ("sl ") - 1
                               + sprintf (last_speed_left_str + sizeof ("sl ") - 1, "%.7g\n", last_speed_left);
         }
       f.add (last_speed_left_str, last_speed_left_len);
   CMP_OUT (level);
   CMP_OUT (attacktype);
   // using memcmp here seems to be a loss - is gcc vectorising?
   for (int i = 0; i < NROFATTACKS; i++)
-    if (expect_false (op->resist[i] != arch->resist[i]))
+    if (ecb_expect_false (op->resist[i] != arch->resist[i]))
       f.put (resist_save[i], op->resist[i]);
   CMP_OUT (path_attuned);
   CMP_OUT (path_repelled);
   CMP_OUT (path_denied);
   CMP_OUT (last_grace);
   CMP_OUT (last_eat);
   CMP_OUT (glow_radius);
   if (op->flag [FLAG_IS_LINKED])
-    if (auto (ol, op->find_link ()))
+    if (auto ol = op->find_link ())
       f.put (KW(connected), ol->id);
   CMP_OUT (randomitems);
   CMP_OUT2 (container, weight_limit);
     f.put (flag_names [FLAG_OBJ_ORIGINAL], op->flag [FLAG_OBJ_ORIGINAL] ? CS(1) : CS(0));
   diff.reset (FLAG_OBJ_ORIGINAL);
   // quickly test whether any other flags differ
-  if (expect_true (diff.any ()))
+  if (ecb_expect_true (diff.any ()))
     for (int i = 0; i < NUM_FLAGS; i++)
-      if (expect_false (diff [i]))
+      if (ecb_expect_false (diff [i]))
         f.put (flag_names [i], op->flag [i] ? CS(1) : CS(0));
 #else
   // use sgi extensions
   for (int i = diff._Find_first (); i < diff.size (); i = diff._Find_next (i))
     f.put (flag_names [i], op->flag [i] ? CS(1) : CS(0));
 #endif
   // save body locations. gcc's memcmp does an abysmal job when used
-  for (int i = 0; i < NUM_BODY_LOCATIONS; i++)
+  for (int i = 0; i < NUM_BODY_LOCATIONS; ++i)
-    if (expect_false (op->slot[i].info != arch->slot[i].info))
+    if (ecb_expect_false (op->slot[i].info != arch->slot[i].info))
       f.put (body_locations[i].kw, op->slot[i].info);
 }
 /*
  * Dumps all variables in an object to a file.
  * the only place this is not set is when saving the player.
  */
 bool
 object::write (object_freezer &f)
 {
-  if (expect_false (is_arch ()))
+  if (ecb_expect_false (is_arch ()))
     {
       f.put (KW(object), arch->archname);
       write_diff (f, this, archetype::empty);
     }
   else

Diff Legend

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

Comparing deliantra/server/common/loader.C (file contents): Revision 1.177 by root, Wed Nov 16 23:41:59 2016 UTC vs. Revision 1.186 by root, Sun Jan 27 11:04:33 2019 UTC

Diff Legend

Comparing deliantra/server/common/loader.C (file contents):
Revision 1.177 by root, Wed Nov 16 23:41:59 2016 UTC vs.
Revision 1.186 by root, Sun Jan 27 11:04:33 2019 UTC