fix -Wshadowing in String.cpp
authorMichael Lee <mzlee@fb.com>
Thu, 11 Feb 2016 04:22:20 +0000 (20:22 -0800)
committerfacebook-github-bot-0 <folly-bot@fb.com>
Thu, 11 Feb 2016 05:20:23 +0000 (21:20 -0800)
Summary: Didn't check this with -Wshadow when I first wrote it.

Reviewed By: ranjeeth

Differential Revision: D2926047

fb-gh-sync-id: 8391986dee2d6b5698491bd2995f039468ec684d
shipit-source-id: 8391986dee2d6b5698491bd2995f039468ec684d

folly/String.cpp

index 6692c78337a11bd161378d6f1ce9a24037b87531..43404026d031324daa76bbf1495f43e00218df00 100644 (file)
@@ -571,7 +571,7 @@ std::string stripLeftMargin(std::string s) {
   const auto sentinel = std::numeric_limits<size_t>::max();
   auto indent = sentinel;
   size_t max_length = 0;
-  for (auto piece = piecer.begin(); piece != piecer.end(); piece++) {
+  for (piece = piecer.begin(); piece != piecer.end(); piece++) {
     needle = std::find_if(piece->begin(),
                           piece->end(),
                           [](char c) { return c != ' ' && c != '\t'; });
@@ -582,11 +582,11 @@ std::string stripLeftMargin(std::string s) {
     }
   }
   indent = indent == sentinel ? max_length : indent;
-  for (auto& piece : piecer) {
-    if (piece.size() < indent) {
-      piece.clear();
+  for (piece = piecer.begin(); piece != piecer.end(); piece++) {
+    if (piece->size() < indent) {
+      piece->clear();
     } else {
-      piece.erase(piece.begin(), piece.begin() + indent);
+      piece->erase(piece->begin(), piece->begin() + indent);
     }
   }
   return join("\n", piecer);