]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/base.cpp
Remove an extern, partly because it's unused, partly because it then gets shadowed...
[user/henk/code/inspircd.git] / src / base.cpp
index 9d03b6afb9725cbfef6d5607a4157a2896401326..eb6ea900ab1b12c2f7101491d53199a449b7fc98 100644 (file)
@@ -2,7 +2,7 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  Inspire is copyright (C) 2002-2004 ChatSpike-Dev.
+ *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
  *                       E-mail:
  *                <brain@chatspike.net>
  *               <Craig@chatspike.net>
 #include "base.h"
 #include <time.h>
 #include <map>
+#include <deque>
 #include <string>
 #include "inspircd.h"
 #include "modules.h"
 #include "helperfuncs.h"
 
+const int bitfields[]           =       {1,2,4,8,16,32,64,128};
+const int inverted_bitfields[]  =       {~1,~2,~4,~8,~16,~32,~64,~128};
+
 extern time_t TIME;
 
-bool Extensible::Extend(std::string key, char* p)
+/* This is now a template in base.h
+ *
+ * bool Extensible::Extend(const std::string &key, char* p)
+ * {
+ *     // only add an item if it doesnt already exist
+ *     if (this->Extension_Items.find(key) == this->Extension_Items.end())
+ *     {
+ *             this->Extension_Items[key] = p;
+ *             log(DEBUG,"Extending object with item %s",key.c_str());
+ *             return true;
+ *     }
+ *     // item already exists, return false
+ *     return false;
+ * }
+ */
+
+bool Extensible::Shrink(const std::string &key)
 {
-       // only add an item if it doesnt already exist
-       if (this->Extension_Items.find(key) == this->Extension_Items.end())
+       /* map::size_type map::erase( const key_type& key );
+        * returns the number of elements removed, std::map
+        * is single-associative so this should only be 0 or 1
+        */
+       if(this->Extension_Items.erase(key))
        {
-               this->Extension_Items[key] = p;
-               log(DEBUG,"Extending object with item %s",key.c_str());
+               log(DEBUG, "Shrinking object with item %s",key.c_str());
                return true;
        }
-       // item already exists, return false
-       return false;
+       else
+       {
+               log(DEBUG, "Tried to shrink object with item %s but no items removed", key.c_str());            
+               return false;
+       }
 }
 
-bool Extensible::Shrink(std::string key)
+char* Extensible::GetExt(const std::string &key)
 {
-       // only attempt to remove a map item that exists
-       if (this->Extension_Items.find(key) != this->Extension_Items.end())
+       /* This was calling ExtensibleStore::find() twice,
+        * once to see if there was a value, and again to
+        * get that value if it was there. Now we store
+        * the iterator so we only have to search for it once.
+        */
+       ExtensibleStore::iterator iter = this->Extension_Items.find(key);
+       
+       if(iter != this->Extension_Items.end())
        {
-               this->Extension_Items.erase(this->Extension_Items.find(key));
-               log(DEBUG,"Shrinking object with item %s",key.c_str());
-               return true;
+               return iter->second;
+       }
+       else
+       {
+               return NULL;
        }
-       return false;
 }
 
-char* Extensible::GetExt(std::string key)
+void Extensible::GetExtList(std::deque<std::string> &list)
 {
-       if (this->Extension_Items.find(key) != this->Extension_Items.end())
+       for (ExtensibleStore::iterator u = Extension_Items.begin(); u != Extension_Items.end(); u++)
        {
-               return (this->Extension_Items.find(key))->second;
+               list.push_back(u->first);
        }
-       log(DEBUG,"Cant find item %s",key.c_str());
-       return NULL;
 }
 
 void BoolSet::Set(int number)
@@ -109,6 +139,6 @@ BoolSet::BoolSet(char bitmask)
 
 bool BoolSet::operator=(BoolSet other)
 {
-        this->bits = other.bits;
-        return true;
+       this->bits = other.bits;
+       return true;
 }