From 40187823a8d45d5b8a966427bdd73eafd654ff0a Mon Sep 17 00:00:00 2001
From: Jose <jose@zeroc.com>
Date: Fri, 1 Sep 2017 20:39:47 +0200
Subject: Fix PHP build warnings

ZEND_MODULE_API_NO number used in Init.cpp bugfix was bogus
required_num_args member of _zend_arg_info was changed to
zend_uint in PHP 5.4.0 that correspond to ZEND_MODULE_API_NO
20100525

See bug (ICE-8479)
---
 php/src/php5/Init.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'php/src/php5')

diff --git a/php/src/php5/Init.cpp b/php/src/php5/Init.cpp
index b2490c9d080..1b767212974 100644
--- a/php/src/php5/Init.cpp
+++ b/php/src/php5/Init.cpp
@@ -24,8 +24,9 @@ ZEND_DECLARE_MODULE_GLOBALS(ice)
 
 //
 // BUGFIX Avoid narrowing conversion warnings with ZEND_BEGIN_ARG_INFO_EX usage
+// in PHP >= 5.4
 //
-#if ZEND_MODULE_API_NO >= 20121212
+#if ZEND_MODULE_API_NO >= 20100525
 ZEND_BEGIN_ARG_INFO_EX(Ice_initialize_arginfo, 1, ZEND_RETURN_VALUE, static_cast<zend_uint>(-1))
 ZEND_END_ARG_INFO()
 
-- 
cgit v1.2.3