Merge pull request #55605 from ronen-fr/wip-rf-warns0224

osd: clean compiler warnings

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
This commit is contained in:
Ronen Friedman 2024-02-16 20:57:45 +02:00 committed by GitHub
commit 0ed764e649
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 2 additions and 8 deletions

View File

@ -10506,7 +10506,7 @@ int PrimaryLogPG::start_dedup(OpRequestRef op, ObjectContextRef obc)
obc_g ? &(obc_g->obs.oi.manifest) : nullptr,
refs);
for (auto p : chunks) {
for (const auto& p : chunks) {
hobject_t target = mop->new_manifest.chunk_map[p.first].oid;
if (refs.find(target) == refs.end()) {
continue;
@ -11534,7 +11534,7 @@ void PrimaryLogPG::submit_log_entries(
}
pgbackend->call_write_ordered(
[this, entries, repop, on_complete]() {
[this, entries, repop, on_complete]() mutable {
ObjectStore::Transaction t;
eversion_t old_last_update = info.last_update;
recovery_state.merge_new_log_entries(
@ -12112,7 +12112,6 @@ int PrimaryLogPG::find_object_context(const hobject_t& oid,
dout(20) << __func__ << " " << soid
<< " snapset " << obc->ssc->snapset
<< dendl;
snapid_t first, last;
auto p = obc->ssc->snapset.clone_snaps.find(soid.snap);
ceph_assert(p != obc->ssc->snapset.clone_snaps.end());
if (p->second.empty()) {

View File

@ -337,11 +337,6 @@ TEST_P(IndepTest, out_progressive) {
if (i > 0) cout << "marked out " << i - 1 << " ";
cout << x << " -> " << out << std::endl;
int num_none = 0;
for (unsigned k=0; k<out.size(); ++k) {
if (out[k] == CRUSH_ITEM_NONE)
num_none++;
}
ASSERT_EQ(0, get_num_dups(out));
// make sure nothing moved