summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorom <om@e03df62e-2008-0410-955e-edbf42e46eb7>2006-04-10 19:30:48 +0000
committerom <om@e03df62e-2008-0410-955e-edbf42e46eb7>2006-04-10 19:30:48 +0000
commit5fb54ac29a36131d2e49991095ef9741abb5f94e (patch)
treeac0bdf4a48e484e847d7685ced77a28bd879169c /src
parent3c272bcc1f53c2a8ebbf5b399118d33b5640cf9b (diff)
Assorted changes here, Extend() is not templated so you can pass it any pointer type and it will automatically be cast to char* for you. Also some speedups, store an iterator rather than calling find() once to check if it exists and again to access the value.
git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@3864 e03df62e-2008-0410-955e-edbf42e46eb7
Diffstat (limited to 'src')
-rw-r--r--src/base.cpp61
1 files changed, 40 insertions, 21 deletions
diff --git a/src/base.cpp b/src/base.cpp
index 53ebda798..eb6ea900a 100644
--- a/src/base.cpp
+++ b/src/base.cpp
@@ -29,43 +29,62 @@ const int inverted_bitfields[] = {~1,~2,~4,~8,~16,~32,~64,~128};
extern time_t TIME;
-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;
-}
+/* 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 attempt to remove a map item that exists
- 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.erase(this->Extension_Items.find(key));
- log(DEBUG,"Shrinking object with item %s",key.c_str());
+ log(DEBUG, "Shrinking object with item %s",key.c_str());
return true;
}
- return false;
+ else
+ {
+ log(DEBUG, "Tried to shrink object with item %s but no items removed", key.c_str());
+ return false;
+ }
}
char* Extensible::GetExt(const std::string &key)
{
- 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())
+ {
+ return iter->second;
+ }
+ else
{
- return (this->Extension_Items.find(key))->second;
+ return NULL;
}
- return NULL;
}
void Extensible::GetExtList(std::deque<std::string> &list)
{
- for (std::map<std::string,char*>::iterator u = Extension_Items.begin(); u != Extension_Items.end(); u++)
+ for (ExtensibleStore::iterator u = Extension_Items.begin(); u != Extension_Items.end(); u++)
{
list.push_back(u->first);
}