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

Update dependencies for codegen #131

Merged
merged 1 commit into from
Sep 23, 2024
Merged

Conversation

morr0ne
Copy link
Contributor

@morr0ne morr0ne commented Sep 7, 2024

Simple pr that updates the depedencies of the codegen code.

  • tempdir: The tempdir crate has been deprecated and has not seen any updates in over six years. I have replaced it with the tempfile crate, which seems to be the successor to tempdir, as its functionality has been merged into tempfile.

  • bindgen: The bindgen crate has been updated to the latest version. While the update introduces only minor improvements, it reduces the number of dependencies and addresses known security vulnerabilities

The only noticeable changes in the generated code are a version header update and what appears to be a more accurate implementation of Clone for unions.

Although these changes may seem minor, they provide a measurable, albeit small, improvement in compile time, especially in some of my older devices. It also just seems like good practice, I see no reason why this dependencies should not be kept up to date

@sunfishcode sunfishcode merged commit c55d479 into sunfishcode:main Sep 23, 2024
5 checks passed
@sunfishcode
Copy link
Owner

Thanks!

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