From a3b4110d511951d92fbe9bcdf87aa51e2cd0a1ba Mon Sep 17 00:00:00 2001 From: brain Date: Sun, 4 Nov 2007 21:57:27 +0000 Subject: [PATCH] Instead of a vector of vectors, use an array of vectors as in the original spec on Development/Hooking. This is faster, as only the inner list resizes, and this avoids calls to std::vector::operator[] git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@8531 e03df62e-2008-0410-955e-edbf42e46eb7 --- include/modules.h | 6 +----- src/modules.cpp | 2 -- 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/include/modules.h b/include/modules.h index 6a56384f5..ef0d3f620 100644 --- a/include/modules.h +++ b/include/modules.h @@ -1521,10 +1521,6 @@ typedef DLLFactory ircd_module; */ typedef std::vector IntModuleList; -/** A list of event handlers - */ -typedef std::vector EventHandlerList; - /** An event handler iterator */ typedef IntModuleList::iterator EventHandlerIter; @@ -1576,7 +1572,7 @@ class CoreExport ModuleManager : public classbase /** Event handler hooks. * This needs to be public to be used by FOREACH_MOD and friends. */ - EventHandlerList EventHandlers; + IntModuleList EventHandlers[I_END]; /** Simple, bog-standard, boring constructor. */ diff --git a/src/modules.cpp b/src/modules.cpp index 7eb97873f..ef494a7c6 100644 --- a/src/modules.cpp +++ b/src/modules.cpp @@ -191,8 +191,6 @@ void Module::OnText(User*, void*, int, const std::string&, char, CUList&) { } ModuleManager::ModuleManager(InspIRCd* Ins) : ModCount(0), Instance(Ins) { - for (int n = I_BEGIN; n != I_END; ++n) - EventHandlers.push_back(std::vector()); } ModuleManager::~ModuleManager() -- 2.39.5