Php7.3

[VHDev]

ZEND_HASH_APPLY_PROTECTION and nApplyCount member are deprecated in php 7.3.

From UPGRADING.INTERNALS

c. Protection from recursion during processing circular data structures was
refactored. HashTable.nApplyCount and IS_OBJ_APPLY_COUNT are replaced by
single flag GC_PROTECTED. Corresponding macros Z_OBJ_APPLY_COUNT,
Z_OBJ_INC_APPLY_COUNT, Z_OBJ_DEC_APPLY_COUNT, ZEND_HASH_GET_APPLY_COUNT,
ZEND_HASH_INC_APPLY_COUNT, ZEND_HASH_DEC_APPLY_COUNT are replaced with
GC_IS_RECURSIVE, GC_PROTECT_RECURSION, GC_UNPROTECT_RECURSION,
Z_IS_RECURSIVE, Z_PROTECT_RECURSION, Z_UNPROTECT_RECURSION.

 HASH_FLAG_APPLY_PROTECTION flag and ZEND_HASH_APPLY_PROTECTION() macro
 are removed. All mutable arrays should use recursion protection.
 Corresponding checks should be replaced by Z_REFCOUNTED() or
 !(GC_GLAGS(p) & GC_IMMUTABLE).

Found excellent examples here: https://news-web.php.net/php.cvs/99639

I'm new to PHP internals, but thought I'd take a crack at this as I need to get it working with php7.3. It seems to have compiled, tested successfully, and is working correctly in the dev environment. I stumbled upon the inline hash protection function idea from PHP Mongo development team.
mongodb/mongo-php-driver@197f3f7

Also, a sincere thank you. This project is really a huge help. Impressive work!
DV

[VHDev]

Patch

[marinhekman]

@patrickallaert so is this going to be merged ? I would like a new pecl build which supports 7.3+ . Also a Windows dll build would be nice (latest is 2.0.3 on https://pecl.php.net/package/APM)