Revert to string but return a reference, should be about the same speed

[user/henk/code/inspircd.git] / include / cull_list.h

diff --git a/include/cull_list.h b/include/cull_list.h
index 67ffcaf967254614af8b6dbfdee223ca37b4b953..683e4493baf97e84808375cf62f10e8c537118de 100644 (file)
--- a/include/cull_list.h
+++ b/include/cull_list.h
@@ -2,7 +2,7 @@
  *       | Inspire Internet Relay Chat Daemon |
  *       +------------------------------------+
  *
- *  Inspire is copyright (C) 2002-2005 ChatSpike-Dev.
+ *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
  *                       E-mail:
  *                <brain@chatspike.net>
  *                <Craig@chatspike.net>
@@ -42,7 +42,7 @@ class CullItem
         userrec* user;
        /** Holds the quit reason to use for this user.
         */
-        std::string reason;
+       char* reason;
  public:
        /** Constrcutor.
         * Initializes the CullItem with a user pointer
@@ -50,13 +50,17 @@ class CullItem
         * @param u The user to add
         * @param r The quit reason of the added user
         */
-        CullItem(userrec* u, std::string r);
+        CullItem(userrec* u, std::string &r);
+       CullItem(userrec* u, const char* r);
+
+       ~CullItem();
+
        /** Returns a pointer to the user
         */
         userrec* GetUser();
        /** Returns the user's quit reason
         */
-        std::string GetReason();
+       std::string& GetReason();
 };
 
 /** The CullList class can be used by modules, and is used
@@ -82,9 +86,17 @@ class CullList
          std::vector<CullItem> list;
         /** A list of users who have already been
          * placed on the list, as a map for fast
-         * reference.
+         * reference. When deleting an item, the
+         * time_t value stored here must match
+         * the one of the actual userrec, otherwise
+         * we don't delete it (its a different user)
+         */
+        std::map<userrec*,time_t> exempt;
+        
+        /** Check if a user pointer is valid
+         * (e.g. it exists in the user hash)
          */
-        std::map<userrec*,int> exempt;
+        bool IsValid(userrec* user);
  public:
         /** Constructor.
          * Clears the CullList::list and CullList::exempt
@@ -96,7 +108,8 @@ class CullList
          * @param user The user to add
          * @param reason The quit reason of the user being added
          */
-         void AddItem(userrec* user, std::string reason);
+         void AddItem(userrec* user, std::string &reason);
+        void AddItem(userrec* user, const char* reason);
         /** Applies the cull list, quitting all the users
          * on the list with their quit reasons all at once.
          * This is a very fast operation compared to