]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
os/ObjectStore.cc: prefer prefix ++operator for non-primitive types
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Tue, 5 Nov 2013 18:40:47 +0000 (19:40 +0100)
committerDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Thu, 7 Nov 2013 22:31:13 +0000 (23:31 +0100)
Prefer prefix ++operator for non-primitive types like iterators for
performance reasons. Prefix ++/-- operators avoid creating a temporary
copy.

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

index 1a1bbcb0b67deee1152e9d5cc639d3f696361806..327c64167d5cf16aa50c77ff1040ef53d36754c5 100644 (file)
@@ -504,7 +504,7 @@ int ObjectStore::collection_list(coll_t c, vector<hobject_t>& o)
   int ret = collection_list(c, go);
   if (ret == 0) {
     o.reserve(go.size());
-    for (vector<ghobject_t>::iterator i = go.begin(); i != go.end() ; i++)
+    for (vector<ghobject_t>::iterator i = go.begin(); i != go.end() ; ++i)
       o.push_back(i->hobj);
   }
   return ret;
@@ -520,7 +520,7 @@ int ObjectStore::collection_list_partial(coll_t c, hobject_t start,
   if (ret == 0) {
     *next = gnext.hobj;
     ls->reserve(go.size());
-    for (vector<ghobject_t>::iterator i = go.begin(); i != go.end() ; i++)
+    for (vector<ghobject_t>::iterator i = go.begin(); i != go.end() ; ++i)
       ls->push_back(i->hobj);
   }
   return ret;
@@ -534,7 +534,7 @@ int ObjectStore::collection_list_range(coll_t c, hobject_t start, hobject_t end,
   int ret = collection_list_range(c, gstart, gend, seq, &go);
   if (ret == 0) {
     ls->reserve(go.size());
-    for (vector<ghobject_t>::iterator i = go.begin(); i != go.end() ; i++)
+    for (vector<ghobject_t>::iterator i = go.begin(); i != go.end() ; ++i)
       ls->push_back(i->hobj);
   }
   return ret;