From da789f40cb72eb126741e229f4b0906b6b6dff52 Mon Sep 17 00:00:00 2001 From: "Adam C. Emerson" Date: Mon, 7 Mar 2022 18:46:04 -0500 Subject: [PATCH] mon: Fix ranged-construct warning In the case of an uint32_t there's no benefit to a reference so just remove the const and make the copy. Signed-off-by: Adam C. Emerson --- src/mon/OSDMonitor.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mon/OSDMonitor.cc b/src/mon/OSDMonitor.cc index be1de29012b14..db9383e86076d 100644 --- a/src/mon/OSDMonitor.cc +++ b/src/mon/OSDMonitor.cc @@ -899,13 +899,13 @@ void OSDMonitor::update_from_paxos(bool *need_bootstrap) t = MonitorDBStore::TransactionRef(); tx_size = 0; } - for (const auto [osd, state] : inc.new_state) { + for (auto [osd, state] : inc.new_state) { if (state & CEPH_OSD_UP) { // could be marked up *or* down, but we're too lazy to check which last_osd_report.erase(osd); } } - for (const auto [osd, weight] : inc.new_weight) { + for (auto [osd, weight] : inc.new_weight) { if (weight == CEPH_OSD_OUT) { // manually marked out, so drop it osd_epochs.erase(osd); -- 2.39.5