Skip to content

Fix for PHP 8.6.0alpha2#16

Merged
codemasher merged 1 commit into
codemasher:mainfrom
remicollet:issue-php86a2
Jul 17, 2026
Merged

Fix for PHP 8.6.0alpha2#16
codemasher merged 1 commit into
codemasher:mainfrom
remicollet:issue-php86a2

Conversation

@remicollet

Copy link
Copy Markdown

. The INI_STR(), INI_INT(), INI_FLT(), and INI_BOOL() macros have been
removed. Instead new zend_ini_{bool|long|double|str|string}_literal()
macros have been added.

@remicollet remicollet changed the title Fix for PP 8.6.0alpha2 Fix for PHP 8.6.0alpha2 Jul 17, 2026
@codemasher

Copy link
Copy Markdown
Owner

Thank you! It looks like we need a version switch if we still want to keep PHP7 compatibility.

@remicollet

Copy link
Copy Markdown
Author

Thank you! It looks like we need a version switch if we still want to keep PHP7 compatibility.

Why ?
zend_ini_long exists since (at least) 5.6

Tested with PHP 7.4 and 8.6.0alpha2

  . The INI_STR(), INI_INT(), INI_FLT(), and INI_BOOL() macros have been
    removed. Instead new zend_ini_{bool|long|double|str|string}_literal()
    macros have been added.
@remicollet

Copy link
Copy Markdown
Author

OK, understood

php70/Zend/zend_ini.h:ZEND_API zend_long zend_ini_long(char *name, uint name_length, int orig);
php71/Zend/zend_ini.h:ZEND_API zend_long zend_ini_long(char *name, uint name_length, int orig);
php72/Zend/zend_ini.h:ZEND_API zend_long zend_ini_long(char *name, uint32_t name_length, int orig);
php73/Zend/zend_ini.h:ZEND_API zend_long zend_ini_long(char *name, size_t name_length, int orig);
php74/Zend/zend_ini.h:ZEND_API zend_long zend_ini_long(char *name, size_t name_length, int orig);
php80/Zend/zend_ini.h:ZEND_API zend_long zend_ini_long(const char *name, size_t name_length, int orig);
php81/Zend/zend_ini.h:ZEND_API zend_long zend_ini_long(const char *name, size_t name_length, int orig);
php82/Zend/zend_ini.h:ZEND_API zend_long zend_ini_long(const char *name, size_t name_length, int orig);
php83/Zend/zend_ini.h:ZEND_API zend_long zend_ini_long(const char *name, size_t name_length, int orig);
php84/Zend/zend_ini.h:ZEND_API zend_long zend_ini_long(const char *name, size_t name_length, int orig);
php85/Zend/zend_ini.h:ZEND_API zend_long zend_ini_long(const char *name, size_t name_length, int orig);
phpmaster/Zend/zend_ini.h:ZEND_API zend_long zend_ini_long(const char *name, size_t name_length, bool orig);

I have replaced false by 0 in the compatibility macro

@codemasher
codemasher merged commit dc65c32 into codemasher:main Jul 17, 2026
27 checks passed
@codemasher

Copy link
Copy Markdown
Owner

Thank you again!

@remicollet
remicollet deleted the issue-php86a2 branch July 17, 2026 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants