Fix 2 clang warnings reported by @SaberUK

diff --git a/src/modules.cpp b/src/modules.cpp
index 5f1081a1c529ad935466857116918dd4ef6215b2..ccd78a8f028c63d6790c04fa499a50c5d2ae9696 100644 (file)
--- a/src/modules.cpp
+++ b/src/modules.cpp
@@ -313,7 +313,7 @@ swap_now:
 
                for (unsigned int j = my_pos; j != swap_pos; j += incrmnt)
                {
-                       if (( j + incrmnt > EventHandlers[i].size() - 1) || (j + incrmnt < 0))
+                       if ((j + incrmnt > EventHandlers[i].size() - 1) || ((incrmnt == -1) && (j == 0)))
                                continue;
 
                        std::swap(EventHandlers[i][j], EventHandlers[i][j+incrmnt]);

diff --git a/src/modules/m_stripcolor.cpp b/src/modules/m_stripcolor.cpp
index aab506fdcb9bee24fabd84b74fa4ef1d931d292e..cd4f35c4b11899982a5805b2689f460120f31bde 100644 (file)
--- a/src/modules/m_stripcolor.cpp
+++ b/src/modules/m_stripcolor.cpp
@@ -75,7 +75,7 @@ class ModuleStripColor : public Module
                std::string::iterator i,safei;
                for (i = sentence.begin(); i != sentence.end();)
                {
-                       if ((*i == 3))
+                       if (*i == 3)
                                seq = 1;
                        else if (seq && (( ((*i >= '0') && (*i <= '9')) || (*i == ',') ) ))
                        {