Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:19160 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 61271 invoked by uid 1010); 25 Sep 2005 17:05:37 -0000 Delivered-To: ezmlm-scan-internals@lists.php.net Delivered-To: ezmlm-internals@lists.php.net Received: (qmail 61255 invoked from network); 25 Sep 2005 17:05:37 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 25 Sep 2005 17:05:37 -0000 X-Host-Fingerprint: 81.169.182.136 ajaxatwork.net Linux 2.4/2.6 Received: from ([81.169.182.136:53485] helo=strato.aixcept.de) by pb1.pair.com (ecelerity 2.0 beta r(6323M)) with SMTP id 66/8B-24510-0E8D6334 for ; Sun, 25 Sep 2005 13:05:36 -0400 Received: from [192.168.1.3] (dsl-082-083-231-108.arcor-ip.net [82.83.231.108]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by strato.aixcept.de (Postfix) with ESMTP id F3B7E35C37F; Sun, 25 Sep 2005 19:05:56 +0200 (CEST) Date: Sun, 25 Sep 2005 19:05:32 +0200 Reply-To: Marcus Boerger X-Priority: 3 (Normal) Message-ID: <1221052118.20050925190532@marcus-boerger.de> To: Andi Gutmans Cc: internals In-Reply-To: <6.2.3.4.2.20050925093007.02219010@localhost> References: <1728334199.20050925143657@marcus-boerger.de> <6.2.3.4.2.20050925093007.02219010@localhost> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Subject: Re: [PHP-DEV] [PATCH] __toString() From: helly@php.net (Marcus Boerger) Hello Andi, i don't see your point here. We've discussed this tons of times. Either fix the engine if you know it's broken, show us a real problem with the pacth or accept the fact that we want to make progress here. Blocking patches because they have a slight potential in causing some problems without telling about the problems and coming up with some test script doesn't help anybody here. You are just waisting my time with that. All other who saw the patch agreed to it already. marcus Sunday, September 25, 2005, 6:31:59 PM, you wrote: > Give me a couple of days to review the patch itself. > I think we should then commit to HEAD, and see how it goes. There's > hopefully enough time in the PHP 6 process to validate that this kind > of patch doesn't hit any conceptual problems re: propagating to > pre-compile/post-shutdown execution stages... > Andi > At 05:36 AM 9/25/2005, Marcus Boerger wrote: >>Hello internals, >> >> the patch implements __toString to have obejcts be automatically converted >>to strings anywhere a string is requested. We have talked a lot about this >>in the past and during OSCON Andi agreed again on it and said that the HEAD >>version of the engine should be ready for it now. Futher more we have enough >>time to fix any outstanding engine issues regarding this. >> >> http://php.net/~helly/php/ext/ze2/ze2-tostring-20050925.diff.txt >> >> If nobody objects with a real technical issue i'll commit the patch early >>in the week. >> >> The patch is a little big longer because it ensures that __toString gets >>treated and especially gets cached just like any other magic function is.