m_channames Call ValidateChans() from cull() on unload

This ensures that if ValidateChans() removes everyone from a channel the
channel is added to the cull list at the right time, that is, before the
destruction of objects begin.

The core IsChannel() is less strict than us regardless of our
configuration (for now) but nevertheless this is the correct behavior.

diff --git a/src/modules/m_channames.cpp b/src/modules/m_channames.cpp
index 69b501792c77830bf407ecb7549e048d4ccdaff5..be9eda2b9ca5a9cdc053e952d134db020f2f29e2 100644 (file)
--- a/src/modules/m_channames.cpp
+++ b/src/modules/m_channames.cpp
@@ -141,10 +141,11 @@ class ModuleChannelNames : public Module
                }
        }
 
-       ~ModuleChannelNames()
+       CullResult cull() CXX11_OVERRIDE
        {
                ServerInstance->IsChannel = rememberer;
                ValidateChans();
+               return Module::cull();
        }
 
        Version GetVersion() CXX11_OVERRIDE