]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
ErasureCodeJerasure.cc: prefer prefix ++operator for non-primitive types
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Tue, 5 Nov 2013 18:42:29 +0000 (19:42 +0100)
committerDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Thu, 7 Nov 2013 22:31:14 +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/osd/ErasureCodePluginJerasure/ErasureCodeJerasure.cc

index f2be1ed06e711a1cc142210a4260963470349dd1..fe656e58ee00f3a109b8a4bcff5fe08574711928 100644 (file)
@@ -54,7 +54,7 @@ int ErasureCodeJerasure::minimum_to_decode(const set<int> &want_to_read,
       return -EIO;
     set<int>::iterator i;
     unsigned j;
-    for (i = available_chunks.begin(), j = 0; j < (unsigned)k; i++, j++)
+    for (i = available_chunks.begin(), j = 0; j < (unsigned)k; ++i, j++)
       minimum->insert(*i);
   }
   return 0;
@@ -67,7 +67,7 @@ int ErasureCodeJerasure::minimum_to_decode_with_cost(const set<int> &want_to_rea
   set <int> available_chunks;
   for (map<int, int>::const_iterator i = available.begin();
        i != available.end();
-       i++)
+       ++i)
     available_chunks.insert(i->first);
   return minimum_to_decode(want_to_read, available_chunks, minimum);
 }