]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
librbd/internal.cc: prefer prefix ++operator for iterators
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Thu, 14 Mar 2013 13:31:04 +0000 (14:31 +0100)
committerDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Thu, 14 Mar 2013 18:18:05 +0000 (19:18 +0100)
Prefer prefix ++operator for iterators, move iterator declaration into
the related for loop.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
src/librbd/internal.cc

index da9201f8b00575d2beb4c0382ccf73d0d2531d23..0afeb0a3ca1db8313e88ac003384fe073004e7c7 100644 (file)
@@ -341,7 +341,7 @@ namespace librbd {
       map<string,bufferlist> m;
       ::decode(header, p);
       ::decode(m, p);
-      for (map<string,bufferlist>::iterator q = m.begin(); q != m.end(); q++) {
+      for (map<string,bufferlist>::iterator q = m.begin(); q != m.end(); ++q) {
        names.push_back(q->first);
       }
     }
@@ -603,9 +603,8 @@ namespace librbd {
     Rados rados(ictx->md_ctx);
     std::list<std::string> pools;
     rados.pool_list(pools);
-    std::list<std::string>::const_iterator it;
     std::set<std::string> children;
-    for (it = pools.begin(); it != pools.end(); it++) {
+    for (std::list<std::string>::const_iterator it = pools.begin(); it != pools.end(); ++it) {
       IoCtx pool_ioctx;
       r = rados.ioctx_create(it->c_str(), pool_ioctx);
       if (r < 0) {