Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:42538 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 23906 invoked from network); 8 Jan 2009 22:49:21 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 8 Jan 2009 22:49:21 -0000 Authentication-Results: pb1.pair.com header.from=andrei@gravitonic.com; sender-id=unknown Authentication-Results: pb1.pair.com smtp.mail=andrei@gravitonic.com; spf=permerror; sender-id=unknown Received-SPF: error (pb1.pair.com: domain gravitonic.com from 72.14.204.229 cause and error) X-PHP-List-Original-Sender: andrei@gravitonic.com X-Host-Fingerprint: 72.14.204.229 qb-out-0506.google.com Received: from [72.14.204.229] ([72.14.204.229:16238] helo=qb-out-0506.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 3B/37-20920-0F286694 for ; Thu, 08 Jan 2009 17:49:20 -0500 Received: by qb-out-0506.google.com with SMTP id d8so3554494qbc.23 for ; Thu, 08 Jan 2009 14:49:17 -0800 (PST) Received: by 10.142.14.18 with SMTP id 18mr10388357wfn.35.1231454957542; Thu, 08 Jan 2009 14:49:17 -0800 (PST) Received: from barley.local (64-71-7-198.static.wiline.com [64.71.7.198]) by mx.google.com with ESMTPS id 32sm43156217wfc.39.2009.01.08.14.49.14 (version=TLSv1/SSLv3 cipher=RC4-MD5); Thu, 08 Jan 2009 14:49:15 -0800 (PST) Message-ID: <496682E9.4000604@gravitonic.com> Date: Thu, 08 Jan 2009 14:49:13 -0800 User-Agent: Thunderbird 2.0.0.6 (Macintosh/20070807) MIME-Version: 1.0 To: robin@soal.org CC: PHP internals , David Coallier , Dmitry Stogov References: <4963D8AD.5090502@gravitonic.com> <5a8807d10901080900w5d794c54gc67df2198e2237f7@mail.gmail.com> In-Reply-To: <5a8807d10901080900w5d794c54gc67df2198e2237f7@mail.gmail.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [PHP-DEV] Private member/method access inconsistency From: andrei@gravitonic.com (Andrei Zmievski) Robin Fernandes wrote: > Cool, looks like this has gone in already. A couple of comments: > 1. I think the change has not been made for callbacks (so callbacks > now behave differently from normal method invocations). > 2. Perhaps on 5_3 and HEAD, the equivalent change should be made for > __callStatic()? > > Addressing those two points would make things even more consistent. > I'm adding some testcases to illustrate. I fixed the callback case in 5_2. Need to talk to Dmitry about 5_3 and HEAD because zend_is_callable_check_func() is damn complex now. -Andrei