Move RPL_SYNTAX to 650 to prevent a collision with RPL_TEXT.

Also move the command name to a parameter so that it is more easily
parseable by software.
This commit is contained in:
Peter Powell 2017-09-02 19:53:48 +01:00
parent b13b1b8a06
commit 5a3f8af9f9
2 changed files with 3 additions and 2 deletions

View File

@ -73,7 +73,6 @@ enum
RPL_AWAY = 301,
RPL_USERHOST = 302,
RPL_ISON = 303,
RPL_SYNTAX = 304, // insp-specific
RPL_UNAWAY = 305,
RPL_NOWAWAY = 306,
@ -187,6 +186,8 @@ enum
ERR_CANTJOINOPERSONLY = 520, // unrealircd, but crap to have so many numerics for cant join..
ERR_CANTSENDTOUSER = 531, // ???
RPL_SYNTAX = 650, // insp-specific
ERR_CHANOPEN = 713,
ERR_KNOCKONCHAN = 714,

View File

@ -307,7 +307,7 @@ void CommandParser::ProcessCommand(LocalUser *user, std::string &cmd)
user->CommandFloodPenalty += failpenalty;
user->WriteNumeric(ERR_NEEDMOREPARAMS, command, "Not enough parameters.");
if ((ServerInstance->Config->SyntaxHints) && (user->registered == REG_ALL) && (handler->syntax.length()))
user->WriteNumeric(RPL_SYNTAX, InspIRCd::Format("SYNTAX %s %s", handler->name.c_str(), handler->syntax.c_str()));
user->WriteNumeric(RPL_SYNTAX, handler->name, handler->syntax);
return;
}