Add ConfigTag::getUInt for reading unsigned config values.

[user/henk/code/inspircd.git] / src / modules / m_blockcaps.cpp

diff --git a/src/modules/m_blockcaps.cpp b/src/modules/m_blockcaps.cpp
index c36eeabffd1711fc789304550fe296aaa220c740..b79e126a3e5658b0f8cb03f0eb295b7d585eb36f 100644 (file)
--- a/src/modules/m_blockcaps.cpp
+++ b/src/modules/m_blockcaps.cpp
@@ -44,14 +44,14 @@ public:
                tokens["EXTBAN"].push_back('B');
        }
 
-       ModResult OnUserPreMessage(User* user, void* dest, int target_type, std::string& text, char status, CUList& exempt_list, MessageType msgtype) CXX11_OVERRIDE
+       ModResult OnUserPreMessage(User* user, const MessageTarget& target, MessageDetails& details) CXX11_OVERRIDE
        {
-               if (target_type == TYPE_CHANNEL)
+               if (target.type == MessageTarget::TYPE_CHANNEL)
                {
                        if (!IS_LOCAL(user))
                                return MOD_RES_PASSTHRU;
 
-                       Channel* c = (Channel*)dest;
+                       Channel* c = target.Get<Channel>();
                        ModResult res = CheckExemption::Call(exemptionprov, user, c, "blockcaps");
 
                        if (res == MOD_RES_ALLOW)
@@ -61,17 +61,17 @@ public:
                        {
                                // If the message is a CTCP then we skip it unless it is
                                // an ACTION in which case we strip the prefix and suffix.
-                               std::string::const_iterator text_begin = text.begin();
-                               std::string::const_iterator text_end = text.end();
-                               if (text[0] == '\1')
+                               std::string::const_iterator text_begin = details.text.begin();
+                               std::string::const_iterator text_end = details.text.end();
+                               if (details.text[0] == '\1')
                                {
                                        // If the CTCP is not an action then skip it.
-                                       if (text.compare(0, 8, "\1ACTION ", 8))
+                                       if (details.text.compare(0, 8, "\1ACTION ", 8))
                                                return MOD_RES_PASSTHRU;
 
                                        // Skip the CTCP message characters.
                                        text_begin += 8;
-                                       if (*text.rbegin() == '\1')
+                                       if (*details.text.rbegin() == '\1')
                                                text_end -= 1;
                                }
 
@@ -86,9 +86,10 @@ public:
                                size_t upper = 0;
                                for (std::string::const_iterator iter = text_begin; iter != text_end; ++iter)
                                {
-                                       if (uppercase.test(*iter))
+                                       unsigned char chr = static_cast<unsigned char>(*iter);
+                                       if (uppercase.test(chr))
                                                upper += 1;
-                                       else if (!lowercase.test(*iter))
+                                       else if (!lowercase.test(chr))
                                                length -= 1;
                                }
 
@@ -108,18 +109,18 @@ public:
        void ReadConfig(ConfigStatus& status) CXX11_OVERRIDE
        {
                ConfigTag* tag = ServerInstance->Config->ConfValue("blockcaps");
-               percent = tag->getInt("percent", 100, 1, 100);
-               minlen = tag->getInt("minlen", 1, 1, ServerInstance->Config->Limits.MaxLine);
+               percent = tag->getUInt("percent", 100, 1, 100);
+               minlen = tag->getUInt("minlen", 1, 1, ServerInstance->Config->Limits.MaxLine);
 
                lowercase.reset();
                const std::string lower = tag->getString("lowercase", "abcdefghijklmnopqrstuvwxyz");
                for (std::string::const_iterator iter = lower.begin(); iter != lower.end(); ++iter)
-                       lowercase.set(*iter);
+                       lowercase.set(static_cast<unsigned char>(*iter));
 
                uppercase.reset();
                const std::string upper = tag->getString("uppercase", tag->getString("capsmap", "ABCDEFGHIJKLMNOPQRSTUVWXYZ"));
                for (std::string::const_iterator iter = upper.begin(); iter != upper.end(); ++iter)
-                       uppercase.set(*iter);
+                       uppercase.set(static_cast<unsigned char>(*iter));
        }
 
        Version GetVersion() CXX11_OVERRIDE