Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: replace match with explicit if-else #125

Closed
wants to merge 1 commit into from
Closed

fix: replace match with explicit if-else #125

wants to merge 1 commit into from

Conversation

alexlnkp
Copy link

this fixes warnings of float not being supported in pattern-matching

although this is uglier than the original, it produces no warnings during build

this fixes warnings of float not being supported in pattern-matching
@valadaptive
Copy link
Owner

There was actually an explicit #[allow(...)] here before, but Rust actually removed the lint warning in more recent versions.

@alexlnkp
Copy link
Author

There was actually an explicit #[allow(...)] here before, but Rust actually removed the lint warning in more recent versions.

So... uhh.. What would be the best solution then?

@valadaptive
Copy link
Owner

Update your Rust version (probably via rustup).

@alexlnkp
Copy link
Author

Update your Rust version (probably via rustup).

Yeah, that worked. Sorry, I'm just more used to package managers doing all the work for me lol

@alexlnkp alexlnkp closed this Jun 28, 2024
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