clippy: map -> inspect when original item is returned

[muzarski]

Fixing a clippy lints regarding a usage of map() instead of inspect(), when original item (closure argument) is returned from the closure.

Pre-review checklist

• [x] I have split my patch into logically separate commits.
• [x] All commit messages clearly explain what they change and why.
• ~[ ] I added relevant tests for new features and bug fixes.~
• ~[ ] All commits compile, pass static checks and pass test.~
• [x] PR description sums up the changes and reasons why they should be introduced.
• ~[ ] I have provided docstrings for the public items that I want to introduce.~
• ~[ ] I have adjusted the documentation in ./docs/source/.~
• ~[ ] I added appropriate Fixes: annotations to PR description.~

[github-actions[bot]]

cargo semver-checks found no API-breaking changes in this PR! 🎉🥳 Checked commit: 0314f5341f77a95422e0a488b13d67460973e080

[muzarski]

min_rust is not passing unfortunately. Should we simply ignore these clippy lints (#[allow(clippy::manual_inspect)]) or bump MSRV?

It would require to bump MSRV to 1.76.0 (current is 1.66.0). I think ignoring lints is more reasonable.

[Lorak-mmk]

The error that I see in CI is that inspect is unstable on MSRV version. In that case you can stop using it: let socket = socket.connect(real_addr).await; /* do something with socket */; socket.

[muzarski]

The error that I see in CI is that inspect is unstable on MSRV version. In that case you can stop using it: let socket = socket.connect(real_addr).await; /* do something with socket */; socket.

Done