From: Kefu Chai Date: Sat, 15 Aug 2020 11:05:46 +0000 (+0800) Subject: crimson/osd: do not capture unused variables X-Git-Tag: wip-pdonnell-testing-20200918.022351~358^2 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=446cc601a282dcd1b7013fda58854ad81afcca74;p=ceph-ci.git crimson/osd: do not capture unused variables this change silences warnings like src/crimson/osd/pg_backend.cc:232:34: warning: lambda capture 'this' is not used [-Wunused-lambda-capture] [&os, &osd_op, &extents, this](auto&& bl) -> read_errorator::future<> { Signed-off-by: Kefu Chai --- diff --git a/src/crimson/osd/pg_backend.cc b/src/crimson/osd/pg_backend.cc index 72e49451946..3a35ffc2f2c 100644 --- a/src/crimson/osd/pg_backend.cc +++ b/src/crimson/osd/pg_backend.cc @@ -229,7 +229,7 @@ PGBackend::sparse_read(const ObjectState& os, OSDOp& osd_op) [&os, &osd_op, this](auto&& extents) { return store->readv(coll, ghobject_t{os.oi.soid}, extents, osd_op.op.flags).safe_then( - [&os, &osd_op, &extents, this](auto&& bl) -> read_errorator::future<> { + [&os, &osd_op, &extents](auto&& bl) -> read_errorator::future<> { if (_read_verify_data(os.oi, bl)) { osd_op.op.extent.length = bl.length(); // re-encode since it might be modified diff --git a/src/crimson/osd/replicated_recovery_backend.cc b/src/crimson/osd/replicated_recovery_backend.cc index 9f97eea4579..6299957eea8 100644 --- a/src/crimson/osd/replicated_recovery_backend.cc +++ b/src/crimson/osd/replicated_recovery_backend.cc @@ -459,7 +459,7 @@ seastar::future ReplicatedRecoveryBackend::build_push_op recovery_info.copy_subset, progress.data_recovered_to, available, pop); - }).then([this, &recovery_info, &v, &progress, &new_progress, stat, pop] + }).then([&recovery_info, &v, &progress, &new_progress, stat, pop] (uint64_t recovered_to) { new_progress.data_recovered_to = recovered_to; if (new_progress.is_complete(recovery_info)) {