ErasureCodeLrc.cc: prefer ++operator for non-primitive iterators

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>

diff --git a/src/erasure-code/lrc/ErasureCodeLrc.cc b/src/erasure-code/lrc/ErasureCodeLrc.cc
index 249f9f7a5ac0e68c7acca364242a2119d45e2c64..2dd8fa9c12d1640ee69bb3cf0552afc6ef5403f0 100644 (file)
--- a/src/erasure-code/lrc/ErasureCodeLrc.cc
+++ b/src/erasure-code/lrc/ErasureCodeLrc.cc
@@ -129,7 +129,7 @@ int ErasureCodeLrc::layers_parse(string description_string,
   int position = 0;
   for (vector<json_spirit::mValue>::iterator i = description.begin();
        i != description.end();
-       i++, position++) {
+       ++i, position++) {
     if (i->type() != json_spirit::array_type) {
       stringstream json_string;
       json_spirit::write(*i, json_string);
@@ -417,7 +417,7 @@ int ErasureCodeLrc::parse_ruleset(const map<string,string> &parameters,
     int position = 0;
     for (vector<json_spirit::mValue>::iterator i = description.begin();
         i != description.end();
-        i++, position++) {
+        ++i, position++) {
       if (i->type() != json_spirit::array_type) {
        stringstream json_string;
        json_spirit::write(*i, json_string);
@@ -447,7 +447,7 @@ int ErasureCodeLrc::parse_ruleset_step(string description_string,
   int position = 0;
   for (vector<json_spirit::mValue>::iterator i = description.begin();
        i != description.end();
-       i++, position++) {
+       ++i, position++) {
     if ((position == 0 || position == 1) &&
        i->type() != json_spirit::str_type) {
       *ss << "element " << position << " of the array "
@@ -628,7 +628,7 @@ int ErasureCodeLrc::minimum_to_decode(const set<int> &want_to_read,
          //
          for (set<int>::const_iterator j = erasures.begin();
               j != erasures.end();
-              j++) {
+              ++j) {
            erasures_not_recovered.erase(*j);
            if (erasures_want.count(*j))
              erasures_want.erase(*j);
@@ -641,7 +641,7 @@ int ErasureCodeLrc::minimum_to_decode(const set<int> &want_to_read,
       minimum->insert(want_to_read.begin(), want_to_read.end());
       for (set<int>::const_iterator i = erasures_total.begin();
           i != erasures_total.end();
-          i++) {
+          ++i) {
        if (minimum->count(*i))
          minimum->erase(*i);
       }
@@ -688,7 +688,7 @@ int ErasureCodeLrc::minimum_to_decode(const set<int> &want_to_read,
        //
        for (set<int>::const_iterator j = layer_erasures.begin();
             j != layer_erasures.end();
-            j++) {
+            ++j) {
          erasures_total.erase(*j);
        }
       }
@@ -730,7 +730,7 @@ int ErasureCodeLrc::encode_chunks(const set<int> &want_to_encode,
     int j = 0;
     for (vector<int>::const_iterator c = layer.chunks.begin();
         c != layer.chunks.end();
-        c++) {
+        ++c) {
       layer_encoded[j] = (*encoded)[*c];
       if (want_to_encode.find(*c) != want_to_encode.end())
        layer_want_to_encode.insert(j);
@@ -783,7 +783,7 @@ int ErasureCodeLrc::decode_chunks(const set<int> &want_to_read,
       int j = 0;
       for (vector<int>::const_iterator c = layer->chunks.begin();
           c != layer->chunks.end();
-          c++) {
+          ++c) {
        //
        // Pick chunks from *decoded* instead of *chunks* to re-use
        // chunks recovered by previous layers. In other words
@@ -809,7 +809,7 @@ int ErasureCodeLrc::decode_chunks(const set<int> &want_to_read,
       j = 0;
       for (vector<int>::const_iterator c = layer->chunks.begin();
           c != layer->chunks.end();
-          c++) {
+          ++c) {
        (*decoded)[*c] = layer_decoded[j];
        ++j;
        if (erasures.count(*c) != 0)