From f659ee5ff152b05c8c5956d3ca2a4ab6fb8b2cc8 Mon Sep 17 00:00:00 2001
From: Craig Tiller <ctiller@google.com>
Date: Tue, 18 Apr 2017 17:07:16 -0700
Subject: [PATCH] Move comment
---
src/cpp/thread_manager/thread_manager.cc | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/cpp/thread_manager/thread_manager.cc b/src/cpp/thread_manager/thread_manager.cc
index ebcc4dd378..a463a4388a 100644
--- a/src/cpp/thread_manager/thread_manager.cc
+++ b/src/cpp/thread_manager/thread_manager.cc
@@ -159,15 +159,15 @@ void ThreadManager::MainWorkLoop() {
// Take the lock again to check post conditions
lock.lock();
// If we're shutdown, we should finish at this point.
- // If not, there's a chance that we'll exceed the max poller count: that
- // is explicitly ok - we'll decrease after one poll timeout, and prevent
- // some thrashing starting up and shutting down threads
if (shutdown_) done = true;
break;
}
// If we decided to finish the thread, break out of the while loop
if (done) break;
// ... otherwise increase poller count and continue
+ // There's a chance that we'll exceed the max poller count: that is
+ // explicitly ok - we'll decrease after one poll timeout, and prevent
+ // some thrashing starting up and shutting down threads
num_pollers_++;
};
--
GitLab