--- deliantra/server/include/object.h	2008/12/23 00:39:48	1.189
+++ deliantra/server/include/object.h	2009/01/06 19:17:06	1.195
@@ -310,36 +310,36 @@
   key_value *key_values;	/* Fields not explictly known by the loader. */
 
   // privates / perl
-  const shstr &kv_get (const shstr &key) const;
-  void kv_del (const shstr &key);
-  void kv_set (const shstr &key, const shstr &value);
+  shstr_tmp kv_get (shstr_tmp key) const;
+  void kv_del (shstr_tmp key);
+  void kv_set (shstr_tmp key, shstr_tmp value);
 
   // custom extra fields management
   struct key_value_access_proxy
   {
     object &ob;
-    shstr key;
+    shstr_tmp key;
 
-    key_value_access_proxy (object &ob, const shstr &key)
+    key_value_access_proxy (object &ob, shstr_tmp key)
     : ob (ob), key (key)
     {
     }
 
-    const key_value_access_proxy &operator =(const shstr &value) const
+    const key_value_access_proxy &operator =(shstr_tmp value) const
     {
       ob.kv_set (key, value);
       return *this;
     }
 
-    operator const shstr &() const { return ob.kv_get (key); }
-    operator const char  *() const { return ob.kv_get (key); }
+    operator const shstr_tmp () const { return ob.kv_get (key); }
+    operator const char     *() const { return ob.kv_get (key); }
 
   private:
     void operator =(int);
   };
 
   // operator [] is too annoying to use
-  const key_value_access_proxy kv (const shstr &key)
+  const key_value_access_proxy kv (shstr_tmp key)
   {
     return key_value_access_proxy (*this, key);
   }
@@ -385,6 +385,14 @@
     insert_at (this, this);
   }
 
+  // high-level move functions, return true if successful
+  int move (int dir, object *originator);
+
+  int move (int dir)
+  {
+    return move (dir, this);
+  }
+
   static bool can_merge_slow (object *op1, object *op2);
 
   // this is often used in time-critical code, so optimise
@@ -406,8 +414,8 @@
     open_container (0);
   }
 
-  MTH object *force_find (const shstr name);
-  MTH object *force_add (const shstr name, int duration = 0);
+  MTH object *force_find (shstr_tmp name);
+  MTH object *force_add (shstr_tmp name, int duration = 0);
 
   // overwrite the attachable should_invoke function with a version that also checks ev_want_type
   bool should_invoke (event_type event)
@@ -453,7 +461,7 @@
   // return the volume of this object in cm³
   MTH uint64 volume () const
   {
-    return total_weight ()
+    return (uint64)total_weight ()
            * 1000
            * (type == CONTAINER ? 1000 : 1)
            / dominant_material ()->density;
@@ -540,11 +548,21 @@
     return !flag [FLAG_REMOVED];
   }
 
+  MTH bool is_player () const
+  {
+    return !!contr;
+  }
+
   MTH bool affects_los () const
   {
     return glow_radius || flag [FLAG_BLOCKSVIEW];
   }
 
+  MTH bool has_carried_lights () const
+  {
+    return glow_radius;
+  }
+
   // returns the player that cna see this object, if any
   MTH object *visible_to () const;
 
@@ -694,8 +712,8 @@
   struct oblinkpt *next;
 } oblinkpt;
 
-object *find_skill_by_name (object *who, const char *name);
-object *find_skill_by_name (object *who, const shstr &sh);
+object *find_skill_by_name_fuzzy (object *who, const char *name);
+object *find_skill_by_name (object *who, shstr_cmp sh);
 object *find_skill_by_number (object *who, int skillno);
 
 /*
@@ -759,7 +777,7 @@
 extern archvec   archetypes;
 
 // "safely" iterate over inv in a way such that the current item is removable
-// quite horrible, thats why its hidden in some macro
+// quite horrible, that's why its hidden in some macro
 #define for_inv_removable(op,var)			\
   for (object *var, *next_ = (op)->inv; (var = next_), var && (next_ = var->below), var; )