Skip to content

merge: cascade 8.4 into master - #220

Merged
lisachenko merged 2 commits into
masterfrom
8.4
Aug 14, 2026
Merged

merge: cascade 8.4 into master#220
lisachenko merged 2 commits into
masterfrom
8.4

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Automated cascade merge of 8.4 into master (branch flow defined in .github/branch-flow.json).

Resolve conflicts in favour of the newer engine structures where they touch include/ - regenerate headers on the target branch instead of merging them textually. See AGENTS.md.

claude and others added 2 commits August 14, 2026 19:16
…yntax

The get_closure hook still materialized its fallback closures through
Closure::fromCallable(), the two sites the syntax pass missed when it
converted the trampoline installers in AbstractHook. Since PHP 8.1
`$callable(...)` is defined to be exactly Closure::fromCallable($callable)
and works for both shapes used here - the [$instance, $methodName] array
for a bound method and the 'Scope::method' string for a static one.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RnoZ7wuGepCTzsmFQ5sKxG
…-6pfu7y-syntax-84-followup

refactor: syntax-pass follow-up for the sites the merged pass missed
@lisachenko
lisachenko merged commit 5c390e2 into master Aug 14, 2026
21 checks passed
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