crimson,crimson/test: clean-up 'unused' warnings

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>

diff --git a/src/crimson/net/io_handler.cc b/src/crimson/net/io_handler.cc
index c08bdc87422fb2df7078d0458941c00424701ef4..f9e9198c748ea4f1b47e112caef94c58881c4024 100644 (file)
--- a/src/crimson/net/io_handler.cc
+++ b/src/crimson/net/io_handler.cc
@@ -1266,6 +1266,7 @@ IOHandler::close_io(
   } else {
     return shard_states->close(
     ).then([this] {
+      std::ignore = this; // as we are 'assert'ing, not ceph_assert'ing
       assert(shard_states->assert_closed_and_exit());
     });
   }

diff --git a/src/crimson/os/seastore/object_data_handler.cc b/src/crimson/os/seastore/object_data_handler.cc
index a586a606d68423510512b6b5f65bc5d633966501..65e6b9cf47c4884105d88da2c6a5770226f35203 100644 (file)
--- a/src/crimson/os/seastore/object_data_handler.cc
+++ b/src/crimson/os/seastore/object_data_handler.cc
@@ -1453,6 +1453,7 @@ ObjectDataHandler::clear_ret ObjectDataHandler::trim_data_reservation(
     ctx.t, unaligned_begin.get_aligned_laddr(ctx.tm.get_block_size())
   ).si_then([ctx, data_base, size, this,
            unaligned_begin, &object_data](auto mapping) {
+    std::ignore = unaligned_begin;
     assert(mapping.get_key() <= unaligned_begin &&
       mapping.get_key() + mapping.get_length() > unaligned_begin);
     auto data_len = object_data.get_reserved_data_len();

diff --git a/src/crimson/os/seastore/transaction_manager.cc b/src/crimson/os/seastore/transaction_manager.cc
index f0e586220b9ef4f6f4607a843529e2f01b6893e1..62d24945517d96339af6417975d378876d74f5dd 100644 (file)
--- a/src/crimson/os/seastore/transaction_manager.cc
+++ b/src/crimson/os/seastore/transaction_manager.cc
@@ -945,7 +945,7 @@ TransactionManager::move_region(
             ).handle_error_interruptible(
               move_region_iertr::pass_further(),
               crimson::ct_error::assert_all("invalid error"));
-            auto off = 0;
+            [[maybe_unused]] auto off = 0;
             auto bl = maybe_indirect_extent.get_range(
               src.get_intermediate_offset(),
               src.get_length());

diff --git a/src/crimson/osd/ec_backend.cc b/src/crimson/osd/ec_backend.cc
index 368d29c71479ce21f143bc3fc1c88f87f4e78958..4ce37c77920eea886782ebbe11ee4eb9baef4be3 100644 (file)
--- a/src/crimson/osd/ec_backend.cc
+++ b/src/crimson/osd/ec_backend.cc
@@ -457,6 +457,7 @@ ECBackend::handle_rep_write_op(
   return handle_sub_write(
     m->op.from, std::move(m->op), pg
   ).si_then([&pg] {
+    std::ignore = pg;
     assert(!pg.pgb_is_primary());
     return write_iertr::now();
   }, crimson::ct_error::assert_all("unexpected error"));

diff --git a/src/test/crimson/seastore/test_cbjournal.cc b/src/test/crimson/seastore/test_cbjournal.cc
index ffd9c96c938c1477d93a82596efce00958a314d9..c97b771cf3b81b39e8a5d166286ae6bd85cc3a31 100644 (file)
--- a/src/test/crimson/seastore/test_cbjournal.cc
+++ b/src/test/crimson/seastore/test_cbjournal.cc
@@ -249,7 +249,7 @@ struct cbjournal_test_t : public seastar_test_suite_t, JournalTrimmer
             auto &dirty_seq,
             auto &alloc_seq,
             auto last_modified) {
-      bool found = false;
+      [[maybe_unused]] bool found = false;
       for (auto &i : entries) {
        paddr_t base = offsets.write_result.start_seq.offset; 
        rbm_abs_addr addr = convert_paddr_to_abs_addr(base);
@@ -583,7 +583,7 @@ TEST_F(cbjournal_test_t, multiple_submit_at_end)
          }
        });
       }).get();
-    auto old_written_to = get_written_to();
+    [[maybe_unused]] auto old_written_to = get_written_to();
     cbj->close().unsafe_get();
     cbj->replay(
       [](const auto &offsets,