From b7d84270bce15507775c7ec216dc1d5ad8b02a53 Mon Sep 17 00:00:00 2001 From: attilamolnar Date: Wed, 18 Sep 2013 10:27:06 +0200 Subject: [PATCH] Fix a few issues - Rehash notices - Modes in CAPAB - GetTargetAndAction() not being static in m_callerid - Loading custom configuration files using --config. (@SaberUK) - ServerConfig::Read not using std::endl. (@SaberUK) - Out of date comments in opers.conf.example, issue #624 --- docs/conf/opers.conf.example | 2 -- src/commands/cmd_rehash.cpp | 2 +- src/configreader.cpp | 2 +- src/inspircd.cpp | 2 +- src/modules/m_callerid.cpp | 2 +- src/modules/m_spanningtree/capab.cpp | 2 +- 6 files changed, 5 insertions(+), 7 deletions(-) diff --git a/docs/conf/opers.conf.example b/docs/conf/opers.conf.example index e7b65b675..b39c73e43 100644 --- a/docs/conf/opers.conf.example +++ b/docs/conf/opers.conf.example @@ -54,8 +54,6 @@ & parameters, Use /* Don't do anything with the logs here -- logs are restarted * after the config thread has completed. */ - ServerInstance->Rehash(); + ServerInstance->Rehash(user->uuid); } else { diff --git a/src/configreader.cpp b/src/configreader.cpp index 0df69071b..7493d980c 100644 --- a/src/configreader.cpp +++ b/src/configreader.cpp @@ -521,7 +521,7 @@ void ServerConfig::Read() catch (CoreException& err) { valid = false; - errstr << err.GetReason(); + errstr << err.GetReason() << std::endl; } } diff --git a/src/inspircd.cpp b/src/inspircd.cpp index 8336a4847..368ca6e8b 100644 --- a/src/inspircd.cpp +++ b/src/inspircd.cpp @@ -335,7 +335,7 @@ InspIRCd::InspIRCd(int argc, char** argv) : { case 'c': /* Config filename was set */ - ConfigFileName = optarg; + ConfigFileName = ServerInstance->Config->Paths.PrependConfig(optarg); break; case 0: /* getopt_long_only() set an int variable, just keep going */ diff --git a/src/modules/m_callerid.cpp b/src/modules/m_callerid.cpp index 33c874946..1a2fd3a84 100644 --- a/src/modules/m_callerid.cpp +++ b/src/modules/m_callerid.cpp @@ -158,7 +158,7 @@ class CommandAccept : public Command */ typedef std::pair ACCEPTAction; - ACCEPTAction GetTargetAndAction(std::string& tok) + static ACCEPTAction GetTargetAndAction(std::string& tok) { bool remove = (tok[0] == '-'); if ((remove) || (tok[0] == '+')) diff --git a/src/modules/m_spanningtree/capab.cpp b/src/modules/m_spanningtree/capab.cpp index 7afcc50e5..52404aa1d 100644 --- a/src/modules/m_spanningtree/capab.cpp +++ b/src/modules/m_spanningtree/capab.cpp @@ -64,8 +64,8 @@ static std::string BuildModeList(ModeType type) { if (pm->GetPrefix()) mdesc.push_back(pm->GetPrefix()); - mdesc.push_back(mh->GetModeChar()); } + mdesc.push_back(mh->GetModeChar()); modes.push_back(mdesc); } } -- 2.39.5