Bug #76820 [Ver->Csd]: Z_COPYABLE invalid definition

Edit report at https://bugs.php.net/bug.php?id=76820&edit=1 ID: 76820 Updated by: cmb@php.net Reported by: mvdwerve at outlook dot com Summary: Z_COPYABLE invalid definition -Status: Verified +Status: Closed Type: Bug Package: *General Issues Operating System: All PHP Version: master-Git-2018-08-30 (Git) Assigned To: cmb Block user comment: N Private report: N New Comment: Automatic comment on behalf of cmbecker69@gmx.de Revision: http://git.php.net/?p=php-src.git;a=commit;h=c5231ad394fa3a6c59b9786af977d61da3f1e208 Log: Fix #76820: Z_COPYABLE invalid definition Previous Comments: ------------------------------------------------------------------------ [2018-08-30 15:49:19] cmb@php.net Thanks! ------------------------------------------------------------------------ [2018-08-30 15:24:32] mvdwerve at outlook dot com Description: ------------ As of commit 742d5a01ed5bd39a18929c04a5f5c5596a4b005b, IS_TYPE_COPYABLE was apparantly removed. We use that in our wrapper of the Zend engine. Although it is deprecated, a good definition should be provided, right? https://github.com/php/php-src/blob/master/Zend/zend_types.h#L593 https://github.com/php/php-src/blob/master/Zend/zend_types.h#L613 Currently, it simply doesn't compile when you use the macro and don't include an _extra_ closing bracket, as this is the current definition in master: #define Z_COPYABLE(zval) ((Z_TYPE(zval) == IS_ARRAY) where it should be #define Z_COPYABLE(zval) (Z_TYPE(zval) == IS_ARRAY) ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=76820&edit=1