@dunglas can someone take a look at this?

@dunglas what should be done about that AppVeyor?

You can rebase, AppVeyor is gone, we moved to GithubActions even for windows tests

Ugh wrong button 😑

Need to rebase manually tomorrow

Some HttpClient problems, but that shouldn't be related to my PR?

@OskarStark what should I do about the failed Unit Tests? These look unrelated?

Can someone check the failed tests?

XliffFileDumperTest::testEmptyMetadataNotes should be unrelated to this MR
it was #59747 that was faulty

tried to rebase on a previous commit
Rebase worked locally, but i can't trick the Unit Tests there

@nicolas-grekas there all tests are green too (except the Semaphore)

Thanks for the ping. Looking at the code, I'm wondering if what you want isn't mean to be achieve by using the TYPE_CAST_ATTRIBUTES context option instead of changing anything in the code.

Given the discussion in #58479, I think this should be considered as an improvement, so for 7.3
About the implementation, the logic should be moved to the main foreach loop just above, where all similar concerns are located.

@nicolas-grekas there are some unrelated problems with testCreateTypeContextOrUseProvided

but only for PHP 8.5, i try to pinpoint when it broke

Thank you @Hanmac.