]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
FileStoreTracker.cc: use empty() to check for emptiness
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Wed, 13 Feb 2013 18:01:12 +0000 (19:01 +0100)
committerGreg Farnum <greg@inktank.com>
Thu, 14 Feb 2013 18:27:40 +0000 (10:27 -0800)
Use !empty() instead of size() to check for emptiness for
performance reasons.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
src/test/filestore/FileStoreTracker.cc

index 2777a968704da683f52b5ac449737b904cd2b296..afdc31bad23e9cd460b0a98bc38e85784476aec5 100644 (file)
@@ -52,7 +52,7 @@ int FileStoreTracker::init()
   map<string, bufferlist> got;
   db->get("STATUS", to_get, &got);
   restart_seq = 0;
-  if (got.size()) {
+  if (!got.empty()) {
     bufferlist::iterator bp = got.begin()->second.begin();
     ::decode(restart_seq, bp);
   }
@@ -240,7 +240,7 @@ ObjStatus get_obj_status(const pair<string, string> &obj,
   map<string, bufferlist> got;
   db->get(obj_to_meta_prefix(obj), to_get, &got);
   ObjStatus retval;
-  if (got.size()) {
+  if (!got.empty()) {
     bufferlist::iterator bp = got.begin()->second.begin();
     ::decode(retval, bp);
   }
@@ -357,7 +357,7 @@ ObjectContents FileStoreTracker::get_content(
   map<string, bufferlist> got;
   to_get.insert(seq_to_key(version));
   db->get(obj_to_prefix(obj), to_get, &got);
-  if (!got.size())
+  if (got.empty())
     return ObjectContents();
   pair<uint64_t, bufferlist> val;
   bufferlist::iterator bp = got.begin()->second.begin();