diff options
| author | attilamolnar <attilamolnar@hush.com> | 2012-07-01 22:22:38 +0200 |
|---|---|---|
| committer | attilamolnar <attilamolnar@hush.com> | 2012-07-07 15:57:35 +0200 |
| commit | 9c5629085ae4914f9db514ca6fbd94e8d58c6f31 (patch) | |
| tree | d2b207eb720710e2e7b0d1b9f12486f3cc9c2d90 | |
| parent | 6c2edc2c5ab07a1fa8c32d3fa9abd6b9149b804c (diff) | |
m_mlock Remove unnecessary iteration
| -rw-r--r-- | src/modules/m_mlock.cpp | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/src/modules/m_mlock.cpp b/src/modules/m_mlock.cpp index e9ca3bfd0..3fb60a3d2 100644 --- a/src/modules/m_mlock.cpp +++ b/src/modules/m_mlock.cpp @@ -54,17 +54,15 @@ public: return MOD_RES_PASSTHRU; std::string *mlock_str = mlock.get(channel); - if (!mlock_str || mlock_str->empty()) + if (!mlock_str) return MOD_RES_PASSTHRU; - for (const char *modes = parameters[1].c_str(); *modes; modes++) + std::string::size_type p = parameters[1].find_first_of(*mlock_str); + if (p != std::string::npos) { - if (mlock_str->find(*modes) != std::string::npos) - { - source->WriteNumeric(742, "%s %c %s :MODE cannot be set due to channel having an active MLOCK restriction policy", - channel->name.c_str(), *modes, mlock_str->c_str()); - return MOD_RES_DENY; - } + source->WriteNumeric(742, "%s %c %s :MODE cannot be set due to channel having an active MLOCK restriction policy", + channel->name.c_str(), parameters[1][p], mlock_str->c_str()); + return MOD_RES_DENY; } return MOD_RES_PASSTHRU; |