From 091d958b0c22e1bfd88144b4cda0c7ca00f4cf28 Mon Sep 17 00:00:00 2001 From: faketruth Date: Wed, 28 Mar 2012 18:11:06 +0000 Subject: Got rid of sPluginManagerState /pluginlist now also shows the number of active plugins Fixed crash caused by lingering bound Lua commands http://forum.mc-server.org/showthread.php?tid=212&pid=2541#pid2541 Added error messages when binding commands to an invalid plugin git-svn-id: http://mc-server.googlecode.com/svn/trunk@440 0a769ca7-a7f5-676a-18bf-c427514a06d6 --- source/cPluginManager.cpp | 131 ++++++++++++++++++++++++---------------------- 1 file changed, 67 insertions(+), 64 deletions(-) (limited to 'source/cPluginManager.cpp') diff --git a/source/cPluginManager.cpp b/source/cPluginManager.cpp index 1aeb0eb2f..0b1d11299 100644 --- a/source/cPluginManager.cpp +++ b/source/cPluginManager.cpp @@ -10,6 +10,7 @@ #include "cRoot.h" #include "cLuaCommandBinder.h" #include "../iniFile/iniFile.h" +#include "tolua++.h" #include "SquirrelBindings.h" #if USE_SQUIRREL @@ -22,24 +23,6 @@ -typedef std::list< cPlugin_Lua* > LuaPluginList; -typedef std::map< cPluginManager::PluginHook, cPluginManager::PluginList > HookMap; - - - - - -struct cPluginManager::sPluginManagerState -{ - LuaPluginList LuaPlugins; - cPluginManager::PluginList Plugins; - HookMap Hooks; -}; - - - - - cPluginManager* cPluginManager::GetPluginManager() { LOGWARN("WARNING: Using deprecated function cPluginManager::GetPluginManager() use cRoot::Get()->GetPluginManager() instead!"); @@ -51,8 +34,7 @@ cPluginManager* cPluginManager::GetPluginManager() cPluginManager::cPluginManager() - : m_pState( new sPluginManagerState ) - , m_LuaCommandBinder( new cLuaCommandBinder() ) + : m_LuaCommandBinder( new cLuaCommandBinder() ) , m_bReloadPlugins(false) { } @@ -66,7 +48,6 @@ cPluginManager::~cPluginManager() UnloadPluginsNow(); delete m_LuaCommandBinder; - delete m_pState; } @@ -192,8 +173,8 @@ void cPluginManager::Tick(float a_Dt) ReloadPluginsNow(); } - HookMap::iterator Plugins = m_pState->Hooks.find( E_PLUGIN_TICK ); - if( Plugins != m_pState->Hooks.end() ) + HookMap::iterator Plugins = m_Hooks.find( E_PLUGIN_TICK ); + if( Plugins != m_Hooks.end() ) { for( PluginList::iterator itr = Plugins->second.begin(); itr != Plugins->second.end(); ++itr ) { @@ -208,7 +189,7 @@ void cPluginManager::Tick(float a_Dt) bool cPluginManager::CallHook( PluginHook a_Hook, unsigned int a_NumArgs, ... ) { - HookMap::iterator Plugins = m_pState->Hooks.find( a_Hook ); + HookMap::iterator Plugins = m_Hooks.find( a_Hook ); // Special case for chat hook, since you can also bind commands (bound commands don't use chat hook) if( a_Hook == E_PLUGIN_CHAT ) @@ -223,7 +204,7 @@ bool cPluginManager::CallHook( PluginHook a_Hook, unsigned int a_NumArgs, ... ) if( m_LuaCommandBinder->HandleCommand( std::string( Message ), Player ) ) return true; - if( Plugins != m_pState->Hooks.end() ) + if( Plugins != m_Hooks.end() ) { for( PluginList::iterator itr = Plugins->second.begin(); itr != Plugins->second.end(); ++itr ) { @@ -235,7 +216,7 @@ bool cPluginManager::CallHook( PluginHook a_Hook, unsigned int a_NumArgs, ... ) return false; } - if( Plugins == m_pState->Hooks.end() ) + if( Plugins == m_Hooks.end() ) { return false; } @@ -395,9 +376,9 @@ bool cPluginManager::CallHook( PluginHook a_Hook, unsigned int a_NumArgs, ... ) -cPlugin* cPluginManager::GetPlugin( const char* a_Plugin ) +cPlugin* cPluginManager::GetPlugin( const char* a_Plugin ) const { - for( PluginList::iterator itr = m_pState->Plugins.begin(); itr != m_pState->Plugins.end(); ++itr ) + for( PluginList::const_iterator itr = m_Plugins.begin(); itr != m_Plugins.end(); ++itr ) { if (strcmp((*itr)->GetName(), a_Plugin) == 0 ) { @@ -411,9 +392,9 @@ cPlugin* cPluginManager::GetPlugin( const char* a_Plugin ) -const cPluginManager::PluginList & cPluginManager::GetAllPlugins() +const cPluginManager::PluginList & cPluginManager::GetAllPlugins() const { - return m_pState->Plugins; + return m_Plugins; } @@ -422,23 +403,23 @@ const cPluginManager::PluginList & cPluginManager::GetAllPlugins() void cPluginManager::UnloadPluginsNow() { - m_pState->Hooks.clear(); + m_Hooks.clear(); - while( m_pState->LuaPlugins.size() > 0 ) + while( m_LuaPlugins.size() > 0 ) { - cPlugin_Lua* LuaPlugin = *m_pState->LuaPlugins.begin(); + cPlugin_Lua* LuaPlugin = *m_LuaPlugins.begin(); if( LuaPlugin ) { cWebAdmin* WebAdmin = cRoot::Get()->GetWebAdmin(); if( WebAdmin ) WebAdmin->RemovePlugin( LuaPlugin->GetLuaState() ); delete LuaPlugin; } - m_pState->LuaPlugins.remove( LuaPlugin ); + m_LuaPlugins.remove( LuaPlugin ); } - while( m_pState->Plugins.size() > 0 ) + while( m_Plugins.size() > 0 ) { - RemovePlugin( *m_pState->Plugins.begin(), true ); + RemovePlugin( *m_Plugins.begin(), true ); } //SquirrelVM::Shutdown(); // This breaks shit @@ -450,19 +431,19 @@ void cPluginManager::UnloadPluginsNow() void cPluginManager::RemoveHooks( cPlugin* a_Plugin ) { - m_pState->Hooks[ E_PLUGIN_TICK].remove ( a_Plugin ); - m_pState->Hooks[ E_PLUGIN_CHAT].remove ( a_Plugin ); - m_pState->Hooks[ E_PLUGIN_COLLECT_ITEM].remove( a_Plugin ); - m_pState->Hooks[ E_PLUGIN_BLOCK_DIG].remove ( a_Plugin ); - m_pState->Hooks[ E_PLUGIN_BLOCK_PLACE].remove ( a_Plugin ); - m_pState->Hooks[ E_PLUGIN_DISCONNECT].remove ( a_Plugin ); - m_pState->Hooks[ E_PLUGIN_HANDSHAKE].remove ( a_Plugin ); - m_pState->Hooks[ E_PLUGIN_LOGIN].remove ( a_Plugin ); - m_pState->Hooks[ E_PLUGIN_PLAYER_SPAWN].remove( a_Plugin ); - m_pState->Hooks[ E_PLUGIN_PLAYER_JOIN].remove ( a_Plugin ); - m_pState->Hooks[ E_PLUGIN_PLAYER_MOVE].remove ( a_Plugin ); - m_pState->Hooks[ E_PLUGIN_TAKE_DAMAGE].remove ( a_Plugin ); - m_pState->Hooks[ E_PLUGIN_KILLED].remove ( a_Plugin ); + m_Hooks[ E_PLUGIN_TICK].remove ( a_Plugin ); + m_Hooks[ E_PLUGIN_CHAT].remove ( a_Plugin ); + m_Hooks[ E_PLUGIN_COLLECT_ITEM].remove( a_Plugin ); + m_Hooks[ E_PLUGIN_BLOCK_DIG].remove ( a_Plugin ); + m_Hooks[ E_PLUGIN_BLOCK_PLACE].remove ( a_Plugin ); + m_Hooks[ E_PLUGIN_DISCONNECT].remove ( a_Plugin ); + m_Hooks[ E_PLUGIN_HANDSHAKE].remove ( a_Plugin ); + m_Hooks[ E_PLUGIN_LOGIN].remove ( a_Plugin ); + m_Hooks[ E_PLUGIN_PLAYER_SPAWN].remove( a_Plugin ); + m_Hooks[ E_PLUGIN_PLAYER_JOIN].remove ( a_Plugin ); + m_Hooks[ E_PLUGIN_PLAYER_MOVE].remove ( a_Plugin ); + m_Hooks[ E_PLUGIN_TAKE_DAMAGE].remove ( a_Plugin ); + m_Hooks[ E_PLUGIN_KILLED].remove ( a_Plugin ); } @@ -474,7 +455,7 @@ void cPluginManager::RemovePlugin( cPlugin* a_Plugin, bool a_bDelete /* = false if( a_bDelete ) { m_LuaCommandBinder->RemoveBindingsForPlugin( a_Plugin ); - m_pState->Plugins.remove( a_Plugin ); + m_Plugins.remove( a_Plugin ); RemoveHooks( a_Plugin ); a_Plugin->OnDisable(); if( a_Plugin->GetLanguage() != cPlugin::E_SQUIRREL ) // Squirrel needs to clean it up himself! @@ -484,7 +465,7 @@ void cPluginManager::RemovePlugin( cPlugin* a_Plugin, bool a_bDelete /* = false } else { - for( LuaPluginList::iterator itr = m_pState->LuaPlugins.begin(); itr != m_pState->LuaPlugins.end(); ++itr ) + for( LuaPluginList::iterator itr = m_LuaPlugins.begin(); itr != m_LuaPlugins.end(); ++itr ) { (*itr)->RemovePlugin( a_Plugin ); } @@ -497,13 +478,15 @@ void cPluginManager::RemovePlugin( cPlugin* a_Plugin, bool a_bDelete /* = false bool cPluginManager::AddPlugin( cPlugin* a_Plugin ) { + a_Plugin->m_bCanBindCommands = true; if( a_Plugin->Initialize() ) { - m_pState->Plugins.remove( a_Plugin ); - m_pState->Plugins.push_back( a_Plugin ); + m_Plugins.remove( a_Plugin ); + m_Plugins.push_back( a_Plugin ); return true; } + a_Plugin->m_bCanBindCommands = false; RemoveHooks( a_Plugin ); // Undo any damage the Initialize() might have done return false; } @@ -516,13 +499,23 @@ bool cPluginManager::AddPlugin( lua_State* a_LuaState, cPlugin* a_Plugin ) { a_Plugin->SetLanguage( cPlugin::E_LUA ); cPlugin_Lua* LuaPlugin = GetLuaPlugin( a_LuaState ); + if( LuaPlugin == NULL ) + { + lua_Debug ar; + lua_getstack(a_LuaState, 1, &ar); + lua_getinfo(a_LuaState, "nSl", &ar); + LOGERROR("ERROR: Trying to add an 'old style' plugin from within a 'new style' plugin.\nIn file: %s at line: %i", ar.source, ar.currentline); + } + a_Plugin->m_bCanBindCommands = true; if( LuaPlugin && a_Plugin->Initialize() ) { - m_pState->Plugins.remove( a_Plugin ); - m_pState->Plugins.push_back( a_Plugin ); + m_Plugins.remove( a_Plugin ); + m_Plugins.push_back( a_Plugin ); LuaPlugin->AddPlugin( a_Plugin ); return true; } + + a_Plugin->m_bCanBindCommands = false; return false; } @@ -532,13 +525,13 @@ bool cPluginManager::AddPlugin( lua_State* a_LuaState, cPlugin* a_Plugin ) bool cPluginManager::AddLuaPlugin( cPlugin_Lua* a_Plugin ) { - m_pState->LuaPlugins.push_back( a_Plugin ); // It HAS to be in here before calling Initialize, so it can be found by AddPlugin() + m_LuaPlugins.push_back( a_Plugin ); // It HAS to be in here before calling Initialize, so it can be found by AddPlugin() if(a_Plugin->Initialize() ) { return true; } LOG(">>>>>>> Could not initialize a plugin! "); - m_pState->LuaPlugins.remove( a_Plugin ); + m_LuaPlugins.remove( a_Plugin ); return false; } @@ -548,12 +541,12 @@ bool cPluginManager::AddLuaPlugin( cPlugin_Lua* a_Plugin ) void cPluginManager::RemoveLuaPlugin( std::string a_FileName ) { - for( LuaPluginList::iterator itr = m_pState->LuaPlugins.begin(); itr != m_pState->LuaPlugins.end(); ++itr ) + for( LuaPluginList::iterator itr = m_LuaPlugins.begin(); itr != m_LuaPlugins.end(); ++itr ) { if( (*itr)->GetFileName() == a_FileName ) { cPlugin_Lua* Plugin = *itr; - m_pState->LuaPlugins.remove( Plugin ); + m_LuaPlugins.remove( Plugin ); delete Plugin; return; } @@ -566,14 +559,14 @@ void cPluginManager::RemoveLuaPlugin( std::string a_FileName ) cPlugin_Lua* cPluginManager::GetLuaPlugin( lua_State* a_State ) { - for( LuaPluginList::iterator itr = m_pState->LuaPlugins.begin(); itr != m_pState->LuaPlugins.end(); ++itr ) + for( LuaPluginList::iterator itr = m_LuaPlugins.begin(); itr != m_LuaPlugins.end(); ++itr ) { if( (*itr)->GetLuaState() == a_State ) { return *itr; } } - return 0; + return NULL; } @@ -587,7 +580,7 @@ void cPluginManager::AddHook( cPlugin* a_Plugin, PluginHook a_Hook ) LOGWARN("Called cPluginManager::AddHook while a_Plugin is NULL"); return; } - PluginList & Plugins = m_pState->Hooks[ a_Hook ]; + PluginList & Plugins = m_Hooks[ a_Hook ]; Plugins.remove( a_Plugin ); Plugins.push_back( a_Plugin ); } @@ -596,11 +589,21 @@ void cPluginManager::AddHook( cPlugin* a_Plugin, PluginHook a_Hook ) -unsigned int cPluginManager::GetNumPlugins() +unsigned int cPluginManager::GetNumPlugins() const { - return m_pState->Plugins.size(); + return m_Plugins.size(); } + +bool cPluginManager::HasPlugin( cPlugin* a_Plugin ) const +{ + for( PluginList::const_iterator itr = m_Plugins.begin(); itr != m_Plugins.end(); ++itr ) + { + if( *itr == a_Plugin ) + return true; + } + return false; +} \ No newline at end of file -- cgit v1.2.3