Revert "Use a do..while() loop instead of this ugly continue statement."

[loba.git] / named_object_list.h

diff --git a/named_object_list.h b/named_object_list.h
index ae7229dcdef8c91c218961277a7e7151ca24cb3b..94d281eb6da6de48025800f5d985f5716ebf9d85 100644 (file)
--- a/named_object_list.h
+++ b/named_object_list.h
@@ -4,6 +4,17 @@
 #include <map>
 #include <string>
 
+// Define an associative container that maps a name with a class and a
+// description.  All classes must be derived from a same base class.
+//
+// We can then use the name to create an object of the associated
+// class, and to retrieve a pointer to this object.
+//
+// Furthermore, it is possible to iterate over the elements to get
+// their name and their description.
+
+// I am too lazy to comment the code, which should be obvious...
+
 //===== arity 0 =====
 
 template <typename Base>
@@ -56,11 +67,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; }
@@ -74,9 +80,6 @@ public:
 
 //===== arity 2 =====
 
-#include <map>
-#include <string>
-
 template <typename Base, typename Arg1, typename Arg2>
 class named_object_list2 {
 protected:
@@ -128,11 +131,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; }
@@ -146,6 +144,7 @@ public:
 
 //===================
 
+// "NOL" like in Named_Object_List....
 #define NOL_INSERT(name, descr, class) insert(name, new creator<class>(descr))
 
 #endif // !NAMED_OBJECT_LIST_H