parse_date.c needs zend.h

php.internals

Sebastian Bergmann

20 years ago
After updating my PHP_5_1 checkout and the libxml2/libxslt libraries today, the ext/date/lib/parse_date.c source file seems to need an #include "zend.h" to find strncasecmp(). I know that this change has to be made to the corresponding .re file and since I do not want to mess with it it would be great if someone else could look into this. Thanks!
-- Sebastian Bergmann http://www.sebastian-bergmann.de/ GnuPG Key: 0xB85B5D69 / 27A7 2B14 09E4 98CD 6277 0E5B 6867 C514 B85B 5D69

Jani Taskinen

20 years ago
What it really needs is strings.h to be included. (zend.h is included via php.h anyway) --Jani On Mon, 12 Dec 2005, Sebastian Bergmann wrote:

Sebastian Bergmann

20 years ago
Jani Taskinen schrieb:
> What it really needs is strings.h to be included.
It seems to need zend_config.w32.h on Windows, as that is where strncasecmp gets #defined.
-- Sebastian Bergmann http://www.sebastian-bergmann.de/ GnuPG Key: 0xB85B5D69 / 27A7 2B14 09E4 98CD 6277 0E5B 6867 C514 B85B 5D69

Jani Taskinen

20 years ago
On Mon, 12 Dec 2005, Sebastian Bergmann wrote:
> > Jani Taskinen schrieb: >> What it really needs is strings.h to be included. > > It seems to need zend_config.w32.h on Windows, as that is where > strncasecmp gets #defined.
Actually not. This is separate code from PHP/Zend. I just committed a potential fix, can you test please? --Jani

Sebastian Bergmann

20 years ago
Jani Taskinen schrieb:
> I just committed a potential fix, can you test please?
Works, thanks.
-- Sebastian Bergmann http://www.sebastian-bergmann.de/ GnuPG Key: 0xB85B5D69 / 27A7 2B14 09E4 98CD 6277 0E5B 6867 C514 B85B 5D69