From 639d221df3ba639f4b7f0df29c4cdf9b83c100c7 Mon Sep 17 00:00:00 2001 From: Felipe Pena Date: Sun, 15 Jun 2008 18:27:37 +0000 Subject: [PATCH] - Improved warning message to call-time pass-by-reference when used with internal function (http://news.php.net/php.internals/38252) --- Zend/zend_compile.c | 21 +++++++++++++-------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/Zend/zend_compile.c b/Zend/zend_compile.c index da0757bb97..8580e35bef 100644 --- a/Zend/zend_compile.c +++ b/Zend/zend_compile.c @@ -2019,14 +2019,19 @@ void zend_do_pass_param(znode *param, zend_uchar op, int offset TSRMLS_DC) zend_stack_top(&CG(function_call_stack), (void **) &function_ptr_ptr); function_ptr = *function_ptr_ptr; - if (original_op==ZEND_SEND_REF - && !CG(allow_call_time_pass_reference)) { - zend_error(E_DEPRECATED, - "Call-time pass-by-reference has been deprecated; " - "If you would like to pass it by reference, modify the declaration of %s(). " - "If you would like to enable call-time pass-by-reference, you can set " - "allow_call_time_pass_reference to true in your INI file", - (function_ptr ? function_ptr->common.function_name : "[runtime function name]")); + if (original_op == ZEND_SEND_REF && !CG(allow_call_time_pass_reference)) { + if (function_ptr && + function_ptr->common.function_name && + function_ptr->common.type == ZEND_USER_FUNCTION && + !ARG_SHOULD_BE_SENT_BY_REF(function_ptr, (zend_uint) offset)) { + zend_error(E_DEPRECATED, + "Call-time pass-by-reference has been deprecated; " + "If you would like to pass it by reference, modify the declaration of %s(). " + "If you would like to enable call-time pass-by-reference, you can set " + "allow_call_time_pass_reference to true in your INI file", function_ptr->common.function_name); + } else { + zend_error(E_DEPRECATED, "Call-time pass-by-reference has been deprecated"); + } } if (function_ptr) { -- 2.40.0