folly/io/async/test/EventBaseTest.cpp: doubly-nested for-loop, do not reuse index
authorJim Meyering <meyering@fb.com>
Mon, 19 Dec 2016 17:30:11 +0000 (09:30 -0800)
committerFacebook Github Bot <facebook-github-bot@users.noreply.github.com>
Mon, 19 Dec 2016 17:32:58 +0000 (09:32 -0800)
Summary:
Fix a warning exposed by gcc's -Wshadow-compatible-local option.
Rename inner "i" to "j".

Reviewed By: knekritz

Differential Revision: D4346505

fbshipit-source-id: f0a4ac32fefd6614900e58d1c8c7f15e8934fd04

folly/io/async/test/EventBaseTest.cpp

index 4fa84ed7eeec3004b7651ddd023e213dd80837a8..1ca9d2d25e98fd5bb49e63663ed4f3f0f38873cb 100644 (file)
@@ -1838,7 +1838,7 @@ TEST(EventBaseTest, LoopKeepAliveAtomic) {
   for (size_t i = 0; i < kNumThreads; ++i) {
     ts.emplace_back([ evbPtr = evb.get(), batonPtr = batons[i].get(), &done ] {
       std::vector<EventBase::LoopKeepAlive> keepAlives;
-      for (size_t i = 0; i < kNumTasks; ++i) {
+      for (size_t j = 0; j < kNumTasks; ++j) {
         keepAlives.emplace_back(evbPtr->loopKeepAliveAtomic());
       }