From 5fcddda37be7ff68b05e90bd5714253380589bc0 Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Tue, 9 Feb 2021 00:35:59 +0800 Subject: [PATCH] crimson/os: disable code block only executed by Debug build silences warning like: warning: lambda capture 'this' is not used [-Wunused-lambda-capture] ).safe_then([this, target_size] { ^~~~~ Signed-off-by: Kefu Chai --- .../onode_manager/staged-fltree/value.cc | 22 ++++++++++++------- 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/src/crimson/os/seastore/onode_manager/staged-fltree/value.cc b/src/crimson/os/seastore/onode_manager/staged-fltree/value.cc index ed37ae0eb76..fe134ae02cd 100644 --- a/src/crimson/os/seastore/onode_manager/staged-fltree/value.cc +++ b/src/crimson/os/seastore/onode_manager/staged-fltree/value.cc @@ -35,21 +35,27 @@ Value::~Value() {} future<> Value::extend(Transaction& t, value_size_t extend_size) { - auto target_size = get_payload_size() + extend_size; - return p_cursor->extend_value(get_context(t), extend_size - ).safe_then([this, target_size] { + [[maybe_unused]] auto target_size = get_payload_size() + extend_size; + return p_cursor->extend_value(get_context(t), extend_size) +#ifndef NDEBUG + .safe_then([this, target_size] { assert(target_size == get_payload_size()); - }); + }) +#endif + ; } future<> Value::trim(Transaction& t, value_size_t trim_size) { assert(get_payload_size() > trim_size); - auto target_size = get_payload_size() - trim_size; - return p_cursor->trim_value(get_context(t), trim_size - ).safe_then([this, target_size] { + [[maybe_unused]] auto target_size = get_payload_size() - trim_size; + return p_cursor->trim_value(get_context(t), trim_size) +#ifndef NDEBUG + .safe_then([this, target_size] { assert(target_size == get_payload_size()); - }); + }) +#endif + ; } const value_header_t* Value::read_value_header() const -- 2.39.5