Skip to content

zend_types: Remove ZEND_RESULT_CODE typedef #21850

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
TimWolla merged 1 commit into from
Apr 23, 2026
+3 −4
Merged

zend_types: Remove ZEND_RESULT_CODE typedef #21850

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions UPGRADING.INTERNALS
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,7 @@ PHP 8.6 INTERNALS UPGRADE NOTES
. Functions using zend_forbid_dynamic_call() *must* be flagged with
ZEND_ACC2_FORBID_DYN_CALLS (@forbid-dynamic-calls in stubs). In debug
builds, failing to include that flag will lead to assertion failures.
. The ZEND_RESULT_CODE type has been removed. Use zend_result directly.

========================
2. Build system changes
Expand Down
6 changes: 2 additions & 4 deletions Zend/zend_types.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,12 +54,10 @@

typedef unsigned char zend_uchar;

typedef enum {
typedef enum zend_result {
SUCCESS = 0,
FAILURE = -1, /* this MUST stay a negative number, or it may affect functions! */
} ZEND_RESULT_CODE;

typedef ZEND_RESULT_CODE zend_result;
} zend_result;

/* This constant is deprecated, use SIZE_MAX instead */
#define ZEND_SIZE_MAX SIZE_MAX
Expand Down
Loading