Update documentation and remove unused classes/functions

[user/henk/code/inspircd.git] / src / inspircd.cpp

diff --git a/src/inspircd.cpp b/src/inspircd.cpp
index a1273d651638a1821ca464da833881099b7db7ef..d9f1d042366c8653802bf8a822aea03d71e02dc6 100644 (file)
--- a/src/inspircd.cpp
+++ b/src/inspircd.cpp
@@ -83,25 +83,20 @@ const char* ExitCodes[] =
 
 template<typename T> static void DeleteZero(T*&n)
 {
-       if (n != NULL)
-       {
-               delete n;
-               n = NULL;
-       }
+       T* t = n;
+       n = NULL;
+       delete t;
 }
 
 void InspIRCd::Cleanup()
 {
-       if (Config)
+       for (unsigned int i = 0; i < ports.size(); i++)
        {
-               for (unsigned int i = 0; i < ports.size(); i++)
-               {
-                       /* This calls the constructor and closes the listening socket */
-                       delete ports[i];
-               }
-
-               ports.clear();
+               /* This calls the constructor and closes the listening socket */
+               ports[i]->cull();
+               delete ports[i];
        }
+       ports.clear();
 
        /* Close all client sockets, or the new process inherits them */
        std::vector<User*>::reverse_iterator i = Users->local_users.rbegin();
@@ -111,25 +106,13 @@ void InspIRCd::Cleanup()
                Users->QuitUser(u, "Server shutdown");
        }
 
-       /* We do this more than once, so that any service providers get a
-        * chance to be unhooked by the modules using them, but then get
-        * a chance to be removed themsleves.
-        *
-        * XXX there may be a better way to do this
-        */
-       for (int tries = 0; tries < 4; tries++)
-       {
-               std::vector<std::string> module_names = Modules->GetAllModuleNames(0);
-               for (std::vector<std::string>::iterator k = module_names.begin(); k != module_names.end(); ++k)
-               {
-                       /* Unload all modules, so they get a chance to clean up their listeners */
-                       this->Modules->Unload(k->c_str());
-               }
-               GlobalCulls.Apply();
-       }
+       GlobalCulls.Apply();
+       Modules->UnloadAll();
 
        /* Delete objects dynamically allocated in constructor (destructor would be more appropriate, but we're likely exiting) */
        /* Must be deleted before modes as it decrements modelines */
+       if (FakeClient)
+               FakeClient->cull();
        DeleteZero(this->FakeClient);
        DeleteZero(this->Users);
        DeleteZero(this->Modes);
@@ -788,7 +771,8 @@ int InspIRCd::Run()
                this->SE->DispatchEvents();
 
                /* if any users were quit, take them out */
-               this->GlobalCulls.Apply();
+               GlobalCulls.Apply();
+               AtomicActions.Run();
 
                if (this->s_signal)
                {