Re: ZEND_ENGINE_4

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