From: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Date: Tue, 3 Feb 2015 22:02:59 +0000 (+0100)
Subject: mds/MDS.cc: don't compare integer with bool
X-Git-Tag: suse_latest~36^2~9^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=0d467900407b10a3425bc0aa56fa18e381ddfbc6;p=ceph.git

mds/MDS.cc: don't compare integer with bool

No need to use an extra variable to check the result of
a function returning bool if the result isn't used again.

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

diff --git a/src/mds/MDS.cc b/src/mds/MDS.cc
index 412e62acfcf8..2528f32e9e64 100644
--- a/src/mds/MDS.cc
+++ b/src/mds/MDS.cc
@@ -2894,8 +2894,7 @@ bool MDS::ms_verify_authorizer(Connection *con, int peer_type,
 
       dout(10) << __func__ << ": parsing auth_cap_str='" << auth_cap_str << "'" << dendl;
       std::ostringstream errstr;
-      int parse_success = s->auth_caps.parse(auth_cap_str, &errstr);
-      if (parse_success == false) {
+      if (!s->auth_caps.parse(auth_cap_str, &errstr)) {
         dout(1) << __func__ << ": auth cap parse error: " << errstr.str()
           << " parsing '" << auth_cap_str << "'" << dendl;
       }