Enable the resource traits classes to hold non-static members

ps: the Resource class inherits from the Traits class instead of holding it as a member because
it's more size-efficient (base classes can have zero size, members cannot).

2 files changed, 19 insertions, 11 deletions

diff --git a/src/FbTk/ResTraits.hh b/src/FbTk/ResTraits.hh
index 3284801..19decb5 100644
--- a/src/FbTk/ResTraits.hh
+++ b/src/FbTk/ResTraits.hh
@@ -160,14 +160,17 @@ struct EnumTraits {
     }
 };
 
-template<typename Traits, const char *delim>
+template<typename Traits>
 struct VectorTraits {
     typedef std::vector<typename Traits::Type> Type;
-    static std::string toString(const Type &x) {
+
+    VectorTraits(const std::string &delim) : m_delim(delim) {}
+
+    std::string toString(const Type &x) const {
         std::string retval;
         for(size_t i = 0; i < x.size(); ++i) {
-            retval.append(Traits::toString(x[i]));
-            retval.append(" ");
+            retval += Traits::toString(x[i]);
+            retval += m_delim[0];
         }
 
         return retval;
@@ -184,9 +187,9 @@ struct VectorTraits {
         }
     }
 
-    static Type fromString(const std::string &x) {
+    Type fromString(const std::string &x) const {
         std::vector<std::string> val;
-        StringUtil::stringtok(val, x, delim);
+        StringUtil::stringtok(val, x, m_delim.c_str());
         Type retval;
 
         for(size_t i = 0; i < val.size(); i++) {
@@ -218,6 +221,9 @@ struct VectorTraits {
         throw ConversionError( std::string("Cannot convert to vector from lua type ")
                                 + l.type_name(l.type(-1)) );
     }
+
+private:
+    std::string m_delim;
 };
 
 } // end namespace FbTk
diff --git a/src/FbTk/Resource.hh b/src/FbTk/Resource.hh
index 32d24b6..c6f12d6 100644
--- a/src/FbTk/Resource.hh
+++ b/src/FbTk/Resource.hh
@@ -188,19 +188,21 @@ private:
  * If there is no traits class for your type, you have to implement one.
  */
 template <typename T, typename Traits>
-class Resource:public Resource_base, public Accessor<T> {
+class Resource:public Resource_base, public Accessor<T>, private Traits {
 public:
     typedef T Type;
 
-    Resource(ResourceManager_base &rm, T val, const std::string &name, const std::string &altname):
-        Resource_base(name, altname), m_value(val), m_defaultval(val), m_rm(rm) {
+    Resource(ResourceManager_base &rm, T val, const std::string &name,
+            const std::string &altname, const Traits &traits = Traits() ):
+        Resource_base(name, altname), Traits(traits), m_value(val), m_defaultval(val), m_rm(rm) {
         m_rm.addResource(*this); // add this to resource handler
     }
 
     // LResourceManager does not use altname, so we provide a constructor which initializes
     // altname to name
-    Resource(ResourceManager_base &rm, T val, const std::string &name):
-        Resource_base(name, name), m_value(val), m_defaultval(val), m_rm(rm) {
+    Resource(ResourceManager_base &rm, T val, const std::string &name,
+            const Traits &traits = Traits() ):
+        Resource_base(name, name), Traits(traits), m_value(val), m_defaultval(val), m_rm(rm) {
         m_rm.addResource(*this); // add this to resource handler
     }
     virtual ~Resource() {