return 1;
/* We dont handle any hash types except for plaintext - Thanks tra26 */
- if (hashtype != "" && hashtype != "plaintext")
+ if (!hashtype.empty() && hashtype != "plaintext")
/* See below. 1 because they dont match */
return 1;
std::vector<std::string> new_parameters(parameters);
if (!items2.GetToken(extrastuff))
- extrastuff = "";
+ extrastuff.clear();
new_parameters[splithere] = item;
if (extra >= 0)
if (n != cmdlist.end())
{
+ if ((!parameters.empty()) && (parameters.back().empty()) && (!n->second->allow_empty_last_param))
+ return CMD_INVALID;
+
if (parameters.size() >= n->second->min_params)
{
bool bOkay = false;
* a
* test
*/
- std::string lparam = "";
+ std::string lparam;
/*
* The '-1' here is a clever trick, we'll go backwards throwing everything into a temporary param
command.c_str(), user->nick.c_str(), user->ident.c_str(), user->host.c_str());
return do_more;
}
+
+ if ((!command_p.empty()) && (command_p.back().empty()) && (!cm->second->allow_empty_last_param))
+ command_p.pop_back();
+
if (command_p.size() < cm->second->min_params)
{
user->WriteNumeric(ERR_NEEDMOREPARAMS, "%s %s :Not enough parameters.", user->nick.c_str(), command.c_str());