fix: Compiler warning about unused map_or

[MichaReiser]

Summary

Fixes a compile error about an unused map_or return value by rewritting the statement to if let Some(..) = .. {}

Test Plan

cargo test

[MichaReiser]

Current dependencies on/for this PR:

• main
  • PR fix: Compiler warning about unused map_or #7508 👈

This comment was auto-generated by Graphite.

[github-actions]

PR Check Results

Ecosystem

✅ ecosystem check detected no changes.

[mkniewallner]

Sorry for that. Out of curiosity since I'm new to Rust, how could have I had spotted that? Isn't it something that is reported on the CI?

[MichaReiser]

Sorry for that. Out of curiosity since I'm new to Rust, how could have I had spotted that? Isn't it something that is reported on the CI?

No worries. I saw the message when running cargo build or cargo test locally. But I'm also using Rust's nightly compiler. So it's quite possible that the warning message is new and not something you could have seen.

[konstin]

Rust makes it easy to even find the PR where this was introduced: https://github.com/rust-lang/rust/pulls?q=is%3Apr+is%3Aclosed+map_or points to rust-lang/rust#112655. Given this was merged for rust 1.73, we can expect it to become available on stable 2023-10-05 (https://releases.rs/).