commit 772ff6f1f1d8d98f33666179b074f5539ab873b9
author Eugene Zhuk <eugene.zhuk@gmail.com> Mon Jan 20 23:18:25 2014 -0800
committer Eugene Zhuk <eugene.zhuk@gmail.com> Mon Jan 20 23:18:25 2014 -0800
tree c7734adc0f152a3d6304955ce90bca6b1cb68bf2
parent ceea4cb380890872362e19d3a2a5ae908fe30146

[#4] Cleaned up FastClock background thread.

src/benchmark.cc

diff --git a/src/benchmark.cc b/src/benchmark.cc
index 9e68436..b33f1c9 100644
--- a/src/benchmark.cc
+++ b/src/benchmark.cc
@@ -509,7 +509,9 @@
   }
 
   void BGThread() {
-    do {
+    mutex_lock l(&bg_mutex_);
+    while (!bg_done_)
+    {
       struct timeval tv;
       gettimeofday(&tv, nullptr);
 
@@ -523,13 +525,12 @@
       ts.tv_nsec %= kNumNanosPerSecond;
 
       // NOTE: this should probably be platform specific.
-      mutex_lock l(&bg_mutex_);
       pthread_cond_timedwait(&bg_cond_, &bg_mutex_, &ts);
 
       std::atomic_store(&approx_time_, NowMicros());
       // NOTE: same code but no memory barrier. think on it.
       // base::subtle::Release_Store(&approx_time_, NowMicros());
-    } while (!bg_done_);
+    }
   }
 
   DISALLOW_COPY_AND_ASSIGN(FastClock)