Skip to content

Conversation

pull[bot]
Copy link

@pull pull bot commented Jul 7, 2025

See Commits and Changes for more details.


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

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

petk and others added 8 commits July 7, 2025 07:54
This also omits defining unused HAVE_WINSOCK_H macro when building
ext/sockets.
re2c version 4 enabled some warnings by default. This fixes re2c code
for the `-Wuseless-escape` warnings.

There are two same issues reported.
Issue: GH-17523
Closes: GH-17204
* PHP-8.3:
  Fix -Wuseless-escape warnings emitted by re2c (#19050)
* PHP-8.4:
  Fix -Wuseless-escape warnings emitted by re2c (#19050)
Since `ZSTR_LEN()` returns a `size_t` (unsigned integer), the value can only be either "not equal to 0" or "equal to 0". The third `else` branch was unreachable, making the `*handled_output = NULL;` assignment dead code.
IR commit: af6dc83bcd91c3123f40efcdcbeba8794b9b2abf
* PHP-8.4:
  Update IR
@pull pull bot locked and limited conversation to collaborators Jul 7, 2025
@pull pull bot added the ⤵️ pull label Jul 7, 2025
@pull pull bot merged commit dd69b65 into wudi:master Jul 7, 2025
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.

3 participants