Skip to content

Fix non-canonical casts #19438

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

Merged
merged 1 commit into from
Aug 9, 2025
Merged

Fix non-canonical casts #19438

merged 1 commit into from
Aug 9, 2025

Conversation

nikic
Copy link
Member

@nikic nikic commented Aug 9, 2025

They may be deprecated, but they should still produce the correct type...

They may be deprecated, but they should still produce the correct
type...
Copy link
Member

@nielsdos nielsdos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welp, thanks for catching

@nikic nikic merged commit bc475ad into php:master Aug 9, 2025
9 checks passed
@Girgias
Copy link
Member

Girgias commented Aug 9, 2025

Yeah, that was a colossal brain fart.

@jrfnl
Copy link
Contributor

jrfnl commented Aug 9, 2025

Thanks for fixing this, I was already breaking my brain trying to figure out why the tokenization had changed (test in PHP_CodeSniffer which was failing).

@jrfnl
Copy link
Contributor

jrfnl commented Aug 9, 2025

For the record - this fixes an issue introduced via #19372.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants