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

Rust 2024 #799

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Rust 2024 #799

wants to merge 5 commits into from

Conversation

rkuris
Copy link
Collaborator

@rkuris rkuris commented Feb 20, 2025

Basic changes:

  • Specified minimum version so compiles will fail unless you're on a rust version that supports 2024
  • Some unsafe blocks were not required before, now they are
  • ffi's no_mangle is now unsafe
  • Some lifetime defaults changed
  • Switched from clippy allow to clippy expect, removing some disabling checks
  • Removed some dead code as a result of that

Basic changes:
 - Specified minimum version so compiles will fail unless you're on a
   rust version that supports 2024
 - Some unsafe blocks were not required before, now they are
 - ffi's no_mangle is now unsafe
 - Some lifetime defaults changed
 - Switched from clippy allow to clippy expect, removing some disabling
   checks
 - Removed some dead code as a result of that
@rkuris rkuris self-assigned this Feb 20, 2025
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.

1 participant