--- deliantra/server/common/loader.C 2007/06/04 12:19:08 1.87
+++ deliantra/server/common/loader.C 2007/09/04 05:43:20 1.100
@@ -1,23 +1,22 @@
/*
- * This file is part of Crossfire TRT, the Multiplayer Online Role Playing Game.
+ * This file is part of Crossfire TRT, the Roguelike Realtime MORPG.
*
* Copyright (©) 2005,2006,2007 Marc Alexander Lehmann / Robin Redeker / the Crossfire TRT team
* Copyright (©) 2002,2007 Mark Wedel & Crossfire Development Team
* Copyright (©) 1992,2007 Frank Tore Johansen
*
- * Crossfire TRT is free software; you can redistribute it and/or modify it
- * under the terms of the GNU General Public License as published by the Free
- * Software Foundation; either version 2 of the License, or (at your option)
- * any later version.
+ * Crossfire TRT is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
*
- * This program is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
- * or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
- * for more details.
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
*
- * You should have received a copy of the GNU General Public License along
- * with Crossfire TRT; if not, write to the Free Software Foundation, Inc. 51
- * Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see .
*
* The authors can be reached via e-mail to
*/
@@ -327,7 +326,7 @@
* Otherwise, what happens is that the the plural name will lose
* information (appear as just 'hearts' and not 'goblins heart')
*/
- if (arch && name != arch->clone.name && name_pl == arch->clone.name_pl)
+ if (arch && name != arch->object::name && name_pl == arch->object::name_pl)
name_pl = 0;
if (!name_pl)
@@ -412,7 +411,7 @@
insert_ob_in_ob (tmp, this);
randomitems = NULL; /* So another spell isn't created for this object */
/* without this, value is all screwed up */
- value = arch->clone.value * inv->value;
+ value = arch->value * inv->value;
}
if (QUERY_FLAG (this, FLAG_MONSTER))
@@ -604,6 +603,10 @@
LOG (llevError, "%s uses unknown other_arch '%s'.\n", debug_desc (), f.get_str ());
break;
+ case KW_owner:
+ f.delayed_deref (this, owner, f.get_str ());
+ break;
+
case KW_animation:
{
CLEAR_FLAG (this, FLAG_ANIMATE);
@@ -640,6 +643,18 @@
face = face_find (f.get_str ());
break;
+ case KW_sound:
+ sound = sound_find (f.get_str ());
+ if (!sound)
+ f.parse_warn ("sound not found");
+ break;
+
+ case KW_sound_destroy:
+ sound_destroy = sound_find (f.get_str ());
+ if (!sound_destroy)
+ f.parse_warn ("sound not found");
+ break;
+
case KW_x: f.get (x); break;
case KW_y: f.get (y); break;
@@ -961,23 +976,10 @@
case KW_msg:
f.get_ml (KW_endmsg, msg);
- //TODO: allow longer messages
- if (strlen (msg) >= HUGE_BUF)
- {
- 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:
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 (lore) > (HUGE_BUF / 2))
- LOG (llevDebug, "\tWarning lore length > %d (max allowed=%d): %d\n>%.80s<\n",
- HUGE_BUF / 2, HUGE_BUF, strlen (lore), &lore);
break;
case KW_editable:
@@ -1026,7 +1028,7 @@
object *op = object::create ();
op->map = map;
- arch->clone.copy_to (op);
+ arch->copy_to (op);
// copy_to activates, this should be fixed properly
op->deactivate ();
@@ -1051,7 +1053,6 @@
{
object_thawer f (buf, (AV *)0);
- f.next ();
return op->parse_kv (f);
}
@@ -1273,8 +1274,11 @@
f.put (KW_uuid, (const char *)uids);
}
-#define CMP_OUT(v) if (op->v != tmp->v) f.put (KW_ ## v, op->v)
-#define CMP_OUT2(k,v) if (op->v != tmp->v) f.put (KW_ ## k, op->v)
+#define CMP_OUT(v) if (expect_false (op->v != tmp->v)) f.put (KW_ ## v, op->v)
+#define CMP_OUT2(k,v) if (expect_false (op->v != tmp->v)) f.put (KW_ ## k, op->v)
+
+ if (object *owner = op->owner)
+ f.put (KW_owner, static_cast(owner->ref ()));
CMP_OUT (name);
CMP_OUT (name_pl);
@@ -1282,15 +1286,14 @@
CMP_OUT (title);
CMP_OUT (race);
CMP_OUT (slaying);
+ CMP_OUT (other_arch);
- if (op->msg != tmp->msg)
- f.put (KW_msg, KW_endmsg, op->msg);
- if (op->lore != tmp->lore)
- f.put (KW_lore, KW_endlore, op->lore);
- if (op->other_arch != tmp->other_arch)
- f.put (KW_other_arch, op->other_arch ? &op->other_arch->archname : 0);
+ if (op->msg != tmp->msg ) f.put (KW_msg , KW_endmsg , op->msg );
+ if (op->lore != tmp->lore) f.put (KW_lore, KW_endlore, op->lore);
- if (op->face != tmp->face) f.put (KW_face, op->face ? &faces [op->face] : 0);
+ if (op->face != tmp->face ) f.put (KW_face , op->face ? &faces [op->face ] : 0);
+ if (op->sound != tmp->sound ) f.put (KW_sound , op->sound ? &faces [op->sound ] : 0);
+ if (op->sound_destroy != tmp->sound_destroy) f.put (KW_sound_destroy, op->sound_destroy ? &faces [op->sound_destroy] : 0);
if (op->animation_id != tmp->animation_id)
if (op->animation_id)
@@ -1298,10 +1301,10 @@
f.put (KW_animation, animations[GET_ANIM_ID (op)].name);
if (!QUERY_FLAG (op, FLAG_ANIMATE))
- f.put (KW_is_animated, (sint32) 0);
+ f.put (KW_is_animated, (sint32)0);
}
else
- f.put (KW_animation, (const char *) 0);
+ f.put (KW_animation, (const char *)0);
CMP_OUT2 (str, stats.Str);
CMP_OUT2 (dex, stats.Dex);
@@ -1342,7 +1345,7 @@
CMP_OUT (attacktype);
for (i = 0; i < NROFATTACKS; i++)
- if (op->resist[i] != tmp->resist[i])
+ if (expect_false (op->resist[i] != tmp->resist[i]))
f.put (resist_save[i], op->resist[i]);
CMP_OUT (path_attuned);
@@ -1394,12 +1397,12 @@
if (op->flag != tmp->flag)
for (i = 0; i <= NUM_FLAGS; i++)
- if (flag_names [i] && op->flag [i] != tmp->flag [i])
+ if (expect_false (flag_names [i] && op->flag [i] != tmp->flag [i]))
f.put (flag_names [i], op->flag [i] ? "1" : "0");
// save body locations
for (i = 0; i < NUM_BODY_LOCATIONS; i++)
- if (op->slot[i].info != tmp->slot[i].info)
+ if (expect_false (op->slot[i].info != tmp->slot[i].info))
f.put (body_locations[i].save_name, op->slot[i].info);
}
@@ -1411,16 +1414,10 @@
bool
object::write (object_freezer &f)
{
- /* Even if the object does have an owner, it would seem that we should
- * still save it.
- */
- if (owner)
- return true;
-
archetype *at = arch ? (archetype *)arch : empty_archetype;
f.put (KW_arch, at->archname);
- write_diff (f, this, &at->clone);
+ write_diff (f, this, at);
for (object *tmp = inv; tmp; tmp = tmp->below)
tmp->write (f);
@@ -1442,8 +1439,6 @@
bool success = false;
bool seen_arch = false;
- f.next ();
-
for (;;)
{
switch (f.kw)