Le 26/11/2020 à 03:41, Sara Golemon a écrit :
Maybe just a wee bit late to ask this question, but shouldn't we have long since changed `#define ZEND_ENGINE_3` to `#degine ZEND_ENGINE_4` ?
From a quick search I see some usage of ZEND_ENGINE_3 (eg, imagick and gmagick ext, probably others)
Was that deliberate since (unlike with 4->5 and 5->7) the engine's API surface area hasn't skewed so bad?
Indeed, This is mostly used to test for PHP 5 vs 7/8 API
I ask because ZEND_VERSION did get updated to the 4.x.x series.
IMHO, we should keep ZEND_ENGINE_3 to not break things so late and encourage dev to not use it anymore And drop it in 8.1 (without ZEND_ENGINE_4) Remi
-Sara
P.S. https://github.com/Imagick/imagick/pull/363 -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: https://www.php.net/unsub.php
