]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
ErasureCodeExample.h: prefer prefix ++operator for non-primitive types
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Tue, 5 Nov 2013 18:51:14 +0000 (19:51 +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/test/osd/ErasureCodeExample.h

index 0fd55187559f4164036b4646764b95c4b689f645..07694ea409aaecf64524097df85785e476b66a3c 100644 (file)
@@ -76,7 +76,7 @@ public:
     set <int> available_chunks;
     for (map<int, int>::const_iterator i = c2c.begin();
         i != c2c.end();
-        i++)
+        ++i)
       available_chunks.insert(i->first);
     return minimum_to_decode(want_to_read, available_chunks, minimum);
   }
@@ -109,7 +109,7 @@ public:
     const bufferptr ptr = out.buffers().front();
     for (set<int>::iterator j = want_to_encode.begin();
          j != want_to_encode.end();
-         j++) {
+         ++j) {
       bufferptr chunk(ptr, (*j) * chunk_length, chunk_length);
       (*encoded)[*j].push_front(chunk);
     }
@@ -125,7 +125,7 @@ public:
     unsigned chunk_length = (*chunks.begin()).second.length();
     for (set<int>::iterator i = want_to_read.begin();
          i != want_to_read.end();
-         i++) {
+         ++i) {
       if (chunks.find(*i) != chunks.end()) {
        //
        // If the chunk is available, just copy the bufferptr pointer
@@ -146,7 +146,7 @@ public:
         bufferptr chunk(chunk_length);
         map<int, bufferlist>::const_iterator k = chunks.begin();
         const char *a = k->second.buffers().front().c_str();
-        k++;
+        ++k;
         const char *b = k->second.buffers().front().c_str();
         for (unsigned j = 0; j < chunk_length; j++) {
           chunk[j] = a[j] ^ b[j];