diff options
author | Attila Molnar <attilamolnar@hush.com> | 2014-04-08 17:03:30 +0200 |
---|---|---|
committer | Attila Molnar <attilamolnar@hush.com> | 2014-04-08 17:03:30 +0200 |
commit | 44259095793bfcc9c3de4acf41820b2f8464ab7e (patch) | |
tree | 41d586d40a659eddd33a56fb7e3f3f8fe2e6ee7c | |
parent | 288254e67922a2302b7910464449b31ae41f02b9 (diff) |
m_blockcaps Get rid of needless cast, use unsigned types
-rw-r--r-- | src/modules/m_blockcaps.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/modules/m_blockcaps.cpp b/src/modules/m_blockcaps.cpp index ca34840ea..0a64a75b5 100644 --- a/src/modules/m_blockcaps.cpp +++ b/src/modules/m_blockcaps.cpp @@ -34,7 +34,7 @@ class BlockCaps : public SimpleChannelModeHandler class ModuleBlockCAPS : public Module { BlockCaps bc; - int percent; + unsigned int percent; unsigned int minlen; char capsmap[256]; @@ -63,7 +63,7 @@ public: if (!c->GetExtBanStatus(user, 'B').check(!c->IsModeSet(bc))) { - int caps = 0; + std::string::size_type caps = 0; unsigned int offset = 0; // Ignore the beginning of the text if it's a CTCP ACTION (/me) if (!text.compare(0, 8, "\1ACTION ", 8)) @@ -72,7 +72,7 @@ public: for (std::string::const_iterator i = text.begin() + offset; i != text.end(); ++i) caps += capsmap[(unsigned char)*i]; - if ( ((caps*100)/(int)text.length()) >= percent ) + if (((caps * 100) / text.length()) >= percent) { user->WriteNumeric(ERR_CANNOTSENDTOCHAN, "%s :Your message cannot contain more than %d%% capital letters if it's longer than %d characters", c->name.c_str(), percent, minlen); return MOD_RES_DENY; |