Skip to content

[pull] master from php:master - #1183

Merged
pull[bot] merged 5 commits into
turkdevops:masterfrom
php:master
Aug 13, 2026
Merged

[pull] master from php:master#1183
pull[bot] merged 5 commits into
turkdevops:masterfrom
php:master

Conversation

@pull

@pull pull Bot commented Aug 13, 2026

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

orlitzky and others added 5 commits August 12, 2026 21:53
As of v1.2.6, musl's bind_textdomain_codeset() no longer returns NULL
to indicate that the codeset is UTF-8 "by default." Instead you get
"UTF-8". Two tests require minor updates for this.

musl commit: https://git.musl-libc.org/cgit/musl/commit/src/locale/bind_textdomain_codeset.c?id=cabbd8697d39c09ca37fb3d63b86b42526b81fda

Close GH-23235
* PHP-8.4:
  ext/gettext/tests: updates for musl-1.2.6
  ext/snmp: Add XFAIL on several tests (#22476) (#23231)
  ext/readline: Fix class constant completion in readline interactive shell (#23218)
* PHP-8.5:
  ext/gettext/tests: updates for musl-1.2.6
And related Z_PARAM_GET_PREV_ZVAL()

These were introduced in 1179622 to simplify memory management.

But this is no longer relevant (well never was as one could have grabbed the zval from the FCI) since the refactoring to only use FCCs in 0e5d654

The following SourceGraph searches confirm there is no usage:
- https://sourcegraph.com/search?q=context:global+Z_PARAM_FUNC_OR_NULL_WITH_ZVAL&patternType=keyword&sm=0
- https://sourcegraph.com/search?q=context:global+Z_PARAM_GET_PREV_ZVAL&patternType=keyword&sm=0
@pull pull Bot locked and limited conversation to collaborators Aug 13, 2026
@pull pull Bot added the ⤵️ pull label Aug 13, 2026
@pull
pull Bot merged commit c127a96 into turkdevops:master Aug 13, 2026
1 of 3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants