From dc7f10f42c5ea2f996799ff74f6f8c471656ef49 Mon Sep 17 00:00:00 2001 From: Haomai Wang Date: Wed, 21 Sep 2016 11:32:32 +0800 Subject: [PATCH] bluestore/BitmapFreelistManager: remove extra if/else judgement Signed-off-by: Haomai Wang --- src/os/bluestore/BitmapFreelistManager.cc | 20 +++++++++----------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/src/os/bluestore/BitmapFreelistManager.cc b/src/os/bluestore/BitmapFreelistManager.cc index ad651b14574..8de18d8f86d 100644 --- a/src/os/bluestore/BitmapFreelistManager.cc +++ b/src/os/bluestore/BitmapFreelistManager.cc @@ -521,17 +521,15 @@ void BitmapFreelistManager::_xor( first_key += bytes_per_key; } // middle keys - if (first_key < last_key) { - while (first_key < last_key) { - string k; - make_offset_key(first_key, &k); - dout(30) << __func__ << " 0x" << std::hex << first_key << std::dec - << ": "; - all_set_bl.hexdump(*_dout, false); - *_dout << dendl; - txn->merge(bitmap_prefix, k, all_set_bl); - first_key += bytes_per_key; - } + while (first_key < last_key) { + string k; + make_offset_key(first_key, &k); + dout(30) << __func__ << " 0x" << std::hex << first_key << std::dec + << ": "; + all_set_bl.hexdump(*_dout, false); + *_dout << dendl; + txn->merge(bitmap_prefix, k, all_set_bl); + first_key += bytes_per_key; } assert(first_key == last_key); { -- 2.39.5