From: Wanlong Gao <wanlong.gao@easystack.cn>
Date: Wed, 29 Jun 2016 05:57:50 +0000 (+0800)
Subject: kv: RocksDBStore: fix compile warning
X-Git-Tag: ses5-milestone5~491^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=f3ce402fa4556063406a4b5dc58432967721dd2a;p=ceph.git

kv: RocksDBStore: fix compile warning

/git/ceph/src/kv/RocksDBStore.cc:700:3: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
   if (valid())
   ^~
/git/ceph/src/kv/RocksDBStore.cc:702:5: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’
     return dbiter->status().ok() ? 0 : -1;
     ^~~~~~

Signed-off-by: Wanlong Gao <wanlong.gao@easystack.cn>
---

diff --git a/src/kv/RocksDBStore.cc b/src/kv/RocksDBStore.cc
index 826328ea32e4..dbe569bf4b01 100644
--- a/src/kv/RocksDBStore.cc
+++ b/src/kv/RocksDBStore.cc
@@ -691,15 +691,17 @@ bool RocksDBStore::RocksDBWholeSpaceIteratorImpl::valid()
 }
 int RocksDBStore::RocksDBWholeSpaceIteratorImpl::next()
 {
-  if (valid())
-  dbiter->Next();
+  if (valid()) {
+    dbiter->Next();
+  }
   return dbiter->status().ok() ? 0 : -1;
 }
 int RocksDBStore::RocksDBWholeSpaceIteratorImpl::prev()
 {
-  if (valid())
+  if (valid()) {
     dbiter->Prev();
-    return dbiter->status().ok() ? 0 : -1;
+  }
+  return dbiter->status().ok() ? 0 : -1;
 }
 string RocksDBStore::RocksDBWholeSpaceIteratorImpl::key()
 {