From f9c24bc2412b159b3cc153e30b4b94d37ca2d297 Mon Sep 17 00:00:00 2001 From: Samuel Just Date: Tue, 24 Aug 2021 09:29:22 -0700 Subject: [PATCH] crimson/os/seastore: fix some clang capture warnings Signed-off-by: Samuel Just --- .../os/seastore/onode_manager/staged-fltree/tree_utils.h | 2 ++ src/crimson/tools/store_nbd/tm_driver.cc | 2 +- src/tools/crimson/perf_crimson_msgr.cc | 2 +- 3 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/crimson/os/seastore/onode_manager/staged-fltree/tree_utils.h b/src/crimson/os/seastore/onode_manager/staged-fltree/tree_utils.h index 9015520917b7f..e061cf8c53dae 100644 --- a/src/crimson/os/seastore/onode_manager/staged-fltree/tree_utils.h +++ b/src/crimson/os/seastore/onode_manager/staged-fltree/tree_utils.h @@ -419,6 +419,8 @@ class TreeBuilder { auto ref_kv_iter = seastar::make_lw_shared(); auto cursors = seastar::make_lw_shared>(); return eagain_iertr::now().si_then([&t, this, cursors, ref_kv_iter] { + (void)this; // silence clang warning for !TRACK + (void)t; // silence clang warning for !TRACK if constexpr (TRACK) { logger().info("Tracking cursors before erase ..."); *ref_kv_iter = kvs.begin(); diff --git a/src/crimson/tools/store_nbd/tm_driver.cc b/src/crimson/tools/store_nbd/tm_driver.cc index e5c19a70ade23..8f4acf0b917fa 100644 --- a/src/crimson/tools/store_nbd/tm_driver.cc +++ b/src/crimson/tools/store_nbd/tm_driver.cc @@ -72,7 +72,7 @@ TMDriver::read_extents_ret TMDriver::read_extents( return tm->pin_to_extent( t, std::move(pin) - ).si_then([this, &ret](auto ref) mutable { + ).si_then([&ret](auto ref) mutable { ret.push_back(std::make_pair(ref->get_laddr(), ref)); logger().debug( "read_extents: got extent {}", diff --git a/src/tools/crimson/perf_crimson_msgr.cc b/src/tools/crimson/perf_crimson_msgr.cc index 35de1d6e73d7c..2455582e7af33 100644 --- a/src/tools/crimson/perf_crimson_msgr.cc +++ b/src/tools/crimson/perf_crimson_msgr.cc @@ -535,7 +535,7 @@ static seastar::future<> run( if (client.is_active()) { client.do_dispatch_messages(client.active_conn.get()); } - }).then([this, ramptime] { + }).then([ramptime] { logger().info("[all clients]: ramping up {} seconds...", ramptime); return seastar::sleep(std::chrono::seconds(ramptime)); }).then([this] { -- 2.39.5