]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
crimson/os: disable code block only executed by Debug build 39345/head
authorKefu Chai <kchai@redhat.com>
Mon, 8 Feb 2021 16:35:59 +0000 (00:35 +0800)
committerKefu Chai <kchai@redhat.com>
Tue, 9 Feb 2021 03:38:22 +0000 (11:38 +0800)
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>
src/crimson/os/seastore/onode_manager/staged-fltree/value.cc

index ed37ae0eb764f28114eb971b7223251f3a0b7354..fe134ae02cd495ee9324503a99357d5d569b219f 100644 (file)
@@ -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