--- deliantra/server/common/shstr.C	2006/09/03 00:18:40	1.3
+++ deliantra/server/common/shstr.C	2009/01/01 21:15:12	1.38
@@ -1,109 +1,164 @@
 /*
+ * This file is part of Deliantra, the Roguelike Realtime MMORPG.
+ * 
+ * Copyright (©) 2005,2006,2007,2008 Marc Alexander Lehmann / Robin Redeker / the Deliantra team
+ * 
+ * Deliantra 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.
+ * 
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ * 
+ * The authors can be reached via e-mail to <support@deliantra.net>
+ */
+
+/*
  * shstr.C
  */
 
 #include <cstring>
 #include <cstdlib>
-
+#include <glib.h>
 #include <tr1/unordered_set>
 
-#include "shstr.h"
+#include "global.h"
 
-struct hash
-{
-  std::size_t operator ()(const char *s) const
-  {
-    unsigned long hash = 0;
-    unsigned int i = 0;
-
-    /* use the one-at-a-time hash function, which supposedly is
-     * better than the djb2-like one used by perl5.005, but
-     * certainly is better then the bug used here before.
-     * see http://burtleburtle.net/bob/hash/doobs.html
-     */
-    while (*s)
-      {
-        hash += *s++;
-        hash += hash << 10;
-        hash ^= hash >>  6;
-      }
-
-    hash += hash <<  3;
-    hash ^= hash >> 11;
-    hash += hash << 15;
-
-    return hash;
-  }
-};
+size_t shstr_alloc;
 
-struct equal
+typedef std::tr1::unordered_set <const char *, str_hash, str_equal, slice_allocator<const char *> > HT;
+
+static HT ht;
+static int next_gc;
+
+static const char *
+makevec (const char *s)
 {
-  bool operator ()(const char *a, const char *b) const
-  {
-    return !strcmp (a, b);
-  }
-};
+  int len = strlen (s);
+  int alloc = sizeof (uint32_t) * 2 + len + 1;
 
-typedef std::tr1::unordered_set<const char *, hash, equal> HT;
+  shstr_alloc += alloc;
+  char *v = (char *)g_slice_alloc (alloc);
+  v += sizeof (uint32_t) * 2;
 
-static HT ht;
+  shstr::length (v) = len;
+  shstr::refcnt (v) = 1;
+
+  memcpy (v, s, len + 1);
+
+  return v;
+}
+
+shstr_vec<sizeof "(null)"> shstr_tmp::nullvec = { 0, 0xffffffff, "(null)" };
 
 const char *
 shstr::find (const char *s)
 {
-  HT::iterator i = ht.find (s);
+  if (!s)
+    return s;
 
-  return i != ht.end ()
-    ? (char *)*i
-    : 0;
+  auto (i, ht.find (s));
+
+  return i != ht.end ()? *i : 0;
 }
 
 const char *
 shstr::intern (const char *s)
 {
-  HT::iterator i = ht.find (s);
+  if (!s)
+    return null ();
 
-  if (i != ht.end ())
-    return (char *)*i;
+  if (const char *found = find (s))
+    {
+      ++refcnt (found);
+      return found;
+    }
+
+  --next_gc;
+  s = makevec (s);
+  ht.insert (s);
+  return s;
+}
 
-  int len = strlen (s);
+// periodically test refcounts == 0 for a few strings
+// this is the ONLY thing that erases stuff from ht. keep it that way.
+void
+shstr::gc ()
+{
+  if (expect_true (next_gc > 0))
+    return;
 
-  int *v = (int *)malloc (sizeof (int) * 2 + len + 1);
+  static const char *curpos;
 
-  v [0] = len;
-  v [1] = 0;
+  auto (i, curpos ? ht.find (curpos) : ht.begin ());
 
-  v += 2;
+  if (i == ht.end ())
+    i = ht.begin ();
+
+  int n = ht.size () / 256 + 16;
+  next_gc += n >> 1;
+
+  for (;;)
+    {
+      if (i == ht.end ())
+        {
+          curpos = 0;
+          return;
+        }
+      else if (!--n)
+        break;
+      else if (!refcnt (*i))
+        {
+          auto (o, i++);
+          const char *s = *o;
+
+          ht.erase (o);
+
+          //printf ("GC %4d %3d %d >%s<%d\n", (int)ht.size (), n, shstr::refcnt (s), s, shstr::length (s));
+          int alloc = sizeof (uint32_t) * 2 + length (s) + 1;
+          shstr_alloc -= alloc;
+          g_slice_free1 (alloc, (void *)(s - sizeof (uint32_t) * 2));
+        }
+      else
+        ++i;
+    }
 
-  memcpy (v, s, len + 1);
+  curpos = *i;
+}
 
-  ht.insert ((char *)v);
+// declare these here to get correct initialisation order
+#define def2(id,str) const shstr id (str);
+#define def(id)      def2(shstr_ ## id, # id)
+# include "shstrinc.h"
+#undef def
+#undef def2
 
-  return (char *)v;
-}
+shstr skill_names[NUM_SKILLS];
 
-// TODO: periodically test refcounts == 0 for a few strings (e.g. one hash bucket,
-// exploiting the fatc that iterators stay valid for unordered_set).
-void
-shstr::gc ()
-{
-}
+//TODO: this should of course not be here
 
 /* buf_overflow() - we don't want to exceed the buffer size of 
  * buf1 by adding on buf2! Returns true if overflow will occur.
  */
-
-int 
+int
 buf_overflow (const char *buf1, const char *buf2, int bufsize)
 {
-    int len1 = 0, len2 = 0;
+  int len1 = 0, len2 = 0;
 
-    if (buf1)
-        len1 = strlen (buf1);
-    if (buf2)
-        len2 = strlen (buf2);
-    if ((len1 + len2) >= bufsize)
-        return 1;
-    return 0;
-}
+  if (buf1)
+    len1 = strlen (buf1);
+
+  if (buf2)
+    len2 = strlen (buf2);
 
+  if ((len1 + len2) >= bufsize)
+    return 1;
+
+  return 0;
+}