From b6e98220589e22e1339a9c9ccc9899ff31448915 Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Thu, 3 Jun 2021 15:28:45 +0800 Subject: [PATCH] test/crimson/seastore: always handle returned future<> this change also silences the [-Wunused-result] warning. Signed-off-by: Kefu Chai --- src/test/crimson/seastore/test_btree_lba_manager.cc | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/test/crimson/seastore/test_btree_lba_manager.cc b/src/test/crimson/seastore/test_btree_lba_manager.cc index 131b7a6d659..11818bdfd05 100644 --- a/src/test/crimson/seastore/test_btree_lba_manager.cc +++ b/src/test/crimson/seastore/test_btree_lba_manager.cc @@ -67,9 +67,8 @@ struct btree_lba_manager_test : [this, t=std::move(t)](auto p) mutable { auto [addr, seq] = p; cache.complete_commit(*t, addr, seq); - lba_manager->complete_transaction(*t); - }, - crimson::ct_error::assert_all{}); + return lba_manager->complete_transaction(*t); + }).handle_error(crimson::ct_error::assert_all{}); } seastar::future<> set_up_fut() final { @@ -137,7 +136,7 @@ struct btree_lba_manager_test : } void submit_test_transaction(test_transaction_t t) { - submit_transaction(std::move(t.t)).get0(); + submit_transaction(std::move(t.t)).get(); test_lba_mappings.swap(t.mappings); } -- 2.39.5