Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:2117
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Date: Sat, 31 May 2003 20:40:59 -0400
To: marcus.boerger@post.rwth-aachen.de
Cc: PHP Developers List <internals@lists.php.net>
Message-ID: <20030601004059.GA24639@codewhore.org>
References: <20030601000427.GA5775@codewhore.org> <5.1.0.14.2.20030601023239.03084758@pop.t-online.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <5.1.0.14.2.20030601023239.03084758@pop.t-online.de>
User-Agent: Mutt/1.4i
Sender: David Brown <dave@lan.codewhore.org>
Subject: Re: [PHP-DEV] Re: CVS commit in ZendEngine2/zend_execute.c
From: dave@codewhore.org (David Brown)

On Sun, Jun 01, 2003 at 02:34:41AM +0200, Marcus B?rger wrote:
| Yes David,
| 
| in ZTS mode it causes much problems, i found them some hours ago. But
| the main reason for revert is that the patch was done in the wrong
| way. Currently we are trying to figure out a new way since the
| performance boost is very good.

Hi Marcus,

The crash I was seeing was under the CLI sapi, without ZTS enabled.
If this is unexpected, I can try for a test case. Otherwise, if you're
pretty sure the patch had more wrong with it than there was right, I'll
hold off. :)

The codebase I was seeing the crash in is large and reasonably complex -
I was just checking to make sure that the patch was indeed causing the
problem, not just making it visible.

Anyway. Much thanks. :)

- Dave