Re: [PHP4BETA] cvs: /php4/ext/standard parsedate.y

From: Date: Mon, 06 Dec 1999 15:17:03 +0000
Subject: Re: [PHP4BETA] cvs: /php4/ext/standard parsedate.y
Groups: php.version4 
Request: Send a blank email to php-version4+get-7524@lists.php.net to get a copy of this message
>Why is parsedate.y including php.h in the first place? I believe that's >where all of the problems originate from. php.h is included by php_reentrancy.h, in order to define PHPAPI. I personally believe that there should be a separation between the zend headers which contain information suitable for public consumption, and from that which is needed by the implementation of zend. The #defines for bison clearly fall in the latter category. To see an illustration of why this is prudent, try removing the undef I added to parsedate.y, and see what results. Like Thies, I believe that the correct fix is to zend itself, but am reluctant to make the fix myself as I am not confident I understand all the ramifications of the change.

« previous php.version4 (#7524) next »