]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - src/hashcomp.cpp
irc::stringjoiner cleanup
[user/henk/code/inspircd.git] / src / hashcomp.cpp
index 67dec4e852c12d1b575849aa681658715639c257..9fded8b4b303f559fea3d13e369ccab81744a0bc 100644 (file)
@@ -1,25 +1,29 @@
-/*       +------------------------------------+
- *       | Inspire Internet Relay Chat Daemon |
- *       +------------------------------------+
+/*
+ * InspIRCd -- Internet Relay Chat Daemon
  *
- *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
- *                    E-mail:
- *             <brain@chatspike.net>
- *             <Craig@chatspike.net>
+ *   Copyright (C) 2009 Daniel De Graaf <danieldg@inspircd.org>
+ *   Copyright (C) 2005-2009 Craig Edwards <craigedwards@brainbox.cc>
+ *   Copyright (C) 2007-2008 Robin Burchell <robin+git@viroteck.net>
+ *   Copyright (C) 2007 Dennis Friis <peavey@inspircd.org>
  *
- * Written by Craig Edwards, Craig McLure, and others.
- * This program is free but copyrighted software; see
- *         the file COPYING for details.
+ * This file is part of InspIRCd.  InspIRCd is free software: you can
+ * redistribute it and/or modify it under the terms of the GNU General Public
+ * License as published by the Free Software Foundation, version 2.
  *
- * ---------------------------------------------------
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+ * FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+ * details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-using namespace std;
+
+/* $Core */
 
 #include "inspircd.h"
 #include "hashcomp.h"
-#include <ext/hash_map>
-#define nspace __gnu_cxx
 
 /******************************************************
  *
@@ -30,7 +34,7 @@ using namespace std;
  * scene spend a lot of time debating (arguing) about
  * the best way to write hash functions to hash irc
  * nicknames, channels etc.
- * We are lucky as C++ developers as hash_map does
+ * We are lucky as C++ developers as unordered_map does
  * a lot of this for us. It does intellegent memory
  * requests, bucketing, search functions, insertion
  * and deletion etc. All we have to do is write some
@@ -44,76 +48,144 @@ using namespace std;
  * Scandanavian Comparisons: The characters [, ], \ will
  * be considered the lowercase of {, } and |.
  *
- * This file also contains hashing methods for hashing
- * insp_inaddr structs, we use this if we want to cache IP
- * addresses.
- *
  ******************************************************/
 
-using namespace std;
-using namespace irc::sockets;
 
-/* convert a string to lowercase. Note following special circumstances
- * taken from RFC 1459. Many "official" server branches still hold to this
- * rule so i will too;
- *
- *  Because of IRC's scandanavian origin, the characters {}| are
- *  considered to be the lower case equivalents of the characters []\,
- *  respectively. This is a critical issue when determining the
- *  equivalence of two nicknames.
+/**
+ * A case insensitive mapping of characters from upper case to lower case for
+ * the ASCII character set.
+ */
+unsigned const char ascii_case_insensitive_map[256] = {
+       0,   1,   2,   3,   4,   5,   6,   7,   8,   9,   // 0-9
+       10,  11,  12,  13,  14,  15,  16,  17,  18,  19,  // 10-19
+       20,  21,  22,  23,  24,  25,  26,  27,  28,  29,  // 20-29
+       30,  31,  32,  33,  34,  35,  36,  37,  38,  39,  // 30-39
+       40,  41,  42,  43,  44,  45,  46,  47,  48,  49,  // 40-49
+       50,  51,  52,  53,  54,  55,  56,  57,  58,  59,  // 50-59
+       60,  61,  62,  63,  64,  97,  98,  99,  100, 101, // 60-69
+       102, 103, 104, 105, 106, 107, 108, 109, 110, 111, // 70-79
+       112, 113, 114, 115, 116, 117, 118, 119, 120, 121, // 80-89
+       122, 91,  92,  93,  94,  95,  96,  97,  98,  99,  // 90-99
+       100, 101, 102, 103, 104, 105, 106, 107, 108, 109, // 100-109
+       110, 111, 112, 113, 114, 115, 116, 117, 118, 119, // 110-119
+       120, 121, 122, 123, 124, 125, 126, 127, 128, 129, // 120-129
+       130, 131, 132, 133, 134, 135, 136, 137, 138, 139, // 130-139
+       140, 141, 142, 143, 144, 145, 146, 147, 148, 149, // 140-149
+       150, 151, 152, 153, 154, 155, 156, 157, 158, 159, // 150-159
+       160, 161, 162, 163, 164, 165, 166, 167, 168, 169, // 160-169
+       170, 171, 172, 173, 174, 175, 176, 177, 178, 179, // 170-179
+       180, 181, 182, 183, 184, 185, 186, 187, 188, 189, // 180-189
+       190, 191, 192, 193, 194, 195, 196, 197, 198, 199, // 190-199
+       200, 201, 202, 203, 204, 205, 206, 207, 208, 209, // 200-209
+       210, 211, 212, 213, 214, 215, 216, 217, 218, 219, // 210-219
+       220, 221, 222, 223, 224, 225, 226, 227, 228, 229, // 220-229
+       230, 231, 232, 233, 234, 235, 236, 237, 238, 239, // 230-249
+       240, 241, 242, 243, 244, 245, 246, 247, 248, 249, // 240-249
+       250, 251, 252, 253, 254, 255,                     // 250-255
+};
+
+
+
+/**
+ * A case insensitive mapping of characters from upper case to lower case for
+ * the character set of RFC 1459. This is identical to ASCII with the small
+ * exception of {}| being considered to be the lower case equivalents of the
+ * characters []\ respectively.
  */
-void nspace::strlower(char *n)
+unsigned const char rfc_case_insensitive_map[256] = {
+       0,   1,   2,   3,   4,   5,   6,   7,   8,   9,   // 0-9
+       10,  11,  12,  13,  14,  15,  16,  17,  18,  19,  // 10-19
+       20,  21,  22,  23,  24,  25,  26,  27,  28,  29,  // 20-29
+       30,  31,  32,  33,  34,  35,  36,  37,  38,  39,  // 30-39
+       40,  41,  42,  43,  44,  45,  46,  47,  48,  49,  // 40-49
+       50,  51,  52,  53,  54,  55,  56,  57,  58,  59,  // 50-59
+       60,  61,  62,  63,  64,  97,  98,  99,  100, 101, // 60-69
+       102, 103, 104, 105, 106, 107, 108, 109, 110, 111, // 70-79
+       112, 113, 114, 115, 116, 117, 118, 119, 120, 121, // 80-89
+       122, 123, 124, 125, 94,  95,  96,  97,  98,  99,  // 90-99
+       100, 101, 102, 103, 104, 105, 106, 107, 108, 109, // 100-109
+       110, 111, 112, 113, 114, 115, 116, 117, 118, 119, // 110-119
+       120, 121, 122, 123, 124, 125, 126, 127, 128, 129, // 120-129
+       130, 131, 132, 133, 134, 135, 136, 137, 138, 139, // 130-139
+       140, 141, 142, 143, 144, 145, 146, 147, 148, 149, // 140-149
+       150, 151, 152, 153, 154, 155, 156, 157, 158, 159, // 150-159
+       160, 161, 162, 163, 164, 165, 166, 167, 168, 169, // 160-169
+       170, 171, 172, 173, 174, 175, 176, 177, 178, 179, // 170-179
+       180, 181, 182, 183, 184, 185, 186, 187, 188, 189, // 180-189
+       190, 191, 192, 193, 194, 195, 196, 197, 198, 199, // 190-199
+       200, 201, 202, 203, 204, 205, 206, 207, 208, 209, // 200-209
+       210, 211, 212, 213, 214, 215, 216, 217, 218, 219, // 210-219
+       220, 221, 222, 223, 224, 225, 226, 227, 228, 229, // 220-229
+       230, 231, 232, 233, 234, 235, 236, 237, 238, 239, // 230-239
+       240, 241, 242, 243, 244, 245, 246, 247, 248, 249, // 240-249
+       250, 251, 252, 253, 254, 255,                     // 250-255
+};
+
+/**
+ * A case sensitive mapping of characters from upper case to lower case for the
+ * character set of RFC 1459. This is identical to ASCII.
+ */
+unsigned const char rfc_case_sensitive_map[256] = {
+       0,   1,   2,   3,   4,   5,   6,   7,   8,   9,   // 0-9
+       10,  11,  12,  13,  14,  15,  16,  17,  18,  19,  // 10-19
+       20,  21,  22,  23,  24,  25,  26,  27,  28,  29,  // 20-29
+       30,  31,  32,  33,  34,  35,  36,  37,  38,  39,  // 30-39
+       40,  41,  42,  43,  44,  45,  46,  47,  48,  49,  // 40-49
+       50,  51,  52,  53,  54,  55,  56,  57,  58,  59,  // 50-59
+       60,  61,  62,  63,  64,  65,  66,  67,  68,  69,  // 60-69
+       70,  71,  72,  73,  74,  75,  76,  77,  78,  79,  // 70-79
+       80,  81,  82,  83,  84,  85,  86,  87,  88,  89,  // 80-89
+       90,  91,  92,  93,  94,  95,  96,  97,  98,  99,  // 90-99
+       100, 101, 102, 103, 104, 105, 106, 107, 108, 109, // 100-109
+       110, 111, 112, 113, 114, 115, 116, 117, 118, 119, // 110-119
+       120, 121, 122, 123, 124, 125, 126, 127, 128, 129, // 120-129
+       130, 131, 132, 133, 134, 135, 136, 137, 138, 139, // 130-139
+       140, 141, 142, 143, 144, 145, 146, 147, 148, 149, // 140-149
+       150, 151, 152, 153, 154, 155, 156, 157, 158, 159, // 150-159
+       160, 161, 162, 163, 164, 165, 166, 167, 168, 169, // 160-169
+       170, 171, 172, 173, 174, 175, 176, 177, 178, 179, // 170-179
+       180, 181, 182, 183, 184, 185, 186, 187, 188, 189, // 180-189
+       190, 191, 192, 193, 194, 195, 196, 197, 198, 199, // 190-199
+       200, 201, 202, 203, 204, 205, 206, 207, 208, 209, // 200-209
+       210, 211, 212, 213, 214, 215, 216, 217, 218, 219, // 210-219
+       220, 221, 222, 223, 224, 225, 226, 227, 228, 229, // 220-229
+       230, 231, 232, 233, 234, 235, 236, 237, 238, 239, // 230-239
+       240, 241, 242, 243, 244, 245, 246, 247, 248, 249, // 240-249
+       250, 251, 252, 253, 254, 255,                     // 250-255
+};
+
+size_t CoreExport irc::hash::operator()(const irc::string &s) const
 {
-       if (n)
-       {
-               for (char* t = n; *t; t++)
-                       *t = lowermap[(unsigned char)*t];
-       }
+       register size_t t = 0;
+       for (irc::string::const_iterator x = s.begin(); x != s.end(); ++x) /* ++x not x++, as its faster */
+               t = 5 * t + national_case_insensitive_map[(unsigned char)*x];
+       return t;
 }
 
-size_t nspace::hash<insp_inaddr>::operator()(const insp_inaddr &a) const
+bool irc::StrHashComp::operator()(const std::string& s1, const std::string& s2) const
 {
-       size_t q;
-       memcpy(&q,&a,sizeof(size_t));
-       return q;
+       const unsigned char* n1 = (const unsigned char*)s1.c_str();
+       const unsigned char* n2 = (const unsigned char*)s2.c_str();
+       for (; *n1 && *n2; n1++, n2++)
+               if (national_case_insensitive_map[*n1] != national_case_insensitive_map[*n2])
+                       return false;
+       return (national_case_insensitive_map[*n1] == national_case_insensitive_map[*n2]);
 }
 
-size_t nspace::hash<string>::operator()(const string &s) const
+size_t irc::insensitive::operator()(const std::string &s) const
 {
        /* XXX: NO DATA COPIES! :)
         * The hash function here is practically
         * a copy of the one in STL's hash_fun.h,
-        * only with *x replaced with lowermap[*x].
+        * only with *x replaced with national_case_insensitive_map[*x].
         * This avoids a copy to use hash<const char*>
         */
        register size_t t = 0;
        for (std::string::const_iterator x = s.begin(); x != s.end(); ++x) /* ++x not x++, as its faster */
-               t = 5 * t + lowermap[(unsigned char)*x];
+               t = 5 * t + national_case_insensitive_map[(unsigned char)*x];
        return t;
 }
 
-bool irc::StrHashComp::operator()(const std::string& s1, const std::string& s2) const
-{
-       unsigned char* n1 = (unsigned char*)s1.c_str();
-       unsigned char* n2 = (unsigned char*)s2.c_str();
-       for (; *n1 && *n2; n1++, n2++)
-               if (lowermap[*n1] != lowermap[*n2])
-                       return false;
-       return (lowermap[*n1] == lowermap[*n2]);
-}
-
-bool irc::InAddr_HashComp::operator()(const insp_inaddr &s1, const insp_inaddr &s2) const
-{
-#ifdef IPV6
-       for (int n = 0; n < 16; n++)
-               if (s2.s6_addr[n] != s1.s6_addr[n])
-                       return false;
-       return true;
-#else
-       return (s1.s_addr == s1.s_addr);
-#endif
-}
-
 /******************************************************
  *
  * This is the implementation of our special irc::string
@@ -121,83 +193,49 @@ bool irc::InAddr_HashComp::operator()(const insp_inaddr &s1, const insp_inaddr &
  * std::string which is not only case-insensitive but
  * can also do scandanavian comparisons, e.g. { = [, etc.
  *
- * This class depends on the const array 'lowermap'.
+ * This class depends on the const array 'national_case_insensitive_map'.
  *
  ******************************************************/
 
 bool irc::irc_char_traits::eq(char c1st, char c2nd)
 {
-       return lowermap[(unsigned char)c1st] == lowermap[(unsigned char)c2nd];
+       return national_case_insensitive_map[(unsigned char)c1st] == national_case_insensitive_map[(unsigned char)c2nd];
 }
 
 bool irc::irc_char_traits::ne(char c1st, char c2nd)
 {
-       return lowermap[(unsigned char)c1st] != lowermap[(unsigned char)c2nd];
+       return national_case_insensitive_map[(unsigned char)c1st] != national_case_insensitive_map[(unsigned char)c2nd];
 }
 
 bool irc::irc_char_traits::lt(char c1st, char c2nd)
 {
-       return lowermap[(unsigned char)c1st] < lowermap[(unsigned char)c2nd];
+       return national_case_insensitive_map[(unsigned char)c1st] < national_case_insensitive_map[(unsigned char)c2nd];
 }
 
 int irc::irc_char_traits::compare(const char* str1, const char* str2, size_t n)
 {
        for(unsigned int i = 0; i < n; i++)
        {
-               if(lowermap[(unsigned char)*str1] > lowermap[(unsigned char)*str2])
-                               return 1;
+               if(national_case_insensitive_map[(unsigned char)*str1] > national_case_insensitive_map[(unsigned char)*str2])
+                       return 1;
 
-               if(lowermap[(unsigned char)*str1] < lowermap[(unsigned char)*str2])
-                       return -1;
+               if(national_case_insensitive_map[(unsigned char)*str1] < national_case_insensitive_map[(unsigned char)*str2])
+                       return -1;
 
                if(*str1 == 0 || *str2 == 0)
-                       return 0;
+                       return 0;
 
-               str1++;
+               str1++;
                str2++;
        }
        return 0;
 }
 
-std::string operator+ (std::string& leftval, irc::string& rightval)
-{
-       return leftval + std::string(rightval.c_str());
-}
-
-irc::string operator+ (irc::string& leftval, std::string& rightval)
-{
-       return leftval + irc::string(rightval.c_str());
-}
-
-bool operator== (std::string& leftval, irc::string& rightval)
-{
-       return (leftval.c_str() == rightval);
-}
-
-bool operator== (irc::string& leftval, std::string& rightval)
-{
-       return (leftval == rightval.c_str());
-}
-
 const char* irc::irc_char_traits::find(const char* s1, int  n, char c)
 {
-       while(n-- > 0 && lowermap[(unsigned char)*s1] != lowermap[(unsigned char)c])
+       while(n-- > 0 && national_case_insensitive_map[(unsigned char)*s1] != national_case_insensitive_map[(unsigned char)c])
                s1++;
-       return s1;
-}
-
-/* See hashcomp.h if you care about these... */
-std::ostream& operator<<(std::ostream &os, const irc::string &str)
-{
-       return os << str.c_str();
-}
-
-std::istream& operator>>(std::istream &is, irc::string &str)
-{
-       std::string tmp;
-       is >> tmp;
-       str = tmp.c_str();
-       return is;
+       return (n >= 0) ? s1 : NULL;
 }
 
 irc::tokenstream::tokenstream(const std::string &source) : tokens(source), last_pushed(false)
@@ -211,7 +249,7 @@ irc::tokenstream::~tokenstream()
 {
 }
 
-const std::string irc::tokenstream::GetToken()
+bool irc::tokenstream::GetToken(std::string &token)
 {
        std::string::iterator lsp = last_starting_position;
 
@@ -229,7 +267,8 @@ const std::string irc::tokenstream::GetToken()
                         */
                        std::string::iterator curr = ++n;
                        n = tokens.end();
-                       return std::string(curr, tokens.end());
+                       token = std::string(curr, tokens.end());
+                       return true;
                }
 
                last_pushed = false;
@@ -239,18 +278,44 @@ const std::string irc::tokenstream::GetToken()
                        /* If we find a space, or end of string, this is the end of a token.
                         */
                        last_starting_position = n+1;
-                       last_pushed = true;
+                       last_pushed = *n == ' ';
 
                        std::string strip(lsp, n+1 == tokens.end() ? n+1  : n++);
                        while ((strip.length()) && (strip.find_last_of(' ') == strip.length() - 1))
                                strip.erase(strip.end() - 1);
 
-                       return strip;
+                       token = strip;
+                       return !token.empty();
                }
 
                n++;
        }
-       return "";
+       token.clear();
+       return false;
+}
+
+bool irc::tokenstream::GetToken(irc::string &token)
+{
+       std::string stdstring;
+       bool returnval = GetToken(stdstring);
+       token = assign(stdstring);
+       return returnval;
+}
+
+bool irc::tokenstream::GetToken(int &token)
+{
+       std::string tok;
+       bool returnval = GetToken(tok);
+       token = ConvToInt(tok);
+       return returnval;
+}
+
+bool irc::tokenstream::GetToken(long &token)
+{
+       std::string tok;
+       bool returnval = GetToken(tok);
+       token = ConvToInt(tok);
+       return returnval;
 }
 
 irc::sepstream::sepstream(const std::string &source, char seperator) : tokens(source), sep(seperator)
@@ -259,7 +324,7 @@ irc::sepstream::sepstream(const std::string &source, char seperator) : tokens(so
        n = tokens.begin();
 }
 
-const std::string irc::sepstream::GetToken()
+bool irc::sepstream::GetToken(std::string &token)
 {
        std::string::iterator lsp = last_starting_position;
 
@@ -268,57 +333,38 @@ const std::string irc::sepstream::GetToken()
                if ((*n == sep) || (n+1 == tokens.end()))
                {
                        last_starting_position = n+1;
-                       std::string strip = std::string(lsp, n+1 == tokens.end() ? n+1  : n++);
+                       token = std::string(lsp, n+1 == tokens.end() ? n+1  : n++);
 
-                       while ((strip.length()) && (strip.find_last_of(sep) == strip.length() - 1))
-                               strip.erase(strip.end() - 1);
+                       while ((token.length()) && (token.find_last_of(sep) == token.length() - 1))
+                               token.erase(token.end() - 1);
 
-                       return strip;
+                       if (token.empty())
+                               n++;
+
+                       return n == tokens.end() ? false : true;
                }
 
                n++;
        }
 
-       return "";
+       token.clear();
+       return false;
 }
 
-irc::sepstream::~sepstream()
+const std::string irc::sepstream::GetRemaining()
 {
+       return std::string(n, tokens.end());
 }
 
-std::string irc::hex(const unsigned char *raw, size_t rawsz)
+bool irc::sepstream::StreamEnd()
 {
-       if (!rawsz)
-               return "";
-
-       /* EWW! This used to be using sprintf, which is WAY inefficient. -Special */
-       
-       const char *hex = "0123456789abcdef";
-       
-       char buf[rawsz*2+1];
-
-       size_t i, j;
-       for (i = 0, j = 0; j < rawsz; ++j)
-       {
-               buf[i++] = hex[raw[j] / 16];
-               buf[i++] = hex[raw[j] % 16];
-       }
-       buf[i] = '\0';
-
-       return buf;
+       return ((n + 1) == tokens.end());
 }
 
-const char* irc::Spacify(char* n)
+irc::sepstream::~sepstream()
 {
-       static char x[MAXBUF];
-       strlcpy(x,n,MAXBUF);
-       for (char* y = x; *y; y++)
-               if (*y == '_')
-                       *y = ' ';
-       return x;
 }
 
-
 irc::modestacker::modestacker(bool add) : adding(add)
 {
        sequence.clear();
@@ -346,74 +392,70 @@ void irc::modestacker::PushMinus()
        this->Push('-',"");
 }
 
-int irc::modestacker::GetStackedLine(std::deque<std::string> &result)
+int irc::modestacker::GetStackedLine(std::vector<std::string> &result, int max_line_size)
 {
-       int n = 0;
-       result.clear();
-       result.push_back(adding ? "+" : "-");
+       if (sequence.empty())
+       {
+               return 0;
+       }
+
+       unsigned int n = 0;
+       int size = 1; /* Account for initial +/- char */
+       int nextsize = 0;
+       int start = result.size();
+       std::string modeline = adding ? "+" : "-";
+       result.push_back(modeline);
 
-       while (!sequence[0].empty() && (sequence.size() > 1) && (result.size() < MAXMODES+1))
+       if (sequence.size() > 1)
+               nextsize = sequence[1].length() + 2;
+
+       while (!sequence[0].empty() && (sequence.size() > 1) && (n < ServerInstance->Config->Limits.MaxModes) && ((size + nextsize) < max_line_size))
        {
-               result[0] += *(sequence[0].begin());
+               modeline += *(sequence[0].begin());
                if (!sequence[1].empty())
+               {
                        result.push_back(sequence[1]);
+                       size += nextsize; /* Account for mode character and whitespace */
+               }
                sequence[0].erase(sequence[0].begin());
                sequence.erase(sequence.begin() + 1);
+
+               if (sequence.size() > 1)
+                       nextsize = sequence[1].length() + 2;
+
                n++;
        }
+       result[start] = modeline;
 
        return n;
 }
 
-irc::stringjoiner::stringjoiner(const std::string &seperator, const std::vector<std::string> &sequence, int begin, int end)
+irc::stringjoiner::stringjoiner(const std::string& seperator, const std::vector<std::string>& sequence, unsigned int begin, unsigned int end)
 {
-       for (int v = begin; v < end; v++)
-               joined.append(sequence[v]).append(seperator);
-       joined.append(sequence[end]);
-}
+       if (end < begin)
+               return; // nothing to do here
 
-irc::stringjoiner::stringjoiner(const std::string &seperator, const std::deque<std::string> &sequence, int begin, int end)
-{
-       for (int v = begin; v < end; v++)
+       for (unsigned int v = begin; v < end; v++)
                joined.append(sequence[v]).append(seperator);
        joined.append(sequence[end]);
 }
 
-irc::stringjoiner::stringjoiner(const std::string &seperator, const char** sequence, int begin, int end)
-{
-       for (int v = begin; v < end; v++)
-               joined.append(sequence[v]).append(seperator);
-       joined.append(sequence[end]);
-}
-
-std::string& irc::stringjoiner::GetJoined()
+const std::string& irc::stringjoiner::GetJoined() const
 {
        return joined;
 }
 
-irc::portparser::portparser(const std::string &source, bool allow_overlapped) : in_range(0), range_begin(0), range_end(0), overlapped(allow_overlapped)
-{
-       sep = new irc::commasepstream(source);
-       overlap_set.clear();
-}
-
-irc::portparser::~portparser()
+irc::portparser::portparser(const std::string &source, bool allow_overlapped)
+       : sep(source), in_range(0), range_begin(0), range_end(0), overlapped(allow_overlapped)
 {
-       delete sep;
 }
 
 bool irc::portparser::Overlaps(long val)
 {
-       if (!overlapped)
+       if (overlapped)
                return false;
 
-       if (overlap_set.find(val) == overlap_set.end())
-       {
-               overlap_set[val] = true;
-               return false;
-       }
-       else
-               return true;
+       return (!overlap_set.insert(val).second);
 }
 
 long irc::portparser::GetToken()
@@ -431,7 +473,7 @@ long irc::portparser::GetToken()
                        {
                                while (((Overlaps(in_range)) && (in_range <= range_end)))
                                        in_range++;
-                               
+
                                if (in_range <= range_end)
                                        return in_range;
                        }
@@ -440,16 +482,15 @@ long irc::portparser::GetToken()
                        in_range = 0;
        }
 
-       std::string x = sep->GetToken();
+       std::string x;
+       sep.GetToken(x);
 
-       if (x == "")
+       if (x.empty())
                return 0;
 
        while (Overlaps(atoi(x.c_str())))
        {
-               x = sep->GetToken();
-
-               if (x == "")
+               if (!sep.GetToken(x))
                        return 0;
        }
 
@@ -477,130 +518,3 @@ long irc::portparser::GetToken()
                return atoi(x.c_str());
        }
 }
-
-irc::dynamicbitmask::dynamicbitmask() : bits_size(4)
-{
-       /* We start with 4 bytes allocated which is room
-        * for 4 items. Something makes me doubt its worth
-        * allocating less than 4 bytes.
-        */
-       bits = new unsigned char[bits_size];
-       memset(bits, 0, bits_size);
-}
-
-irc::dynamicbitmask::~dynamicbitmask()
-{
-       /* Tidy up the entire used memory on delete */
-       delete[] bits;
-}
-                         
-irc::bitfield irc::dynamicbitmask::Allocate()
-{
-       /* Yeah, this isnt too efficient, however a module or the core
-        * should only be allocating bitfields on load, the Toggle and
-        * Get methods are O(1) as these are called much more often.
-        */
-       unsigned char* freebits = this->GetFreeBits();
-       for (unsigned char i = 0; i < bits_size; i++)
-       {
-               /* Yes, this is right. You'll notice we terminate the  loop when !current_pos,
-                * this is because we logic shift our bit off the end of unsigned char, and its
-                * lost, making the loop counter 0 when we're done.
-                */
-               for (unsigned char current_pos = 1; current_pos; current_pos = current_pos << 1)
-               {
-                       if (!(freebits[i] & current_pos))
-                       {
-                               freebits[i] |= current_pos;
-                               return std::make_pair(i, current_pos);
-                       }
-               }
-       }
-       /* We dont have any free space left, increase by one */
-
-       if (bits_size == 255)
-               /* Oh dear, cant grow it any further */
-               throw std::bad_alloc();
-
-       unsigned char old_bits_size = bits_size;
-       bits_size++;
-       /* Allocate new bitfield space */
-       unsigned char* temp_bits = new unsigned char[bits_size];
-       unsigned char* temp_freebits = new unsigned char[bits_size];
-       /* Copy the old data in */
-       memcpy(temp_bits, bits, old_bits_size);
-       memcpy(temp_freebits, freebits, old_bits_size);
-       /* Delete the old data pointers */
-       delete[] bits;
-       delete[] freebits;
-       /* Swap the pointers over so now the new 
-        * pointers point to our member values
-        */
-       bits = temp_bits;
-       freebits = temp_freebits;
-       this->SetFreeBits(freebits);
-       /* Initialize the new byte on the end of
-        * the bitfields, pre-allocate the one bit
-        * for this allocation
-        */
-       bits[old_bits_size] = 0;
-       freebits[old_bits_size] = 1;
-       /* We already know where we just allocated
-        * the bitfield, so no loop needed
-        */
-       return std::make_pair(old_bits_size, 1);
-}
-
-bool irc::dynamicbitmask::Deallocate(irc::bitfield &pos)
-{
-       /* We dont bother to shrink the bitfield
-        * on deallocation, the most we could do
-        * is save one byte (!) and this would cost
-        * us a loop (ugly O(n) stuff) so we just
-        * clear the bit and leave the memory
-        * claimed -- nobody will care about one
-        * byte.
-        */
-       if (pos.first < bits_size)
-       {
-               this->GetFreeBits()[pos.first] &= ~pos.second;
-               return true;
-       }
-       /* They gave a bitfield outside of the
-        * length of our array. BAD programmer.
-        */
-       return false;
-}
-
-void irc::dynamicbitmask::Toggle(irc::bitfield &pos, bool state)
-{
-       /* Range check the value */
-       if (pos.first < bits_size)
-       {
-               if (state)
-                       /* Set state, OR the state in */
-                       bits[pos.first] |= pos.second;
-               else
-                       /* Clear state, AND the !state out */
-                       bits[pos.first] &= ~pos.second;
-       }
-}
-
-bool irc::dynamicbitmask::Get(irc::bitfield &pos)
-{
-       /* Range check the value */
-       if (pos.first < bits_size)
-               return (bits[pos.first] & pos.second);
-       else
-               /* We can't return false, otherwise we can't
-                * distinguish between failure and a cleared bit!
-                * Our only sensible choice is to throw (ew).
-                */
-               throw ModuleException("irc::dynamicbitmask::Get(): Invalid bitfield, out of range");
-}
-
-unsigned char irc::dynamicbitmask::GetSize()
-{
-       return bits_size;
-}
-