From: Sage Weil <sage.weil@dreamhost.com>
Date: Fri, 22 Jul 2011 15:51:04 +0000 (-0700)
Subject: mds: cleanup-let: use is_leased() for lease revocation checks
X-Git-Tag: v0.32~33
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=ee20a8d98f9fc8c88214c91e5752e7b60e5792e9;p=ceph.git

mds: cleanup-let: use is_leased() for lease revocation checks

And add const-ness.

Signed-off-by: Sage Weil <sage.weil@dreamhost.com>
---

diff --git a/src/mds/Locker.cc b/src/mds/Locker.cc
index 7a61d157996a..642caa83b602 100644
--- a/src/mds/Locker.cc
+++ b/src/mds/Locker.cc
@@ -3345,7 +3345,7 @@ void Locker::simple_lock(SimpleLock *lock, bool *need_issue)
   }
 
   int gather = 0;
-  if (lock->get_num_client_lease()) {
+  if (lock->is_leased()) {
     gather++;
     revoke_client_leases(lock);
   }
@@ -4013,7 +4013,7 @@ void Locker::scatter_mix(ScatterLock *lock, bool *need_issue)
 	gather++;
       }
     }
-    if (lock->get_num_client_lease()) {
+    if (lock->is_leased()) {
       revoke_client_leases(lock);
       gather++;
     }
@@ -4081,7 +4081,7 @@ void Locker::file_excl(ScatterLock *lock, bool *need_issue)
     lock->init_gather();
     gather++;
   }
-  if (lock->get_num_client_lease()) {
+  if (lock->is_leased()) {
     revoke_client_leases(lock);
     gather++;
   }
diff --git a/src/mds/SimpleLock.h b/src/mds/SimpleLock.h
index bfed8f3b4623..75cf370f8c5f 100644
--- a/src/mds/SimpleLock.h
+++ b/src/mds/SimpleLock.h
@@ -530,8 +530,10 @@ public:
       try_clear_more();
     }
   }
-  bool is_leased() const { return num_client_lease > 0; }
-  int get_num_client_lease() {
+  bool is_leased() const {
+    return num_client_lease > 0;
+  }
+  int get_num_client_lease() const {
     return num_client_lease;
   }