]> git.ipfire.org Git - thirdparty/pdns.git/commitdiff
rec: prefer prefix ++/-- operators for non-primitive types
authorCharles-Henri Bruyand <charles-henri.bruyand@open-xchange.com>
Tue, 27 Mar 2018 15:49:52 +0000 (17:49 +0200)
committerCharles-Henri Bruyand <charles-henri.bruyand@open-xchange.com>
Fri, 30 Mar 2018 11:10:12 +0000 (13:10 +0200)
pdns/arguments.cc
pdns/recursordist/negcache.cc
pdns/resolver.cc
pdns/zoneparser-tng.cc

index 5e62d41706fdd4e73f0a8e87212f7cf377eddb65..2696ac3c8dea87b98ec6bdd6cfaa5d816ad1a278 100644 (file)
@@ -100,7 +100,7 @@ bool ArgvMap::contains(const string &var, const string &val)
   vector<string>::const_iterator i;
   
   stringtok( parts, param->second, ", \t" );
-  for( i = parts.begin(); i != parts.end(); i++ ) {
+  for( i = parts.begin(); i != parts.end(); ++i ) {
     if( *i == val ) {
       return true;
     }
@@ -118,7 +118,7 @@ string ArgvMap::helpstring(string prefix)
   
   for(map<string,string>::const_iterator i=helpmap.begin();
       i!=helpmap.end();
-      i++)
+      ++i)
     {
       if(!prefix.empty() && i->first.find(prefix) != 0) // only print items with prefix
         continue;
@@ -154,7 +154,7 @@ string ArgvMap::configstring(bool current)
   else
     help="# Autogenerated configuration file template\n";
   
-  for(map<string,string>::const_iterator i=helpmap.begin(); i!=helpmap.end(); i++) {
+  for(map<string,string>::const_iterator i=helpmap.begin(); i!=helpmap.end(); ++i) {
     if(d_typeMap[i->first]=="Command")
       continue;
 
index 573ee1e0816ebee10f8a72a237d2fa028838670f..7b9d33645ab8f6d68af59f0e9b62b9ad0c43b5a9 100644 (file)
@@ -56,7 +56,7 @@ bool NegCache::getRootNXTrust(const DNSName& qname, const struct timeval& now, N
       return true;
     }
     moveCacheItemToFront(d_negcache, ni);
-    ni++;
+    ++ni;
   }
   return false;
 }
@@ -87,7 +87,7 @@ bool NegCache::get(const DNSName& qname, const QType& qtype, const struct timeva
       // expired
       moveCacheItemToFront(d_negcache, ni);
     }
-    ni++;
+    ++ni;
   }
   return false;
 }
index b839f1fd99bdd686e30c8a2bbadae2e1ee3d3e26..82b1df6785711013f616a1de6e28c05d049e37a6 100644 (file)
@@ -115,7 +115,7 @@ Resolver::Resolver()
 
 Resolver::~Resolver()
 {
-  for(std::map<std::string,int>::iterator iter = locals.begin(); iter != locals.end(); iter++) {
+  for(std::map<std::string,int>::iterator iter = locals.begin(); iter != locals.end(); ++iter) {
     if (iter->second >= 0)
       close(iter->second);
   }
@@ -222,7 +222,7 @@ bool Resolver::tryGetSOASerial(DNSName *domain, ComboAddress* remote, uint32_t *
   size_t i = 0, k;
   int sock;
 
-  for(std::map<string,int>::iterator iter=locals.begin(); iter != locals.end(); iter++, i++) {
+  for(std::map<string,int>::iterator iter=locals.begin(); iter != locals.end(); ++iter, ++i) {
     fds[i].fd = iter->second;
     fds[i].events = POLLIN;
   }
index 58ee2e4fca9ed5b7f22c3306413535110cc0698f..2c38e912d2ff202e820115e5ab80fd8796dd0cd0 100644 (file)
@@ -523,7 +523,7 @@ bool ZoneParserTNG::getLine()
   if (d_zonedata.size() > 0) {
     if (d_zonedataline != d_zonedata.end()) {
       d_line = *d_zonedataline;
-      d_zonedataline++;
+      ++d_zonedataline;
       return true;
     }
     return false;