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

Allow the unpredictable_function_pointer_comparisons where needed #4177

Merged
merged 1 commit into from
Dec 7, 2024

Conversation

tgross35
Copy link
Contributor

@tgross35 tgross35 commented Dec 7, 2024

This lint was recently added so this change is needed to fix CI. The suggested alternative is to use ptr::fn_addr_eq which isn't available until 1.85, so allow the lint here.

This is in a PartialEq for a union so hopefully this code should go away in 1.0 anyway.

@rustbot
Copy link
Collaborator

rustbot commented Dec 7, 2024

r? @JohnTitor

rustbot has assigned @JohnTitor.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

This lint was recently added so this change is needed to fix CI. The
suggested alternative is to use `ptr::fn_addr_eq` which isn't available
until 1.85, so allow the lint here.
@tgross35 tgross35 changed the title Allow the unpredictable_function_pointer_comparisons in one place Allow the unpredictable_function_pointer_comparisons where needed Dec 7, 2024
@tgross35 tgross35 enabled auto-merge December 7, 2024 02:17
@tgross35 tgross35 mentioned this pull request Dec 7, 2024
3 tasks
@tgross35 tgross35 added this pull request to the merge queue Dec 7, 2024
@tgross35 tgross35 added the stable-nominated This PR should be considered for cherry-pick to libc's stable release branch label Dec 7, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 7, 2024
@tgross35 tgross35 added this pull request to the merge queue Dec 7, 2024
Merged via the queue into rust-lang:main with commit ad7bc97 Dec 7, 2024
45 checks passed
@tgross35 tgross35 deleted the fix-new-lint branch December 7, 2024 03:24
tgross35 added a commit to tgross35/rust-libc that referenced this pull request Dec 9, 2024
This lint was recently added so this change is needed to fix CI. The
suggested alternative is to use `ptr::fn_addr_eq` which isn't available
until 1.85, so allow the lint here.

(backport <rust-lang#4177>)
(cherry picked from commit 99f4dd9)
@tgross35 tgross35 mentioned this pull request Dec 9, 2024
@tgross35 tgross35 added stable-applied This PR has been cherry-picked to libc's stable release branch and removed stable-nominated This PR should be considered for cherry-pick to libc's stable release branch labels Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-unix O-x86 S-waiting-on-review stable-applied This PR has been cherry-picked to libc's stable release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants