Skip to content

Fix duplicate mb_get_info spine entries (7965→7962 count sync) - #35580

Merged
PurHur merged 1 commit into
masterfrom
agent/advanced-stdlib-spine-mbgetinfo-dedup
Aug 28, 2026
Merged

Fix duplicate mb_get_info spine entries (7965→7962 count sync)#35580
PurHur merged 1 commit into
masterfrom
agent/advanced-stdlib-spine-mbgetinfo-dedup

Conversation

@PurHur

@PurHur PurHur commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Summary

Test plan

./script/aot-smoke.sh
# aot-smoke: 8 passed, 0 failed

./script/docker-exec.sh -- bash -lc 'php script/check-selfhost-spine-count-sync.php && php script/check-selfhost-spine-coverage-sync.php'
# check-selfhost-spine-count-sync: OK (spine 7962/7962)
# check-selfhost-spine-coverage-sync: OK

./script/phpunit.sh --filter MbGetInfoRuntimeAotTest
# OK (2 tests, 16 assertions)

Related: #20014 (mb_get_info stdlib), spine-sync pattern from #35565 (dom dedup).

Made with Cursor

Remove three duplicate require_once lines for JitMbGetInfo, MbGetInfoJitHelper,
and MbGetInfoRuntime introduced when mb_get_info landed — spine count was 7965/7962.

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur
PurHur merged commit 75ad594 into master Aug 28, 2026
1 check failed
@PurHur
PurHur deleted the agent/advanced-stdlib-spine-mbgetinfo-dedup branch August 28, 2026 02:19
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.

1 participant