Be consistent and hold mutex in any case when returning from condition_t::timedwait().

[loba.git] / named_object_list.h

diff --git a/named_object_list.h b/named_object_list.h
index 929a0dbee112ca1d0796de11534980c9f70b4044..2c691faba198aec9b64e308e5535c3dbeff16628 100644 (file)
--- a/named_object_list.h
+++ b/named_object_list.h
@@ -24,6 +24,7 @@ protected:
         std::string description;
         creator_base(const std::string& descr): description(descr) { }
         creator_base(const char* descr): description(descr) { }
+        virtual ~creator_base() { }
         virtual Base* operator()() const = 0;
     };
 
@@ -67,11 +68,6 @@ public:
             return NULL;
     }
 
-    Base* new_instance(const char* name) const
-    {
-        return new_instance(std::string(name));
-    }
-
     const std::string& get_name(iterator& it) const { return it->first; }
     const std::string& get_descr(iterator& it) const
     { return it->second->description; }
@@ -92,6 +88,7 @@ protected:
         std::string description;
         creator_base(const std::string& descr): description(descr) { }
         creator_base(const char* descr): description(descr) { }
+        virtual ~creator_base() { }
         virtual Base* operator()(Arg1, Arg2) const = 0;
     };
 
@@ -136,11 +133,6 @@ public:
             return NULL;
     }
 
-    Base* new_instance(const char* name, Arg1 arg1, Arg2 arg2) const
-    {
-        return new_instance(std::string(name), arg1, arg2);
-    }
-
     const std::string& get_name(iterator& it) const { return it->first; }
     const std::string& get_descr(iterator& it) const
     { return it->second->description; }