Re: [ZEND-ENGINE-CVS] cvs: Zend(PHP_4_3) / zend.c

At 20:43 02.06.2003, Andi Gutmans wrote:

At 08:13 PM 6/1/2003 +0200, Marcus Börger wrote:

At 19:51 01.06.2003, Thies C. Arntzen wrote:

On Sun, Jun 01, 2003 at 05:06:02PM -0000, Marcus Boerger wrote:

helly         Sun Jun  1 13:06:02 2003 EDT

  Modified files:              (Branch: PHP_4_3)
    /Zend     zend.c

Log: Need to initialize here already. # You might wonder but this fixes some ini problems.

    so we're doing this twice now? i actually noticed (and sent
    mail to -dev) the mentioned ini problem some time ago. i'm
    not sure if this will actually fix it completely because the
    per-request memory manager (e*() functions) is not
    guaranteed to be in an defined state beween reqeuest and
    thereby should not be used in the ini-system at all...

You're right that this doesn't fix all problems. So we need a better solution. But the fix helps for CLI/CGI at least.

Well it's better to roll back this patch then and find a real fix.

From my point of view: No it's not. The real problem requires a rewrite of the whole ini facility and maybe parts of the memory manager. Since i don't see someone doing this the patch offers at least a fix for CLI/CGI. Also i am currently not able to debug PHP5 in a threaded webserver environment due to limited time i can spend on PHP development. Sorry for this and feel free to revert the patch if you still think it is the better way. marcus