diff options
| author | Quietust | 2012-01-31 10:55:38 -0600 |
|---|---|---|
| committer | Quietust | 2012-01-31 10:55:38 -0600 |
| commit | 9afcea3debd7588e897eea21b33b8a86e9adcc4d (patch) | |
| tree | 3db1c6e2a1613eef4d8ebac243b5353e18db654b /library/PluginManager.cpp | |
| parent | a82f4c913803038d20b4c92e65b53f9fea9555dc (diff) | |
| download | dfhack-9afcea3debd7588e897eea21b33b8a86e9adcc4d.tar.gz dfhack-9afcea3debd7588e897eea21b33b8a86e9adcc4d.tar.bz2 dfhack-9afcea3debd7588e897eea21b33b8a86e9adcc4d.tar.xz | |
In all loops that iterate across a vector, use a size_t as the index
Diffstat (limited to 'library/PluginManager.cpp')
| -rw-r--r-- | library/PluginManager.cpp | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/library/PluginManager.cpp b/library/PluginManager.cpp index 9acd05aa..84f052ba 100644 --- a/library/PluginManager.cpp +++ b/library/PluginManager.cpp @@ -125,7 +125,7 @@ Plugin::Plugin(Core * core, const std::string & filepath, const std::string & _f filename = filepath; parent = pm; name.reserve(_filename.size()); - for(int i = 0; i < _filename.size();i++) + for(size_t i = 0; i < _filename.size();i++) { char ch = _filename[i]; if(ch == '.') @@ -274,7 +274,7 @@ command_result Plugin::invoke( std::string & command, std::vector <std::string> access->lock_add(); if(state == PS_LOADED) { - for (int i = 0; i < commands.size();i++) + for (size_t i = 0; i < commands.size();i++) { PluginCommand &cmd = commands[i]; if(cmd.name == command) @@ -322,7 +322,7 @@ bool Plugin::can_invoke_hotkey( std::string & command, df::viewscreen *top ) access->lock_add(); if(state == PS_LOADED) { - for (int i = 0; i < commands.size();i++) + for (size_t i = 0; i < commands.size();i++) { PluginCommand &cmd = commands[i]; if(cmd.name == command) @@ -384,7 +384,7 @@ PluginManager::PluginManager(Core * core) cmdlist_mutex = new mutex(); vector <string> filez; getdir(path, filez); - for(int i = 0; i < filez.size();i++) + for(size_t i = 0; i < filez.size();i++) { if(hasEnding(filez[i],searchstr)) { @@ -398,7 +398,7 @@ PluginManager::PluginManager(Core * core) PluginManager::~PluginManager() { - for(int i = 0; i < all_plugins.size();i++) + for(size_t i = 0; i < all_plugins.size();i++) { delete all_plugins[i]; } @@ -408,7 +408,7 @@ PluginManager::~PluginManager() Plugin *PluginManager::getPluginByName (const std::string & name) { - for(int i = 0; i < all_plugins.size(); i++) + for(size_t i = 0; i < all_plugins.size(); i++) { if(name == all_plugins[i]->name) return all_plugins[i]; @@ -441,7 +441,7 @@ bool PluginManager::CanInvokeHotkey(std::string &command, df::viewscreen *top) void PluginManager::OnUpdate( void ) { - for(int i = 0; i < all_plugins.size(); i++) + for(size_t i = 0; i < all_plugins.size(); i++) { all_plugins[i]->on_update(); } @@ -449,7 +449,7 @@ void PluginManager::OnUpdate( void ) void PluginManager::OnStateChange( state_change_event event ) { - for(int i = 0; i < all_plugins.size(); i++) + for(size_t i = 0; i < all_plugins.size(); i++) { all_plugins[i]->on_state_change(event); } @@ -460,7 +460,7 @@ void PluginManager::registerCommands( Plugin * p ) { cmdlist_mutex->lock(); vector <PluginCommand> & cmds = p->commands; - for(int i = 0; i < cmds.size();i++) + for(size_t i = 0; i < cmds.size();i++) { belongs[cmds[i].name] = p; } @@ -472,7 +472,7 @@ void PluginManager::unregisterCommands( Plugin * p ) { cmdlist_mutex->lock(); vector <PluginCommand> & cmds = p->commands; - for(int i = 0; i < cmds.size();i++) + for(size_t i = 0; i < cmds.size();i++) { belongs.erase(cmds[i].name); } |
