]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
crimson/os: do not capture unused variables
authorKefu Chai <kchai@redhat.com>
Tue, 2 Mar 2021 11:45:44 +0000 (19:45 +0800)
committerKefu Chai <kchai@redhat.com>
Tue, 2 Mar 2021 11:54:13 +0000 (19:54 +0800)
Signed-off-by: Kefu Chai <kchai@redhat.com>
src/crimson/os/alienstore/alien_store.cc
src/crimson/os/seastore/collection_manager/flat_collection_manager.cc
src/crimson/os/seastore/omap_manager/btree/omap_btree_node_impl.cc

index 31b29d29acba714dca734669e00136d34537723a..5bde60f8fdb538d42d9d8fa745ce90f8544f2996 100644 (file)
@@ -372,7 +372,7 @@ seastar::future<> AlienStore::do_transaction(CollectionRef ch,
            auto c = static_cast<AlienCollection*>(ch.get());
            return store->queue_transaction(c->collection, std::move(txn));
          });
-       }).then([this, &done] (int r) {
+       }).then([&done] (int r) {
          assert(r == 0);
          return done.get_future();
        });
index 35ee8b3375a612d33d0092558b41685680bd0b78..b6591bbc02a5fcdbd620155a5dfa647888ed8bd7 100644 (file)
@@ -32,7 +32,7 @@ FlatCollectionManager::mkfs(Transaction &t)
   logger().debug("FlatCollectionManager: {}", __func__);
   return tm.alloc_extent<CollectionNode>(
     t, L_ADDR_MIN, MIN_FLAT_BLOCK_SIZE
-  ).safe_then([this](auto&& root_extent) {
+  ).safe_then([](auto&& root_extent) {
     coll_root_t coll_root = coll_root_t(
       root_extent->get_laddr(),
       MIN_FLAT_BLOCK_SIZE
@@ -100,14 +100,14 @@ FlatCollectionManager::create(coll_root_t &coll_root, Transaction &t,
       __builtin_unreachable();
     });
   });
-
 }
+
 FlatCollectionManager::list_ret
 FlatCollectionManager::list(const coll_root_t &coll_root, Transaction &t)
 {
   logger().debug("FlatCollectionManager: {}", __func__);
   return get_coll_root(coll_root, t)
-    .safe_then([this, &t] (auto extent) {
+    .safe_then([] (auto extent) {
     return extent->list();
   });
 }
index 1e94e1bb4f61a0a01f1e9b8a460786e538af1c1c..e1ad01926026ae653d98274bc6e282a0a2f81c4f 100644 (file)
@@ -218,12 +218,12 @@ OMapInnerNode::list(
          return omap_load_extent(
            oc, laddr,
            get_meta().depth - 1
-         ).safe_then([&, max_result_size, oc, this] (auto &&extent) {
+         ).safe_then([&, max_result_size, oc] (auto &&extent) {
            return extent->list(
              oc,
              start,
              max_result_size - result.size()
-           ).safe_then([&, max_result_size, this](auto &&child_ret) mutable {
+           ).safe_then([&, max_result_size](auto &&child_ret) mutable {
              auto &[child_complete, child_result] = child_ret;
              if (result.size() && child_result.size()) {
                assert(child_result.begin()->first > result.rbegin()->first);