From f3ce402fa4556063406a4b5dc58432967721dd2a Mon Sep 17 00:00:00 2001 From: Wanlong Gao Date: Wed, 29 Jun 2016 13:57:50 +0800 Subject: [PATCH] kv: RocksDBStore: fix compile warning MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit /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 --- src/kv/RocksDBStore.cc | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/kv/RocksDBStore.cc b/src/kv/RocksDBStore.cc index 826328ea32e48..dbe569bf4b01f 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() { -- 2.39.5