aboutsummaryrefslogtreecommitdiff
path: root/src/modules/m_sqloper.cpp
diff options
context:
space:
mode:
authorGravatar Sadie Powell2021-04-06 20:06:18 +0100
committerGravatar Sadie Powell2021-04-07 10:36:11 +0100
commit942fd2bcfd384a12c900999fe663202c87319a68 (patch)
treec2bad1906af27afbc3c7d96c3e5ca3c27c83f090 /src/modules/m_sqloper.cpp
parentMerge branch 'insp3' into master. (diff)
Switch simple iterator loops to use range-based for loops.
Diffstat (limited to 'src/modules/m_sqloper.cpp')
-rw-r--r--src/modules/m_sqloper.cpp12
1 files changed, 4 insertions, 8 deletions
diff --git a/src/modules/m_sqloper.cpp b/src/modules/m_sqloper.cpp
index 7ff0b1759..76e50cedc 100644
--- a/src/modules/m_sqloper.cpp
+++ b/src/modules/m_sqloper.cpp
@@ -56,10 +56,8 @@ class OperQuery : public SQL::Query
ServerConfig::OperIndex& oper_blocks = ServerInstance->Config->oper_blocks;
// Remove our previous blocks from oper_blocks for a clean update
- for (std::vector<std::string>::const_iterator i = my_blocks.begin(); i != my_blocks.end(); ++i)
- {
- oper_blocks.erase(*i);
- }
+ for (const auto& block : my_blocks)
+ oper_blocks.erase(block);
my_blocks.clear();
SQL::Row row;
@@ -198,10 +196,8 @@ public:
~ModuleSQLOper() override
{
// Remove all oper blocks that were from the DB
- for (std::vector<std::string>::const_iterator i = my_blocks.begin(); i != my_blocks.end(); ++i)
- {
- ServerInstance->Config->oper_blocks.erase(*i);
- }
+ for (const auto& block : my_blocks)
+ ServerInstance->Config->oper_blocks.erase(block);
}
ModResult OnPreCommand(std::string& command, CommandBase::Params& parameters, LocalUser* user, bool validated) override