summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat
-rw-r--r--include/modules/chanserv/access.h4
-rw-r--r--modules/chanserv/flags.cpp4
-rw-r--r--modules/chanserv/log.cpp2
-rw-r--r--modules/chanserv/xop.cpp4
-rw-r--r--modules/hostserv/main/vhost.cpp20
-rw-r--r--modules/memoserv/main/memoinfo.cpp4
-rw-r--r--src/command.cpp4
-rw-r--r--src/config.cpp20
-rw-r--r--src/logger.cpp8
-rw-r--r--src/process.cpp2
-rw-r--r--src/users.cpp6
-rw-r--r--src/xline.cpp36
12 files changed, 59 insertions, 55 deletions
diff --git a/include/modules/chanserv/access.h b/include/modules/chanserv/access.h
index fc1721e39..0e7f86524 100644
--- a/include/modules/chanserv/access.h
+++ b/include/modules/chanserv/access.h
@@ -55,7 +55,7 @@ class XOPChanAccess : public ChanAccessImpl
using ChanAccessImpl::ChanAccessImpl;
- virtual const Anope::string &GetType() anope_abstract;
+ virtual const Anope::string GetType() anope_abstract;
virtual void SetType(const Anope::string &) anope_abstract;
};
@@ -66,7 +66,7 @@ class FlagsChanAccess : public ChanAccessImpl
using ChanAccessImpl::ChanAccessImpl;
- virtual const Anope::string &GetFlags() anope_abstract;
+ virtual const Anope::string GetFlags() anope_abstract;
virtual void SetFlags(const Anope::string &) anope_abstract;
};
diff --git a/modules/chanserv/flags.cpp b/modules/chanserv/flags.cpp
index 3c365ae6f..ecd8ba9d3 100644
--- a/modules/chanserv/flags.cpp
+++ b/modules/chanserv/flags.cpp
@@ -36,7 +36,7 @@ class FlagsChanAccessImpl : public FlagsChanAccess
public:
using FlagsChanAccess::FlagsChanAccess;
- const Anope::string &GetFlags() override;
+ const Anope::string GetFlags() override;
void SetFlags(const Anope::string &) override;
bool HasPriv(const Anope::string &priv) override
@@ -85,7 +85,7 @@ class FlagsChanAccessType : public ChanAccessType<FlagsChanAccessImpl>
}
};
-const Anope::string &FlagsChanAccessImpl::GetFlags()
+const Anope::string FlagsChanAccessImpl::GetFlags()
{
return Get(&FlagsChanAccessType::flags);
}
diff --git a/modules/chanserv/log.cpp b/modules/chanserv/log.cpp
index 2e69eb2c7..ce6c378d3 100644
--- a/modules/chanserv/log.cpp
+++ b/modules/chanserv/log.cpp
@@ -274,8 +274,6 @@ public:
return;
}
- bool override = !source.AccessFor(ci).HasPriv("SET");
-
std::vector<LogSetting *> ls = ci->GetRefs<LogSetting *>();
for (unsigned i = ls.size(); i > 0; --i)
{
diff --git a/modules/chanserv/xop.cpp b/modules/chanserv/xop.cpp
index 9373a042b..26de3ec41 100644
--- a/modules/chanserv/xop.cpp
+++ b/modules/chanserv/xop.cpp
@@ -40,7 +40,7 @@ class XOPChanAccessImpl : public XOPChanAccess
public:
using XOPChanAccess::XOPChanAccess;
- const Anope::string &GetType() override;
+ const Anope::string GetType() override;
void SetType(const Anope::string &) override;
bool HasPriv(const Anope::string &priv) override
@@ -109,7 +109,7 @@ class XOPChanAccessType : public ChanAccessType<XOPChanAccessImpl>
}
};
-const Anope::string &XOPChanAccessImpl::GetType()
+const Anope::string XOPChanAccessImpl::GetType()
{
return Get(&XOPChanAccessType::type);
}
diff --git a/modules/hostserv/main/vhost.cpp b/modules/hostserv/main/vhost.cpp
index 7c08fe838..35442c9db 100644
--- a/modules/hostserv/main/vhost.cpp
+++ b/modules/hostserv/main/vhost.cpp
@@ -35,9 +35,9 @@ Anope::string VHostImpl::GetIdent()
return Get(&VHostType::vident);
}
-void VHostImpl::SetIdent(const Anope::string &vident)
+void VHostImpl::SetIdent(const Anope::string &vi)
{
- Set(&VHostType::vident, vident);
+ Set(&VHostType::vident, vi);
}
Anope::string VHostImpl::GetHost()
@@ -45,9 +45,9 @@ Anope::string VHostImpl::GetHost()
return Get(&VHostType::vhost);
}
-void VHostImpl::SetHost(const Anope::string &vhost)
+void VHostImpl::SetHost(const Anope::string &vh)
{
- Set(&VHostType::vhost, vhost);
+ Set(&VHostType::vhost, vh);
}
Anope::string VHostImpl::GetCreator()
@@ -55,9 +55,9 @@ Anope::string VHostImpl::GetCreator()
return Get(&VHostType::creator);
}
-void VHostImpl::SetCreator(const Anope::string &creator)
+void VHostImpl::SetCreator(const Anope::string &c)
{
- Set(&VHostType::creator, creator);
+ Set(&VHostType::creator, c);
}
time_t VHostImpl::GetCreated()
@@ -65,9 +65,9 @@ time_t VHostImpl::GetCreated()
return Get(&VHostType::created);
}
-void VHostImpl::SetCreated(time_t created)
+void VHostImpl::SetCreated(time_t cr)
{
- Set(&VHostType::created, created);
+ Set(&VHostType::created, cr);
}
bool VHostImpl::IsDefault()
@@ -75,7 +75,7 @@ bool VHostImpl::IsDefault()
return Get(&VHostType::default_);
}
-void VHostImpl::SetDefault(bool default_)
+void VHostImpl::SetDefault(bool d)
{
- Set(&VHostType::default_, default_);
+ Set(&VHostType::default_, d);
}
diff --git a/modules/memoserv/main/memoinfo.cpp b/modules/memoserv/main/memoinfo.cpp
index 093f8f4fa..09c45d278 100644
--- a/modules/memoserv/main/memoinfo.cpp
+++ b/modules/memoserv/main/memoinfo.cpp
@@ -57,9 +57,9 @@ NickServ::Account *MemoInfoImpl::GetAccount()
return Get(&MemoInfoType::account);
}
-void MemoInfoImpl::SetAccount(NickServ::Account *account)
+void MemoInfoImpl::SetAccount(NickServ::Account *acc)
{
- Set(&MemoInfoType::account, account);
+ Set(&MemoInfoType::account, acc);
}
ChanServ::Channel *MemoInfoImpl::GetChannel()
diff --git a/src/command.cpp b/src/command.cpp
index 7d5b9e46c..57203aba2 100644
--- a/src/command.cpp
+++ b/src/command.cpp
@@ -73,9 +73,9 @@ const Anope::string &CommandSource::GetCommand() const
return this->command.cname;
}
-void CommandSource::SetCommand(const Anope::string &command)
+void CommandSource::SetCommand(const Anope::string &com)
{
- this->command.cname = command;
+ this->command.cname = com;
}
const Anope::string &CommandSource::GetPermission() const
diff --git a/src/config.cpp b/src/config.cpp
index 0222f0e97..302127b97 100644
--- a/src/config.cpp
+++ b/src/config.cpp
@@ -224,13 +224,13 @@ Conf::Conf() : Block("")
this->TimeoutCheck = options->Get<time_t>("timeoutcheck");
this->NickChars = networkinfo->Get<Anope::string>("nick_chars");
- Anope::string locale = options->Get<Anope::string>("locale");
+ Anope::string localename = options->Get<Anope::string>("locale");
Anope::string casemap = options->Get<Anope::string>("casemap");
- if (locale.empty() == casemap.empty())
+ if (localename.empty() == casemap.empty())
throw ConfigException("One of options:locale and options:casemap must be set");
- if (locale.empty())
+ if (localename.empty())
{
// load locale conf
File f(casemap + ".conf", false);
@@ -239,9 +239,9 @@ Conf::Conf() : Block("")
else
{
#if Boost_FOUND
- this->locale = new std::locale(Anope::locale::generate(locale.str()));
+ this->locale = new std::locale(Anope::locale::generate(localename.str()));
#else
- throw ConfigException("Boost.Locale is not enabled, cannot use locale " + locale);
+ throw ConfigException("Boost.Locale is not enabled, cannot use locale " + localename);
#endif
}
@@ -481,21 +481,21 @@ Conf::Conf() : Block("")
for (int i = 0; i < this->CountBlock("casemap"); ++i)
{
- Block *casemap = this->GetBlock("casemap", i);
+ Block *casemapb = this->GetBlock("casemap", i);
- unsigned char upper = casemap->Get<unsigned int>("upper"),
- lower = casemap->Get<unsigned int>("lower");
+ unsigned char upper = casemapb->Get<unsigned int>("upper"),
+ lower = casemapb->Get<unsigned int>("lower");
if (!upper)
{
- Anope::string s = casemap->Get<Anope::string>("upper");
+ Anope::string s = casemapb->Get<Anope::string>("upper");
if (s.length() == 1)
upper = s[0];
}
if (!lower)
{
- Anope::string s = casemap->Get<Anope::string>("lower");
+ Anope::string s = casemapb->Get<Anope::string>("lower");
if (s.length() == 1)
lower = s[0];
}
diff --git a/src/logger.cpp b/src/logger.cpp
index 6bd226880..ec2cf2f67 100644
--- a/src/logger.cpp
+++ b/src/logger.cpp
@@ -488,15 +488,15 @@ Logger Logger::Source(CommandSource *s) const
return l;
}
-Logger Logger::Bot(ServiceBot *bot) const
+Logger Logger::Bot(ServiceBot *b) const
{
Logger l = *this;
- l.bot = bot;
+ l.bot = b;
return l;
}
-Logger Logger::Bot(const Anope::string &bot) const
+Logger Logger::Bot(const Anope::string &botname) const
{
- return Bot(Config ? Config->GetClient(bot) : nullptr);
+ return Bot(Config ? Config->GetClient(botname) : nullptr);
}
diff --git a/src/process.cpp b/src/process.cpp
index 67c5cbf14..7b8bd18e6 100644
--- a/src/process.cpp
+++ b/src/process.cpp
@@ -58,6 +58,8 @@ void Anope::Process(const Anope::string &buffer)
MessageSource src(source);
EventReturn MOD_RESULT = EventManager::Get()->Dispatch(&Event::Message::OnMessage, src, command, params);
+ if (MOD_RESULT == EVENT_STOP)
+ return;
ProcessCommand(src, command, params);
}
diff --git a/src/users.cpp b/src/users.cpp
index 3561e9ea9..fa130204c 100644
--- a/src/users.cpp
+++ b/src/users.cpp
@@ -40,7 +40,11 @@ int OperCount = 0;
std::list<User *> User::quitting_users;
-User::User(const Anope::string &snick, const Anope::string &sident, const Anope::string &shost, const Anope::string &svhost, const Anope::string &uip, Server *sserver, const Anope::string &srealname, time_t ts, const Anope::string &smodes, const Anope::string &suid, NickServ::Account *account) : ip(uip), logger(this)
+User::User(const Anope::string &snick, const Anope::string &sident, const Anope::string &shost, const Anope::string &svhost,
+ const Anope::string &uip, Server *sserver, const Anope::string &srealname, time_t ts, const Anope::string &smodes,
+ const Anope::string &suid, NickServ::Account *account)
+ : logger(this)
+ , ip(uip)
{
if (snick.empty() || sident.empty() || shost.empty())
throw CoreException("Bad args passed to User::User");
diff --git a/src/xline.cpp b/src/xline.cpp
index ca297a49f..c64761297 100644
--- a/src/xline.cpp
+++ b/src/xline.cpp
@@ -37,10 +37,10 @@ void XLine::Recache()
delete c;
c = nullptr;
- Anope::string mask = GetMask();
- if (mask.length() >= 2 && mask[0] == '/' && mask[mask.length() - 1] == '/' && Config->regex_flags)
+ Anope::string xlmask = GetMask();
+ if (xlmask.length() >= 2 && xlmask[0] == '/' && xlmask[xlmask.length() - 1] == '/' && Config->regex_flags)
{
- Anope::string stripped_mask = mask.substr(1, mask.length() - 2);
+ Anope::string stripped_mask = xlmask.substr(1, xlmask.length() - 2);
try
{
@@ -52,37 +52,37 @@ void XLine::Recache()
}
}
- size_t nick_t = mask.find('!');
+ size_t nick_t = xlmask.find('!');
if (nick_t != Anope::string::npos)
- nick = mask.substr(0, nick_t);
+ nick = xlmask.substr(0, nick_t);
- size_t user_t = mask.find('!'), host_t = mask.find('@');
+ size_t user_t = xlmask.find('!'), host_t = xlmask.find('@');
if (host_t != Anope::string::npos)
{
if (user_t != Anope::string::npos && host_t > user_t)
- user = mask.substr(user_t + 1, host_t - user_t - 1);
+ user = xlmask.substr(user_t + 1, host_t - user_t - 1);
else
- user = mask.substr(0, host_t);
+ user = xlmask.substr(0, host_t);
}
- size_t real_t = mask.find('#');
+ size_t real_t = xlmask.find('#');
if (host_t != Anope::string::npos)
{
if (real_t != Anope::string::npos && real_t > host_t)
- host = mask.substr(host_t + 1, real_t - host_t - 1);
+ host = xlmask.substr(host_t + 1, real_t - host_t - 1);
else
- host = mask.substr(host_t + 1);
+ host = xlmask.substr(host_t + 1);
}
else
{
if (real_t != Anope::string::npos)
- host = mask.substr(0, real_t);
+ host = xlmask.substr(0, real_t);
else
- host = mask;
+ host = xlmask;
}
if (real_t != Anope::string::npos)
- real = mask.substr(real_t + 1);
+ real = xlmask.substr(real_t + 1);
if (host.find('/') != Anope::string::npos)
{
@@ -141,9 +141,9 @@ Anope::string XLine::GetReason()
return Get(&XLineType::reason);
}
-void XLine::SetID(const Anope::string &id)
+void XLine::SetID(const Anope::string &i)
{
- Set(&XLineType::id, id);
+ Set(&XLineType::id, i);
}
Anope::string XLine::GetID()
@@ -206,8 +206,8 @@ bool XLine::HasNickOrReal() const
bool XLine::IsRegex()
{
- Anope::string mask = GetMask();
- return mask.length() > 2 && mask[0] == '/' && mask[mask.length() - 1] == '/';
+ Anope::string m = GetMask();
+ return m.length() > 2 && m[0] == '/' && m[m.length() - 1] == '/';
}
XLineManager *XLine::GetManager()
generated by cgit (git 2.34.1) at 2025-06-16 13:30:33 +0000