Skip to content

Use exit instead of E_USER_ERROR#225

Closed
nicolas-grekas wants to merge 1 commit intoamphp:2.xfrom
nicolas-grekas:exit-error
Closed

Use exit instead of E_USER_ERROR#225
nicolas-grekas wants to merge 1 commit intoamphp:2.xfrom
nicolas-grekas:exit-error

Conversation

@nicolas-grekas
Copy link
Contributor

@nicolas-grekas nicolas-grekas commented Mar 13, 2026

Passing E_USER_ERROR to trigger_error() is deprecated since 8.4, throw an exception or call exit with a string message instead

Fix #223

@nicolas-grekas
Copy link
Contributor Author

Let me close as a failed attempt: exit kills the process, not only the thread. Not sure how this should be addressed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

PHP Deprecated , trigger_error

1 participant