Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:31385
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain zend.com designates 63.205.162.114 as permitted sender)
Content-class: urn:content-classes:message
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Date: Thu, 2 Aug 2007 15:12:49 -0700
Message-ID: <698DE66518E7CA45812BD18E807866CE0B6443@us-ex1.zend.net>
Thread-Topic: [PHP-DEV] Fix inconsistencies in OO calls
Thread-Index: AcfU+SKHdec4AG15S7yYVB9Wgm0F9wAWOxpw
References: <46AE49B3.2070100@php.net> <1185883035.23889.31.camel@johannes.nop> <698DE66518E7CA45812BD18E807866CE7C10BB@us-ex1.zend.net> <alpine.DEB.0.98.0708020913130.26720@kossu.ez.no> <f8s8jn$tv3$1@sea.gmane.org> <1621200185.20070802133253@marcus-boerger.de>
To: "Marcus Boerger" <helly@php.net>,
	"Sebastian Bergmann" <sb@sebastian-bergmann.de>
Cc: <internals@lists.php.net>
Subject: RE: [PHP-DEV] Fix inconsistencies in OO calls
From: andi@zend.com ("Andi Gutmans")

Marcus,

It'd be kind if you gave me chance to reply before you tell people to
"just commit stuff".
I'm just about to hop on a plane and will look at the patch again
tomorrow. If people feel so strongly about it and the patch is decent
(doesn't screw up common case and maintainability) I'm OK with it. It's
not a big deal but being a bit more courteous wouldn't hurt you.

Andi=20

> -----Original Message-----
> From: Marcus Boerger [mailto:helly@php.net]=20
> Sent: Thursday, August 02, 2007 4:33 AM
> To: Sebastian Bergmann
> Cc: internals@lists.php.net
> Subject: Re: [PHP-DEV] Fix inconsistencies in OO calls
>=20
> Hello Sebastian,
>=20
>   yep, you and Derick are right here. This is definitively a=20
> fix that should go in. Chinstrap just commit the stuff please.
>=20
> marcus
>=20
> Thursday, August 2, 2007, 11:36:54 AM, you wrote:
>=20
> > Derick Rethans schrieb:
> >> I think that'd be a bad idea. I don't see a problem with=20
> this patch=20
> >> at all, and why should people use reflection here? As=20
> you're always=20
> >> so much for BC, I find it strange that you're suggesting to remove=20
> >> something totally harmless and instead want people to force to use=20
> >> the slow Reflection API which is meant for introspection.
>=20
> >  I agree. The Reflection API is the "right tool for the=20
> job" when you =20
> > are meta-programming (developing tools such as PHPUnit, for=20
> instance) =20
> > but not when you are programming.
>=20
> > --=20
> > Sebastian Bergmann                         =20
> http://sebastian-bergmann.de/
> > GnuPG Key: 0xB85B5D69 / 27A7 2B14 09E4 98CD 6277 0E5B 6867=20
> C514 B85B=20
> > 5D69
>=20
>=20
>=20
>=20
> Best regards,
>  Marcus
>=20
> --
> PHP Internals - PHP Runtime Development Mailing List To=20
> unsubscribe, visit: http://www.php.net/unsub.php
>=20
>=20