Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for PHP 8.2 #1405

Closed
externl opened this issue Jan 10, 2023 · 0 comments
Closed

Add support for PHP 8.2 #1405

externl opened this issue Jan 10, 2023 · 0 comments
Assignees
Labels
Milestone

Comments

@externl
Copy link
Member

externl commented Jan 10, 2023

There are currently several issues that need fixing to support PHP 8.2

  • Various Clang warnings (treated as errors)
/opt/homebrew/Cellar/php/8.2.1/include/php/Zend/zend_ini.h:105:22: error: '\f]' command does not terminate a verbatim text block [-Werror,-Wdocumentation]
 *     ws: [ \t\n\r\v\f]
  • Our __toString methods do not specify a return type. This causes the following warning:
Warning: IcePHP_Properties::__toString() implemented without string return type in Unknown on line 0

Warning: Ice\ObjectPrx::__toString() implemented without string return type in Unknown on line 0

Warning: IcePHP_Logger::__toString() implemented without string return type in Unknown on line 0

Warning: IcePHP_Endpoint::__toString() implemented without string return type in Unknown on line 0

Warning: IcePHP_Connection::__toString() implemented without string return type in Unknown on line 0
  • Lots of crashing with the following error:
Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 2028208128 bytes) in /Users/joe/Developer/zeroc-ice/ice/3.7/php/test/Ice/acm/Client.php on line 44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants