Skip to content

Fix clippy lints - #78

Merged
hsivonen merged 4 commits into
hsivonen:mainfrom
nico-abram:clippy
Aug 6, 2026
Merged

Fix clippy lints#78
hsivonen merged 4 commits into
hsivonen:mainfrom
nico-abram:clippy

Conversation

@nico-abram

Copy link
Copy Markdown
Contributor

This PR fixes clippy lints that showed up on clippy 1.56.1

Lints that showed up multiple times were:

  • Add clippy:: prefix to lint allow()s
  • Use matches!
  • Replace range check with (a..b).contains()
  • Remove unnecessary 'static from statics

@hsivonen

hsivonen commented Apr 3, 2024

Copy link
Copy Markdown
Owner

I acknowledge that I have seen this, but I haven't had the time to evaluate this for performance impact.

@Nopey Nopey mentioned this pull request Aug 3, 2025
@hsivonen
hsivonen merged commit 8c2f7ab into hsivonen:main Aug 6, 2026
4 checks passed
@hsivonen

hsivonen commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants