Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:10053 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 90813 invoked by uid 1010); 23 May 2004 18:56:09 -0000 Delivered-To: ezmlm-scan-internals@lists.php.net Delivered-To: ezmlm-internals@lists.php.net Received: (qmail 90777 invoked by uid 1007); 23 May 2004 18:56:09 -0000 Message-ID: <20040523185609.90776.qmail@pb1.pair.com> To: internals@lists.php.net References: <20040522211237.84677.qmail@pb1.pair.com> <5.1.0.14.2.20040523160427.03309350@127.0.0.1> Date: Sun, 23 May 2004 11:41:22 -0700 Lines: 22 X-Priority: 3 X-MSMail-Priority: Normal X-Newsreader: Microsoft Outlook Express 6.00.2800.1158 X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2800.1165 X-Posted-By: 64.142.6.231 Subject: Re: [PHP-DEV] Analysis and Patch for Bug#28491 From: pollita@php.net ("Sara Golemon") > I am aware that if users really want to create a reference to the locally > defined "$this" variable they can hack around it. As you noted they won't > be able to change the real object though, i.e. EG(This) but only the local > $this symbol table entry. As far as I'm concerned, the behavior is > undefined and I don't want to do any run-time checks for this, as it'll > just slow things down. > Post 5.0 we might want to change "$this" so that it isn't defined in the > symbol table at all but will be dealt with at compile-time. This was my > initial implementation but due to problems of reference assignments I opted > for the current implementation. Maybe after investigating this again, I > might find a different way of doing. > So the bottom line is, let's leave it the way it is, and remind me after > 5.0 to take another look. However, it is quite possible that I'll come up > with the same conclusion that the current implementation is the best one. > Fair enough, I'll mark 28491 suspended and try to sum up your comments for the reporter. -Sara