Anybody have a problem with me committing the patch below?

We still use 2.9.5 for our FreeBSD builds (ya, I know its old) and while building extensions we get:

Zend/zend.h:379: warning: `always_inline' attribute directive ignored
Zend/zend.h:383: warning: `always_inline' attribute directive ignored
Zend/zend.h:387: warning: `always_inline' attribute directive ignored
Zend/zend.h:391: warning: `always_inline' attribute directive ignored
Zend/zend.h:395: warning: `always_inline' attribute directive ignored
Zend/zend.h:399: warning: `always_inline' attribute directive ignored
Zend/zend.h:403: warning: `always_inline' attribute directive ignored
Zend/zend.h:407: warning: `always_inline' attribute directive ignored

Adding the patch removes the warning.

Brian

Index: Zend/zend.h
===================================================================
RCS file: /repository/ZendEngine2/zend.h,v
retrieving revision 1.293.2.11.2.9.2.33
diff -u -p -r1.293.2.11.2.9.2.33 zend.h
--- Zend/zend.h 18 Mar 2009 01:08:12 -0000      1.293.2.11.2.9.2.33
+++ Zend/zend.h 29 Apr 2009 14:16:28 -0000
@@ -361,7 +361,11 @@ struct _zval_struct {
 #define Z_SET_ISREF_TO(z, isref)       Z_SET_ISREF_TO_P(&(z), isref)

 #if defined(__GNUC__)
+#if __GNUC__ >= 3
 #define zend_always_inline inline __attribute__((always_inline))
+#else
+#define zend_always_inline inline
+#endif
 #elif defined(_MSC_VER)
 #define zend_always_inline __forceinline
 #else


--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to