Skip to content

Commit

Permalink
Remove Z_PARAM_ZVAL_DEREF
Browse files Browse the repository at this point in the history
Since PHP 7.4 using this is very likely wrong, and we don't have
any core uses of it for that reason. Extensions shouldn't be using
it either. Accept as Z_PARAM_ZVAL and then assign using
ZEND_TRY_ASSIGN macros.
  • Loading branch information
nikic committed Jul 18, 2019
1 parent 33656f0 commit bbcfa50
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions Zend/zend_API.h
Expand Up @@ -1486,14 +1486,6 @@ ZEND_API ZEND_COLD void ZEND_FASTCALL zend_wrong_callback_error(int num, char *e
#define Z_PARAM_ZVAL(dest) \
Z_PARAM_ZVAL_EX(dest, 0, 0)

/* old "z" (with dereference) */
#define Z_PARAM_ZVAL_DEREF_EX(dest, check_null, separate) \
Z_PARAM_PROLOGUE(1, separate); \
zend_parse_arg_zval_deref(_arg, &dest, check_null);

#define Z_PARAM_ZVAL_DEREF(dest) \
Z_PARAM_ZVAL_DEREF_EX(dest, 0, 0)

/* old "+" and "*" */
#define Z_PARAM_VARIADIC_EX(spec, dest, dest_num, post_varargs) do { \
int _num_varargs = _num_args - _i - (post_varargs); \
Expand Down

0 comments on commit bbcfa50

Please sign in to comment.