From 7cbc54096d8b65daee70367faf8123f864b9d031 Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Mon, 21 Dec 2020 00:50:50 +0800 Subject: [PATCH] crimson/osd: declare variables captured by do_with() using "auto" auto& is just confusing, as `seastar::do_with()` pass the captureud variables using `std::foward`. so these variable would be of the same types as how they are passed to `do_with()`. Signed-off-by: Kefu Chai --- src/crimson/osd/replicated_recovery_backend.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/crimson/osd/replicated_recovery_backend.cc b/src/crimson/osd/replicated_recovery_backend.cc index 70c268e83af..0812003bb31 100644 --- a/src/crimson/osd/replicated_recovery_backend.cc +++ b/src/crimson/osd/replicated_recovery_backend.cc @@ -357,7 +357,7 @@ seastar::future ReplicatedRecoveryBackend::build_push_op( recovery_info.version, PushOp(), [this, &recovery_info, &progress, stat] - (auto& new_progress, auto& available, auto& v, auto& pop) { + (auto new_progress, auto available, auto v, auto pop) { return read_metadata_for_push_op(recovery_info.soid, progress, new_progress, v, &pop).then([&](eversion_t local_ver) mutable { -- 2.47.3