Skip to content

Zend: throw exception for AST error - #25141

Merged
Girgias merged 1 commit into
php:masterfrom
Girgias:2026-11-ast-throw-rather-bailout
Nov 3, 2026
Merged

Girgias merged 1 commit into
php:masterfrom
Girgias:2026-11-ast-throw-rather-bailout

Conversation

@Girgias

@Girgias Girgias commented Nov 2, 2026

Copy link
Copy Markdown
Member

Rather than bailing out with zend_error_noreturn()

Rather than bailing out with zend_error_noreturn()
@Girgias
Girgias marked this pull request as ready for review November 2, 2026 00:31

@arnaud-lb arnaud-lb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@Girgias
Girgias merged commit 8284077 into php:master Nov 3, 2026
17 of 18 checks passed
@Girgias
Girgias deleted the 2026-11-ast-throw-rather-bailout branch November 3, 2026 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants