From 0d467900407b10a3425bc0aa56fa18e381ddfbc6 Mon Sep 17 00:00:00 2001 From: Danny Al-Gaaf Date: Tue, 3 Feb 2015 23:02:59 +0100 Subject: [PATCH] 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 --- src/mds/MDS.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/mds/MDS.cc b/src/mds/MDS.cc index 412e62acfcf..2528f32e9e6 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; } -- 2.47.3