Use pre-increment instead of post-increment for non-primitive types

diff --git a/pdns/misc.hh b/pdns/misc.hh
index df35e855ded836c95393768f89e9f3457ca7976d..bdc535b3aea27ca0f8ff61415005107b0beaf7b0 100644 (file)
--- a/pdns/misc.hh
+++ b/pdns/misc.hh
@@ -378,7 +378,7 @@ struct CIStringComparePOSIX
       while(a!=lhs.end()) {
           if (b==rhs.end() || std::tolower(*b,loc)<std::tolower(*a,loc)) return false;
           else if (std::tolower(*a,loc)<std::tolower(*b,loc)) return true;
-          a++;b++;
+          ++a;++b;
       }
       return (b!=rhs.end());
    }
@@ -479,18 +479,18 @@ public:
  
   bool match(string::const_iterator mi, string::const_iterator mend, string::const_iterator vi, string::const_iterator vend)
   {
-    for(;;mi++) {
+    for(;;++mi) {
       if (mi == mend) {
         return vi == vend;
       } else if (*mi == '?') {
         if (vi == vend) return false;
-        vi++;
+        ++vi;
       } else if (*mi == '*') {
-        while(*mi == '*') mi++;
+        while(*mi == '*') ++mi;
         if (mi == d_mask.end()) return true;
         while(vi != vend) {
           if (match(mi,mend,vi,vend)) return true;
-          vi++;
+          ++vi;
         }
         return false;
       } else {
@@ -501,7 +501,7 @@ public:
         } else {
           if (*mi != *vi) return false;
         }
-        vi++;
+        ++vi;
       }
     }
   }

diff --git a/pdns/pollmplexer.cc b/pdns/pollmplexer.cc
index e155072a77ed233f6bd9991060e8d61a3a308ce1..312c4bfece6919adb7db0e84cf0f50e8014d2064 100644 (file)
--- a/pdns/pollmplexer.cc
+++ b/pdns/pollmplexer.cc
@@ -74,7 +74,7 @@ void PollFDMultiplexer::addFD(callbackmap_t& cbmap, int fd, callbackfunc_t toDo,
 void PollFDMultiplexer::removeFD(callbackmap_t& cbmap, int fd)
 {
   if(d_inrun && d_iter->first==fd)  // trying to remove us!
-    d_iter++;
+    ++d_iter;
 
   if(!cbmap.erase(fd))
     throw FDMultiplexerException("Tried to remove unlisted fd "+std::to_string(fd)+ " from multiplexer");

diff --git a/pdns/statnode.cc b/pdns/statnode.cc
index 2d40f582ad868636cc486acfc9d9398a2abd4d83..1557bbbd6011ad1682f78181188abbf975bf8c07 100644 (file)
--- a/pdns/statnode.cc
+++ b/pdns/statnode.cc
@@ -112,7 +112,7 @@ void StatNode::submit(std::vector<string>::const_iterator end, std::vector<strin
       labelsCount = count;
     }
     //    cerr<<"Not yet end, set our fullname to '"<<fullname<<"', recursing"<<endl;
-    end--;
+    --end;
     children[*end].submit(end, begin, fullname, rcode, remote, count+1);
   }
 }