Skip to content

Commit 9839752

Browse files
committed
Voidify some ZEND_API functions
Closes GH-5805
1 parent 92448d9 commit 9839752

File tree

4 files changed

+122
-141
lines changed

4 files changed

+122
-141
lines changed

UPGRADING.INTERNALS

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ PHP 8.0 INTERNALS UPGRADE NOTES
2020
q. GC_COLLECTABLE flag
2121
r. Cannot implement Traversable only
2222
s. zend_fcall_info no_separation flag removed
23+
t. Signature changes
2324

2425
2. Build system changes
2526
a. Abstract
@@ -89,7 +90,7 @@ PHP 8.0 INTERNALS UPGRADE NOTES
8990
j. compare_objects() handler was removed. Extensions should use compare() object
9091
handler instead and check if both arguments are objects and have the same
9192
compare handler, using ZEND_COMPARE_OBJECTS_FALLBACK() macro.
92-
93+
9394
k. The 'I' length modifier, used to denote 32 and 64bit integer from the custom
9495
snprintf and spprintf implementations has been removed.
9596
Use the ZEND_LONG_FMT, ZEND_ULONG_FMT and ZEND_XLONG_FMT macros defined in
@@ -136,11 +137,22 @@ PHP 8.0 INTERNALS UPGRADE NOTES
136137
zend_create_internal_iterator_zval(return_value, ZEND_THIS);
137138
}
138139

139-
s. The zend_fcall_info no_separation flag has been removed, and separation is
140+
s. The zend_fcall_info no_separation flag has been removed, and separation is
140141
never allowed. If you wish to pass (or allow passing) arguments by
141142
reference, explicitly create those arguments as references using
142143
ZEND_MAKE_REF.
143144

145+
t. The following ZEND_API function have changed signature:
146+
1. Void in Zend Engine 4.0:
147+
- add_assoc_*()
148+
- add_index_*()
149+
- add_property_*()
150+
- object_init()
151+
- zend_declare_class_constant*()
152+
- zend_declare_property*()
153+
- zend_startup_modules()
154+
- zend_wrong_parameters_none_error()
155+
144156
========================
145157
2. Build system changes
146158
========================

0 commit comments

Comments
 (0)