FileStore.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/os/FileStore.cc b/src/os/FileStore.cc
index 9ce7550d9a56adfa61a94bb05dee60b9dbe77c29..c91d47c6d0d4af727260ef6fed8db36e048e5187 100644 (file)
--- a/src/os/FileStore.cc
+++ b/src/os/FileStore.cc
@@ -1495,7 +1495,7 @@ int FileStore::mount()
     }
 
     dout(0) << "mount found snaps " << snaps << dendl;
-    if (cluster_snaps.size())
+    if (!cluster_snaps.empty())
       dout(0) << "mount found cluster snaps " << cluster_snaps << dendl;
   }
 
@@ -3751,7 +3751,7 @@ int FileStore::getattr(coll_t cid, const hobject_t& oid, const char *name, buffe
       dout(10) << __func__ << " get_xattrs err r =" << r << dendl;
       goto out;
     }
-    if (!got.size()) {
+    if (got.empty()) {
       dout(10) << __func__ << " got.size() is 0" << dendl;
       return -ENODATA;
     }
@@ -4307,7 +4307,7 @@ bool FileStore::collection_empty(coll_t c)
     assert(!m_filestore_fail_eio || r != -EIO);
     return false;
   }
-  return ls.size() > 0;
+  return !ls.empty();
 }
 
 int FileStore::collection_list_range(coll_t c, hobject_t start, hobject_t end,
@@ -4328,11 +4328,11 @@ int FileStore::collection_list_range(coll_t c, hobject_t start, hobject_t end,
     ls->insert(ls->end(), next_objects.begin(), next_objects.end());
 
     // special case for empty collection
-    if (ls->size() == 0) {
+    if (ls->empty()) {
       break;
     }
 
-    while (ls->size() > 0 && ls->back() >= end) {
+    while (!ls->empty() && ls->back() >= end) {
       ls->pop_back();
       done = true;
     }