Fix RemoveCommands to remove all commands (this function had some really odd removal logic?)

git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@9554 e03df62e-2008-0410-955e-edbf42e46eb7
This commit is contained in:
brain 2008-04-20 18:58:59 +00:00
parent a36d5a1395
commit e5131f92b3
2 changed files with 4 additions and 14 deletions

View File

@ -180,9 +180,8 @@ class CoreExport CommandParser : public classbase
/** Remove all commands relating to module 'source'.
* @param source A module name which has introduced new commands
* @return True This function returns true if commands were removed
*/
bool RemoveCommands(const char* source);
void RemoveCommands(const char* source);
/** Add a new command to the commands hash
* @param f The new Command to add to the list

View File

@ -367,24 +367,15 @@ bool CommandParser::ProcessCommand(User *user, std::string &cmd)
}
}
bool CommandParser::RemoveCommands(const char* source)
void CommandParser::RemoveCommands(const char* source)
{
Commandable::iterator i,safei;
for (i = cmdlist.begin(); i != cmdlist.end(); i++)
for (i = cmdlist.begin(); i != cmdlist.end();)
{
safei = i;
safei++;
if (safei != cmdlist.end())
{
RemoveCommand(safei, source);
}
}
safei = cmdlist.begin();
if (safei != cmdlist.end())
{
i++;
RemoveCommand(safei, source);
}
return true;
}
void CommandParser::RemoveCommand(Commandable::iterator safei, const char* source)