--- deliantra/server/common/arch.C	2008/04/30 08:29:31	1.77
+++ deliantra/server/common/arch.C	2008/10/05 14:54:05	1.86
@@ -104,7 +104,7 @@
 
   for_all_archetypes (at)
     if (at->skill == skill_cmp && (type == -1 || type == at->type))
-      return arch_to_object (at);
+      return at->instance ();
 
   return 0;
 }
@@ -135,7 +135,7 @@
  * - a corresponding object if found; a singularity object if not found.
  * Note by MSW - it appears that it takes the full name and keeps
  * shortening it until it finds a match.  I re-wrote this so that it
- * doesn't malloc it each time - not that this function is used much,
+ * doesn't allocate it each time - not that this function is used much,
  * but it otherwise had a big memory leak.
  */
 object *
@@ -151,7 +151,7 @@
       tmpname[i] = 0;
 
       if (archetype *at = find_archetype_by_object_name (tmpname))
-        return arch_to_object (at);
+        return at->instance ();
     }
 
   return create_singularity (name);
@@ -442,7 +442,7 @@
 
 fail:
   for (auto (p, parts.begin ()); p != parts.end (); ++p)
-    (*p)->destroy (true);
+    (*p)->destroy ();
 
   return 0;
 }
@@ -512,8 +512,8 @@
         ++idx;
       else
         {
-          LOG (llevDebug, "garbage collect arch %s", at->debug_desc ());
-          assert (at->arch == at); // verify that refcnt == 1 is truely valid
+          LOG (llevDebug, "garbage collect arch %s", &at->archname);
+          assert (at->arch == at); // verify that refcnt == 1 is truly valid
           allarch.erase (idx);
 
           // break chain
@@ -532,32 +532,14 @@
   while (--cnt);
 }
 
-/*
- * Creates and returns a new object which is a copy of the given archetype.
- * This function returns NULL on failure.
- */
 object *
-arch_to_object (archetype *at)
+archetype::instance ()
 {
-  if (!at)
-    {
-      LOG (llevError, "Couldn't find archetype.\n");
-      return 0;
-    }
-
-  object *op = at->clone ();
-  op->arch = at;
+  object *op = clone ();
   op->instantiate ();
-
   return op;
 }
 
-object *
-archetype::instance ()
-{
-  return arch_to_object (this);
-}
-
 /*
  * Creates an object.  This function is called by get_archetype()
  * if it fails to find the appropriate archetype.
@@ -572,11 +554,8 @@
   if (!strcmp (name, "bug"))
     abort ();
 
-  char buf[MAX_BUF];
-  sprintf (buf, "bug, please report (%s)", name);
-
-  object *op = get_archetype ("bug");
-  op->name = op->name_pl = buf;
+  object *op = archetype::get (shstr_bug);
+  op->name = op->name_pl = format ("bug, please report (missing archetype %s)", name);
 
   return op;
 }
@@ -588,18 +567,18 @@
 object *
 get_archetype (const char *name)
 {
-  archetype *at = archetype::find (name);
-
-  if (!at)
-    return create_singularity (name);
-
-  return arch_to_object (at);
+  return archetype::get (name);
 }
 
 object *
 archetype::get (const char *name)
 {
-  return get_archetype (name);
+  archetype *at = find (name);
+
+  if (!at)
+    return create_singularity (name);
+
+  return at->instance ();
 }
 
 /*
@@ -624,17 +603,15 @@
 object *
 clone_arch (int type)
 {
-  archetype *at;
+  archetype *at = type_to_archetype (type);
 
-  if ((at = type_to_archetype (type)) == NULL)
+  if (!at)
     {
       LOG (llevError, "Can't clone archetype %d\n", type);
       return 0;
     }
 
-  object *op = at->clone ();
-  op->instantiate ();
-  return op;
+  return at->instance ();
 }
 
 /*
@@ -647,7 +624,7 @@
 
   while (at)
     {
-      op = arch_to_object (at);
+      op = at->instance ();
 
       op->x = at->x;
       op->y = at->y;