Fix unixtojd long parameter parsing

[andypost]

Expected parameter is long but used time_t may vary depending on arch (32 or 64 bits).
Patch makes always use long (zend_long) for function argument and cast it to time_t if needed

Also it fixes test on 32-bit armv7 and x86

• Test unixtojd() function : error conditions [ext/calendar/tests/unixtojd_error1.phpt]

Source of discussion https://gitlab.alpinelinux.org/alpine/aports/-/issues/11782#note_104912

Patch for PHP 8 a bit different and used in https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/9848

[cmb69]

Indeed, that's a bug; I just filed https://bugs.php.net/80007. Thanks for the PR!

BTW, my assessment in 44c8b74 was wrong, since we're defining D_USE_32BIT_TIME_T; that had been corrected with 954543c.

[andypost]

@cmb69 thanks a lot!
Now build passes fine on 32bits, except x86 still has buggy atof() but I have no patch yet https://gitlab.alpinelinux.org/alpine/aports/-/issues/11645

[cmb69]

If this is possibly a bug in PHP, please file a bug report. :)