]> git.netwichtig.de Git - user/henk/code/inspircd.git/blobdiff - include/modules.h
Remove the -Dssize_t declaration now its typedeffed.
[user/henk/code/inspircd.git] / include / modules.h
index 06c0f1dd54e64694d15744950ec520014616babe..972dabb43042c06964fd262713980825def5a29d 100644 (file)
@@ -30,11 +30,13 @@ class XLine;
 /** Used to define a set of behavior bits for a module
  */
 enum ModuleFlags {
+       VF_NONE = 0,            // module is not special at all
        VF_STATIC = 1,          // module is static, cannot be /unloadmodule'd
        VF_VENDOR = 2,          // module is a vendor module (came in the original tarball, not 3rd party)
        VF_SERVICEPROVIDER = 4, // module provides a service to other modules (can be a dependency)
        VF_COMMON = 8,          // module needs to be common on all servers in a network to link
-       VF_OPTCOMMON = 16       // module should be common on all servers for unsurprising behavior
+       VF_OPTCOMMON = 16,      // module should be common on all servers for unsurprising behavior
+       VF_CORE = 32            // module is a core command, can be assumed loaded on all servers
 };
 
 /** Used with SendToMode()
@@ -102,8 +104,8 @@ struct ModResult {
        }
 };
 
-/** If you change the module API, change this value. */
-#define API_VERSION 13000
+/** If you change the module API in any way, increment this value. */
+#define API_VERSION 132
 
 class ServerConfig;
 
@@ -156,40 +158,16 @@ typedef std::map<std::string, std::pair<int, modulelist> > interfacelist;
        } \
 } while (0);
 
-/**
- * This #define allows us to call a method in all
- * loaded modules in a readable simple way and pass
- * an instance pointer to the macro. e.g.:
- * 'FOREACH_MOD_I(Instance, OnConnect, OnConnect(user));'
- */
-#define FOREACH_MOD_I(z,y,x) do { \
-       EventHandlerIter safei; \
-       for (EventHandlerIter _i = z->Modules->EventHandlers[y].begin(); _i != z->Modules->EventHandlers[y].end(); ) \
-       { \
-               safei = _i; \
-               ++safei; \
-               try \
-               { \
-                       (*_i)->x ; \
-               } \
-               catch (CoreException& modexcept) \
-               { \
-                       z->Logs->Log("MODULE",DEFAULT,"Exception caught: %s",modexcept.GetReason()); \
-               } \
-               _i = safei; \
-       } \
-} while (0);
-
 /**
  * Custom module result handling loop. This is a paired macro, and should only
  * be used with while_each_hook.
  *
  * See src/channels.cpp for an example of use.
  */
-#define DO_EACH_HOOK(z,n,v,args) \
+#define DO_EACH_HOOK(n,v,args) \
 do { \
-       EventHandlerIter iter_ ## n = z->Modules->EventHandlers[I_ ## n].begin(); \
-       while (iter_ ## n != z->Modules->EventHandlers[I_ ## n].end()) \
+       EventHandlerIter iter_ ## n = ServerInstance->Modules->EventHandlers[I_ ## n].begin(); \
+       while (iter_ ## n != ServerInstance->Modules->EventHandlers[I_ ## n].end()) \
        { \
                Module* mod_ ## n = *iter_ ## n; \
                iter_ ## n ++; \
@@ -197,11 +175,11 @@ do { \
                { \
                        v = (mod_ ## n)->n args;
 
-#define WHILE_EACH_HOOK(z,n) \
+#define WHILE_EACH_HOOK(n) \
                } \
                catch (CoreException& except_ ## n) \
                { \
-                       z->Logs->Log("MODULE",DEFAULT,"Exception caught: %s", (except_ ## n).GetReason()); \
+                       ServerInstance->Logs->Log("MODULE",DEFAULT,"Exception caught: %s", (except_ ## n).GetReason()); \
                        (void) mod_ ## n; /* catch mismatched pairs */ \
                } \
        } \
@@ -212,16 +190,16 @@ do { \
  * Runs the given hook until some module returns a useful result.
  *
  * Example: ModResult result;
- * FIRST_MOD_RESULT(ServerInstance, OnUserPreNick, result, (user, newnick))
+ * FIRST_MOD_RESULT(OnUserPreNick, result, (user, newnick))
  */
-#define FIRST_MOD_RESULT(z,n,v,args) do { \
+#define FIRST_MOD_RESULT(n,v,args) do { \
        v = MOD_RES_PASSTHRU; \
-       DO_EACH_HOOK(z,n,v,args) \
+       DO_EACH_HOOK(n,v,args) \
        { \
                if (v != MOD_RES_PASSTHRU) \
                        break; \
        } \
-       WHILE_EACH_HOOK(z,n); \
+       WHILE_EACH_HOOK(n); \
 } while (0)
 
 /** Represents a non-local user.
@@ -239,7 +217,7 @@ do { \
 /** Is a remote user */
 #define IS_REMOTE(x) (x->GetFd() < 0)
 /** Is a fake user */
-#define IS_FAKE(x) (x->GetFd() == FD_FAKEUSER_NUMBER)
+#define IS_SERVER(x) (x->GetFd() == FD_FAKEUSER_NUMBER)
 /** Is a module created user */
 #define IS_MODULE_CREATED(x) (x->GetFd() == FD_MAGIC_NUMBER)
 /** Is an oper */
@@ -248,100 +226,66 @@ do { \
 #define IS_AWAY(x) (!x->awaymsg.empty())
 
 /** Holds a module's Version information.
- *  The four members (set by the constructor only) indicate details as to the version number
- *  of a module. A class of type Version is returned by the GetVersion method of the Module class.
- *  The flags and API values represent the module flags and API version of the module.
- *  The API version of a module must match the API version of the core exactly for the module to
- *  load successfully.
+ * The members (set by the constructor only) indicate details as to the version number
+ * of a module. A class of type Version is returned by the GetVersion method of the Module class.
+ *
+ * The core provides only one implementation of the template, causing a run-time linking
+ * error when attempting to load a module compiled against a different API_VERSION.
  */
-class CoreExport Version : public classbase
+template<int api>
+class CoreExport VersionBase : public classbase
 {
  public:
+       /** Module description
+        */
+       const std::string description;
        /** Version information.
         */
        const std::string version;
 
-       /** Flags and API version
+       /** Flags
         */
-       const int Flags, API;
+       const int Flags;
 
        /** Initialize version class
         */
-       Version(const std::string &customver, int flags,
-               int api_ver = API_VERSION, const std::string& src_rev = VERSION " r" REVISION);
+       VersionBase(const std::string &desc, int flags = VF_NONE, int dummy = 0, const std::string& src_rev = VERSION " r" REVISION);
 };
 
-/** The ModuleMessage class is the base class of Request and Event
- * This class is used to represent a basic data structure which is passed
- * between modules for safe inter-module communications.
- */
-class CoreExport ModuleMessage : public Extensible
-{
- public:
-       /** Destructor
-        */
-       virtual ~ModuleMessage() {};
-};
+typedef VersionBase<API_VERSION> Version;
 
 /** The Request class is a unicast message directed at a given module.
  * When this class is properly instantiated it may be sent to a module
  * using the Send() method, which will call the given module's OnRequest
  * method with this class as its parameter.
  */
-class CoreExport Request : public ModuleMessage
+class CoreExport Request : public classbase
 {
- protected:
-       /** This member holds a pointer to arbitary data set by the emitter of the message
-        */
-       char* data;
+ public:
        /** This should be a null-terminated string identifying the type of request,
         * all modules should define this and use it to determine the nature of the
         * request before they attempt to cast the Request in any way.
         */
-       const char* id;
+       const char* const id;
        /** This is a pointer to the sender of the message, which can be used to
         * directly trigger events, or to create a reply.
         */
-       Module* source;
+       Module* const source;
        /** The single destination of the Request
         */
-       Module* dest;
- public:
-       /** Create a new Request
-        * This is for the 'old' way of casting whatever the data is
-        * to char* and hoping you get the right thing at the other end.
-        * This is slowly being depreciated in favor of the 'new' way.
-        */
-       Request(char* anydata, Module* src, Module* dst);
+       Module* const dest;
+
        /** Create a new Request
         * This is for the 'new' way of defining a subclass
         * of Request and defining it in a common header,
         * passing an object of your Request subclass through
         * as a Request* and using the ID string to determine
-        * what to cast it back to and the other end. This is
-        * much safer as there are no casts not confirmed by
-        * the ID string, and all casts are child->parent and
-        * can be checked at runtime with dynamic_cast<>()
+        * what to cast it back to and the other end.
         */
        Request(Module* src, Module* dst, const char* idstr);
-       /** Fetch the Request data
-        */
-       char* GetData();
-       /** Fetch the ID string
-        */
-       const char* GetId();
-       /** Fetch the request source
-        */
-       Module* GetSource();
-       /** Fetch the request destination (should be 'this' in the receiving module)
-        */
-       Module* GetDest();
        /** Send the Request.
-        * Upon returning the result will be arbitary data returned by the module you
-        * sent the request to. It is up to your module to know what this data is and
-        * how to deal with it.
         */
-       const char* Send();
+       void Send();
 };
 
 
@@ -350,40 +294,27 @@ class CoreExport Request : public ModuleMessage
  * using the Send() method, which will trigger the OnEvent method in
  * all modules passing the object as its parameter.
  */
-class CoreExport Event : public ModuleMessage
+class CoreExport Event : public classbase
 {
- protected:
-       /** This member holds a pointer to arbitary data set by the emitter of the message
-        */
-       char* data;
+ public:
        /** This is a pointer to the sender of the message, which can be used to
         * directly trigger events, or to create a reply.
         */
-       Module* source;
+       Module* const source;
        /** The event identifier.
         * This is arbitary text which should be used to distinguish
         * one type of event from another.
         */
-       std::string id;
- public:
+       const std::string id;
+
        /** Create a new Event
         */
-       Event(char* anydata, Module* src, const std::string &eventid);
-       /** Get the Event data
-        */
-       char* GetData();
-       /** Get the event Source
-        */
-       Module* GetSource();
-       /** Get the event ID.
-        * Use this to determine the event type for safe casting of the data
-        */
-       std::string GetEventID();
+       Event(Module* src, const std::string &eventid);
        /** Send the Event.
         * The return result of an Event::Send() will always be NULL as
         * no replies are expected.
         */
-       char* Send(InspIRCd* ServerInstance);
+       void Send();
 };
 
 /** Priority types which can be returned from Module::Prioritize()
@@ -400,18 +331,17 @@ enum Implementation
        I_OnUserPreInvite, I_OnUserInvite, I_OnUserPreMessage, I_OnUserPreNotice, I_OnUserPreNick,
        I_OnUserMessage, I_OnUserNotice, I_OnMode, I_OnGetServerDescription, I_OnSyncUser,
        I_OnSyncChannel, I_OnDecodeMetaData, I_OnWallops,
-       I_OnChangeHost, I_OnChangeName, I_OnAddLine, I_OnDelLine, I_OnExpireLine, I_OnCleanup,
+       I_OnChangeHost, I_OnChangeName, I_OnAddLine, I_OnDelLine, I_OnExpireLine,
        I_OnUserPostNick, I_OnPreMode, I_On005Numeric, I_OnKill, I_OnRemoteKill, I_OnLoadModule,
        I_OnUnloadModule, I_OnBackgroundTimer, I_OnPreCommand, I_OnCheckReady, I_OnCheckInvite,
-       I_OnRawMode, I_OnCheckKey, I_OnCheckLimit, I_OnCheckBan, I_OnCheckExtBan, I_OnCheckStringExtBan,
+       I_OnRawMode, I_OnCheckKey, I_OnCheckLimit, I_OnCheckBan, I_OnCheckChannelBan, I_OnExtBanCheck,
        I_OnStats, I_OnChangeLocalUserHost, I_OnChangeLocalUserGecos, I_OnPreTopicChange,
-       I_OnPostTopicChange, I_OnEvent, I_OnRequest, I_OnGlobalOper, I_OnPostConnect, I_OnAddBan,
-       I_OnDelBan, I_OnRawSocketAccept, I_OnRawSocketClose, I_OnRawSocketWrite, I_OnRawSocketRead,
-       I_OnChangeLocalUserGECOS, I_OnUserRegister, I_OnChannelPreDelete, I_OnChannelDelete,
+       I_OnPostTopicChange, I_OnEvent, I_OnGlobalOper, I_OnPostConnect, I_OnAddBan,
+       I_OnDelBan, I_OnChangeLocalUserGECOS, I_OnUserRegister, I_OnChannelPreDelete, I_OnChannelDelete,
        I_OnPostOper, I_OnSyncNetwork, I_OnSetAway, I_OnUserList, I_OnPostCommand, I_OnPostJoin,
-       I_OnWhoisLine, I_OnBuildExemptList, I_OnRawSocketConnect, I_OnGarbageCollect, I_OnBufferFlushed,
+       I_OnWhoisLine, I_OnBuildNeighborList, I_OnGarbageCollect,
        I_OnText, I_OnPassCompare, I_OnRunTestSuite, I_OnNamesListItem, I_OnNumeric, I_OnHookIO,
-       I_OnHostCycle, I_OnPreRehash, I_OnModuleRehash, I_OnSendWhoLine, I_OnChangeIdent,
+       I_OnPreRehash, I_OnModuleRehash, I_OnSendWhoLine, I_OnChangeIdent,
        I_END
 };
 
@@ -428,13 +358,21 @@ class CoreExport Module : public Extensible
        /** File that this module was loaded from
         */
        std::string ModuleSourceFile;
+       /** Reference to the dlopen() value
+        */
+       DLLFactory* ModuleDLLFactory;
 
        /** Default constructor.
         * Creates a module class.
         * @param Me An instance of the InspIRCd class which will be saved into ServerInstance for your use
         * \exception ModuleException Throwing this class, or any class derived from ModuleException, causes loading of the module to abort.
         */
-       Module(InspIRCd* Me);
+       Module();
+
+       /** Clean up prior to destruction
+        * If you override, you must call this AFTER your module's cleanup
+        */
+       virtual bool cull();
 
        /** Default destructor.
         * destroys a module class
@@ -449,7 +387,7 @@ class CoreExport Module : public Extensible
         * The method should return a Version object with its version information assigned via
         * Version::Version
         */
-       virtual Version GetVersion();
+       virtual Version GetVersion() = 0;
 
        /** Called when a user connects.
         * The details of the connecting user are available to you in the parameter User *user
@@ -693,16 +631,16 @@ class CoreExport Module : public Extensible
         */
        virtual ModResult OnUserPreNotice(User* user,void* dest,int target_type, std::string &text,char status, CUList &exempt_list);
 
-       /** Called whenever the server wants to build the exemption list for a channel, but is not directly doing a PRIVMSG or NOTICE.
-        * For example, the spanningtree protocol will call this event when passing a privmsg on (but not processing it directly).
-        * @param message_type The message type, either MSG_PRIVMSG or MSG_NOTICE
-        * @param chan The channel to build the exempt list of
-        * @param sender The original sender of the PRIVMSG or NOTICE
-        * @param status The status char to be used for the channel list
-        * @param exempt_list The exempt list to be populated
-        * @param text The original message text causing the exempt list to be built
+       /** Called when sending a message to all "neighbors" of a given user -
+        * that is, all users that share a common channel. This is used in
+        * commands such as NICK, QUIT, etc.
+        * @param source The source of the message
+        * @param include_c Channels to scan for users to include
+        * @param exceptions Map of user->bool that overrides the inclusion decision
+        *
+        * Set exceptions[user] = true to include, exceptions[user] = false to exclude
         */
-       virtual void OnBuildExemptList(MessageType message_type, Channel* chan, User* sender, char status, CUList &exempt_list, const std::string &text);
+       virtual void OnBuildNeighborList(User* source, UserChanList &include_c, std::map<User*,bool> &exceptions);
 
        /** Called before any nickchange, local or remote. This can be used to implement Q-lines etc.
         * Please note that although you can see remote nickchanges through this function, you should
@@ -1105,31 +1043,30 @@ class CoreExport Module : public Extensible
         */
        virtual ModResult OnCheckLimit(User* user, Channel* chan);
 
-       /** Called whenever a user joins a channel, to determine if banlist checks should go ahead or not.
-        * This method will always be called for each join, wether or not the user actually matches a channel ban, and
-        * determines the outcome of an if statement around the whole section of ban checking code.
-        * return 1 to explicitly allow the join to go ahead or 0 to ignore the event.
-        * @param user The user joining the channel
-        * @param chan The channel being joined
-        * @return 1 to explicitly allow the join, 0 to proceed as normal. Return -1 to explicitly deny the
-        * join to the channel.
+       /**
+        * Checks for a user's ban from the channel
+        * @param user The user to check
+        * @param chan The channel to check in
+        * @return MOD_RES_DENY to mark as banned, MOD_RES_ALLOW to skip the
+        * ban check, or MOD_RES_PASSTHRU to check bans normally
         */
-       virtual ModResult OnCheckBan(User* user, Channel* chan);
+       virtual ModResult OnCheckChannelBan(User* user, Channel* chan);
 
-       /* Called whenever checking whether or not a user is matched by an applicable extended bantype.
-        * NOTE: may also trigger extra OnCheckStringExtBan events!
-        * @param u The user to check
-        * @param c The channel the user is on
-        * @param type The type of extended ban to check for.
-        * @returns 1 = exempt, 0 = no match, -1 = banned
+       /**
+        * Checks for a user's match of a single ban
+        * @param user The user to check for match
+        * @param chan The channel on which the match is being checked
+        * @param mask The mask being checked
+        * @return MOD_RES_DENY to mark as banned, MOD_RES_ALLOW to skip the
+        * ban check, or MOD_RES_PASSTHRU to check bans normally
         */
-       virtual ModResult OnCheckExtBan(User *u, Channel *c, char type);
+       virtual ModResult OnCheckBan(User* user, Channel* chan, const std::string& mask);
 
-       /** Called whenever checking whether or not a string is extbanned. NOTE: one OnCheckExtBan will also trigger a number of
-        * OnCheckStringExtBan events for seperate host/IP comnbinations.
-        * @returns 1 = exempt, 0 = no match, -1 = banned
+       /** Checks for a match on a given extban type
+        * @return MOD_RES_DENY to mark as banned, MOD_RES_ALLOW to skip the
+        * ban check, or MOD_RES_PASSTHRU to check bans normally
         */
-       virtual ModResult OnCheckStringExtBan(const std::string &s, Channel *c, char type);
+       virtual ModResult OnExtBanCheck(User* user, Channel* chan, char type);
 
        /** Called on all /STATS commands
         * This method is triggered for all /STATS use, including stats symbols handled by the core.
@@ -1176,22 +1113,17 @@ class CoreExport Module : public Extensible
         */
        virtual void OnPostTopicChange(User* user, Channel* chan, const std::string &topic);
 
-       /** Called whenever an Event class is sent to all module by another module.
-        * Please see the documentation of Event::Send() for further information. The Event sent can
-        * always be assumed to be non-NULL, you should *always* check the value of Event::GetEventID()
-        * before doing anything to the event data, and you should *not* change the event data in any way!
+       /** Called whenever an Event class is sent to all modules by another module.
+        * You should *always* check the value of Event::id to determine the event type.
         * @param event The Event class being received
         */
-       virtual void OnEvent(Event* event);
+       virtual void OnEvent(Event& event);
 
        /** Called whenever a Request class is sent to your module by another module.
-        * Please see the documentation of Request::Send() for further information. The Request sent
-        * can always be assumed to be non-NULL, you should not change the request object or its data.
-        * Your method may return arbitary data in the char* result which the requesting module
-        * may be able to use for pre-determined purposes (e.g. the results of an SQL query, etc).
+        * The value of Request::id should be used to determine the type of request.
         * @param request The Request class being received
         */
-       virtual const char* OnRequest(Request* request);
+       virtual void OnRequest(Request& request);
 
        /** Called whenever a password check is to be made. Replaces the old OldOperCompare API.
         * The password field (from the config file) is in 'password' and is to be compared against
@@ -1243,9 +1175,9 @@ class CoreExport Module : public Extensible
         * @param user The item to possibly install the I/O hook on
         * @param via The port that <user> connected on
         */
-       virtual void OnHookIO(EventHandler* user, ListenSocketBase* via);
+       virtual void OnHookIO(StreamSocket*, ListenSocketBase* via);
 
-       /** Called immediately after any  connection is accepted. This is intended for raw socket
+       /** Called immediately after any connection is accepted. This is intended for raw socket
         * processing (e.g. modules which wrap the tcp connection within another library) and provides
         * no information relating to a user record as the connection has not been assigned yet.
         * There are no return values from this call as all modules get an opportunity if required to
@@ -1255,48 +1187,38 @@ class CoreExport Module : public Extensible
         * @param server The server IP address and port
         * @param localport The local port number the user connected to
         */
-       virtual void OnRawSocketAccept(int fd, irc::sockets::sockaddrs* client, irc::sockets::sockaddrs* server);
+       virtual void OnStreamSocketAccept(StreamSocket*, irc::sockets::sockaddrs* client, irc::sockets::sockaddrs* server);
 
-       /** Called immediately before any write() operation on a user's socket in the core. Because
-        * this event is a low level event no user information is associated with it. It is intended
-        * for use by modules which may wrap connections within another API such as SSL for example.
-        * return a non-zero result if you have handled the write operation, in which case the core
-        * will not call write().
-        * @param fd The file descriptor of the socket
-        * @param buffer A char* buffer being written
-        * @param Number of characters to write
-        * @return Number of characters actually written or 0 if you didn't handle the operation
+       /**
+        * Called when a hooked stream has data to write, or when the socket
+        * engine returns it as writable
+        * @param socket The socket in question
+        * @param sendq Data to send to the socket
+        * @return 1 if the sendq has been completely emptied, 0 if there is
+        *  still data to send, and -1 if there was an error
         */
-       virtual int OnRawSocketWrite(int fd, const char* buffer, int count);
+       virtual int OnStreamSocketWrite(StreamSocket*, std::string& sendq);
 
        /** Called immediately before any socket is closed. When this event is called, shutdown()
         * has not yet been called on the socket.
         * @param fd The file descriptor of the socket prior to close()
         */
-       virtual void OnRawSocketClose(int fd);
+       virtual void OnStreamSocketClose(StreamSocket*);
 
        /** Called immediately upon connection of an outbound BufferedSocket which has been hooked
         * by a module.
         * @param fd The file descriptor of the socket immediately after connect()
         */
-       virtual void OnRawSocketConnect(int fd);
-
-       /** Called immediately before any read() operation on a client socket in the core.
-        * This occurs AFTER the select() or poll() so there is always data waiting to be read
-        * when this event occurs.
-        * Your event should return 1 if it has handled the reading itself, which prevents the core
-        * just using read(). You should place any data read into buffer, up to but NOT GREATER THAN
-        * the value of count. The value of readresult must be identical to an actual result that might
-        * be returned from the read() system call, for example, number of bytes read upon success,
-        * 0 upon EOF or closed socket, and -1 for error. If your function returns a nonzero value,
-        * you MUST set readresult.
-        * @param fd The file descriptor of the socket
-        * @param buffer A char* buffer being read to
-        * @param count The size of the buffer
-        * @param readresult The amount of characters read, or 0
-        * @return nonzero if the event was handled, in which case readresult must be valid on exit
-        */
-       virtual int OnRawSocketRead(int fd, char* buffer, unsigned int count, int &readresult);
+       virtual void OnStreamSocketConnect(StreamSocket*);
+
+       /**
+        * Called when the stream socket has data to read
+        * @param socket The socket that is ready
+        * @param recvq The receive queue that new data should be appended to
+        * @return 1 if new data has been read, 0 if no new data is ready (but the
+        *  socket is still connected), -1 if there was an error or close
+        */
+       virtual int OnStreamSocketRead(StreamSocket*, std::string& recvq);
 
        /** Called whenever a user sets away or returns from being away.
         * The away message is available as a parameter, but should not be modified.
@@ -1337,15 +1259,6 @@ class CoreExport Module : public Extensible
         */
        virtual void OnGarbageCollect();
 
-       /** Called whenever a user's write buffer has been completely sent.
-        * This is called when the user's write buffer is completely empty, and
-        * there are no more pending bytes to be written and no pending write events
-        * in the socket engine's queue. This may be used to refill the buffer with
-        * data which is being spooled in a controlled manner, e.g. LIST lines.
-        * @param user The user who's buffer is now empty.
-        */
-       virtual void OnBufferFlushed(User* user);
-
        /** Add test suite hooks here. These are used for testing functionality of a module
         * via the --testsuite debugging parameter.
         */
@@ -1359,12 +1272,6 @@ class CoreExport Module : public Extensible
 
        virtual ModResult OnNumeric(User* user, unsigned int numeric, const std::string &text);
 
-       /** Called for every time the user's host or ident changes, to indicate wether or not the 'Changing host'
-        * message should be sent, if enabled. Certain modules such as auditorium may opt to hide this message
-        * even if it is enabled.
-        */
-       virtual ModResult OnHostCycle(User* user);
-
        /** Called whenever a result from /WHO is about to be returned
         * @param source The user running the /WHO query
         * @param user The user that this line of the query is about
@@ -1391,7 +1298,6 @@ class CoreExport Module : public Extensible
 class CoreExport ConfigReader : public classbase
 {
   protected:
-       InspIRCd* ServerInstance;
        /** Error code
         */
        long error;
@@ -1401,7 +1307,7 @@ class CoreExport ConfigReader : public classbase
         * This constructor initialises the ConfigReader class to read the inspircd.conf file
         * as specified when running ./configure.
         */
-       ConfigReader(InspIRCd* Instance);
+       ConfigReader();
        /** Default destructor.
         * This method destroys the ConfigReader class.
         */
@@ -1483,7 +1389,6 @@ class CoreExport ConfigReader : public classbase
  */
 class CoreExport FileReader : public classbase
 {
-       InspIRCd* ServerInstance;
        /** The file contents
         */
        file_cache fc;
@@ -1501,14 +1406,14 @@ class CoreExport FileReader : public classbase
         * This method does not load any file into memory, you must use the LoadFile method
         * after constructing the class this way.
         */
-       FileReader(InspIRCd* Instance);
+       FileReader();
 
        /** Secondary constructor.
         * This method initialises the class with a file loaded into it ready for GetLine and
         * and other methods to be called. If the file could not be loaded, FileReader::FileSize
         * returns 0.
         */
-       FileReader(InspIRCd* Instance, const std::string &filename);
+       FileReader(const std::string &filename);
 
        /** Default destructor.
         * This deletes the memory allocated to the file.
@@ -1549,14 +1454,6 @@ class CoreExport FileReader : public classbase
        int FileSize();
 };
 
-/** A DLLFactory (designed to load shared objects) containing a
- * handle to a module's init_module() function. Unfortunately,
- * due to the design of shared object systems we must keep this
- * hanging around, as if we remove this handle, we remove the
- * shared object file from memory (!)
- */
-typedef DLLFactory<Module> ircd_module;
-
 /** A list of modules
  */
 typedef std::vector<Module*> IntModuleList;
@@ -1587,14 +1484,10 @@ class CoreExport ModuleManager : public classbase
         */
        int ModCount;
 
-       /** Our pointer to the main insp instance
-        */
-       InspIRCd* Instance;
-
        /** List of loaded modules and shared object/dll handles
         * keyed by module name
         */
-       std::map<std::string, std::pair<ircd_module*, Module*> > Modules;
+       std::map<std::string, Module*> Modules;
 
        enum {
                PRIO_STATE_FIRST,
@@ -1610,7 +1503,7 @@ class CoreExport ModuleManager : public classbase
 
        /** Simple, bog-standard, boring constructor.
         */
-       ModuleManager(InspIRCd* Ins);
+       ModuleManager();
 
        /** Destructor
         */
@@ -1841,9 +1734,9 @@ class CoreExport ModuleManager : public classbase
 #ifdef WINDOWS
 
 #define MODULE_INIT(y) \
-       extern "C" DllExport Module * init_module(InspIRCd* Me) \
+       extern "C" DllExport Module * init_module() \
        { \
-               return new y(Me); \
+               return new y; \
        } \
        BOOLEAN WINAPI DllMain(HINSTANCE hDllHandle, DWORD nReason, LPVOID Reserved) \
        { \
@@ -1859,9 +1752,9 @@ class CoreExport ModuleManager : public classbase
 #else
 
 #define MODULE_INIT(y) \
-       extern "C" DllExport Module * init_module(InspIRCd* Me) \
+       extern "C" DllExport Module * init_module() \
        { \
-               return new y(Me); \
+               return new y; \
        }
 #endif