common/WorkQueue.cc: use !empty() instead of size() to check for emptiness

Use empty() since it should be prefered as it has, following the
standard, a constant time complexity regardless of the containter
type. The same is not guaranteed for size().

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

diff --git a/src/common/WorkQueue.cc b/src/common/WorkQueue.cc
index a7efcc02870d101570753136b966370773d7455e..66ce6dc2d15735df96755b2faf035a32f2c67b45 100644 (file)
--- a/src/common/WorkQueue.cc
+++ b/src/common/WorkQueue.cc
@@ -99,7 +99,7 @@ void ThreadPool::worker(WorkThread *wt)
       break;
     }
 
-    if (!_pause && work_queues.size()) {
+    if (!_pause && !work_queues.empty()) {
       WorkQueue_* wq;
       int tries = work_queues.size();
       bool did = false;