From: Kefu Chai <kchai@redhat.com>
Date: Mon, 8 Feb 2021 16:35:59 +0000 (+0800)
Subject: crimson/os: disable code block only executed by Debug build
X-Git-Tag: v17.1.0~3020^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=5fcddda37be7ff68b05e90bd5714253380589bc0;p=ceph.git

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 <kchai@redhat.com>
---

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 ed37ae0eb764..fe134ae02cd4 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