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.

Sign up for GitHub

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

Jump to bottom

gh-111178: remove redundant casts for functions with correct signatures #131673

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

gh-111178: remove redundant casts for functions with correct signatures #131673

wants to merge 20 commits into from
+99 −102

Conversation

picnixz
Copy link
Member

@picnixz picnixz commented Mar 24, 2025 •
edited
Loading

This is a cleanup PR but it's easier to hunt non-covered possible UBSans

@bedevere-app bedevere-app bot mentioned this pull request Mar 24, 2025
Open
@picnixz
Copy link
Member Author

picnixz commented Mar 24, 2025 •
edited
Loading

Very interesting. I locally had an issue where my make command hangs (which is the case here) but I still don't know why this happens.

picnixz
picnixz commented Mar 24, 2025
View reviewed changes
@picnixz
Copy link
Member Author

picnixz commented Mar 24, 2025

I think it know what happened! I incorrectly merged main into this branch c:

@picnixz picnixz marked this pull request as ready for review March 30, 2025 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vstinner vstinner

Assignees
No one assigned
Labels
awaiting core review skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@picnixz