I think this was the source of burlex's confusion with the spanningtree propogation bug and a warning in vc8 -- CallCommandHandler returned BOOL?! Yet its casting that bool somehow from a CmdResult.

It should be CmdResult. Fixed.


git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@7101 e03df62e-2008-0410-955e-edbf42e46eb7

2 files changed, 2 insertions, 2 deletions

diff --git a/include/inspircd.h b/include/inspircd.h
index 29493d322..98b8e0c1d 100644
--- a/include/inspircd.h
+++ b/include/inspircd.h
@@ -977,7 +977,7 @@ class CoreExport InspIRCd : public classbase
 	 * @param user The user to execute the command as
 	 * @return True if the command handler was called successfully
 	 */
-	bool CallCommandHandler(const std::string &commandname, const char** parameters, int pcnt, userrec* user);
+	CmdResult CallCommandHandler(const std::string &commandname, const char** parameters, int pcnt, userrec* user);
 
 	/** Return true if the command is a module-implemented command and the given parameters are valid for it
 	 * @param parameters The mode parameters
diff --git a/src/modules.cpp b/src/modules.cpp
index 75ec204b5..0ce577214 100644
--- a/src/modules.cpp
+++ b/src/modules.cpp
@@ -376,7 +376,7 @@ bool InspIRCd::MatchText(const std::string &sliteral, const std::string &spatter
 	return match(sliteral.c_str(),spattern.c_str());
 }
 
-bool InspIRCd::CallCommandHandler(const std::string &commandname, const char** parameters, int pcnt, userrec* user)
+CmdResult InspIRCd::CallCommandHandler(const std::string &commandname, const char** parameters, int pcnt, userrec* user)
 {
 	return (this->Parser->CallHandler(commandname,parameters,pcnt,user) != CMD_INVALID);
 }