Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:56182 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 58673 invoked from network); 9 Nov 2011 02:09:36 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 9 Nov 2011 02:09:36 -0000 Authentication-Results: pb1.pair.com header.from=will.fitch@gmail.com; sender-id=pass Authentication-Results: pb1.pair.com smtp.mail=will.fitch@gmail.com; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.214.42 as permitted sender) X-PHP-List-Original-Sender: will.fitch@gmail.com X-Host-Fingerprint: 209.85.214.42 mail-bw0-f42.google.com Received: from [209.85.214.42] ([209.85.214.42:48024] helo=mail-bw0-f42.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id C5/00-58393-FD0E9BE4 for ; Tue, 08 Nov 2011 21:09:35 -0500 Received: by bkbzt4 with SMTP id zt4so1086725bkb.29 for ; Tue, 08 Nov 2011 18:09:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=references:from:in-reply-to:mime-version:date:message-id:subject:to :cc:content-type:content-transfer-encoding; bh=4sw+SqUQjA5Q6UtadUT9KPPiymwe4ydc1RoUoOQkBjI=; b=VxvVZRzTqXpl3ekuqtbJPSSljsHfRcBUDR7TSne0reBlTcnzw1xuHoxFi5T/SX6ksT r4jMJBrI890dyq0ZKEzRMLKSjdop7JDG1n/EByKJwWFKunW52643mkeND1AZxJoBiQW5 xhctLSZ57Yq6vOmx6TM4X5DkCHi1jeS+BZxo8= Received: by 10.204.15.137 with SMTP id k9mr172344bka.74.1320804255553; Tue, 08 Nov 2011 18:04:15 -0800 (PST) References: <4EB81703.7030605@hoa-project.net> <4EB81C84.8010609@hoa-project.net> <4EB820BB.2030009@lsces.co.uk> <4EB822D4.4060306@hoa-project.net> <4EB89C36.5000503@garfieldtech.com> <4EB8A22B.8000607@lerdorf.com> In-Reply-To: Mime-Version: 1.0 (1.0) Date: Tue, 8 Nov 2011 21:04:10 -0500 Message-ID: <-4079824451837834664@unknownmsgid> To: "guilhermeblanco@gmail.com" Cc: Nikita Popov , Rasmus Lerdorf , Larry Garfield , "internals@lists.php.net" Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [PHP-DEV] SplClassLoader RFC Voting phase From: will.fitch@gmail.com (Will Fitch) I don't argue with your logic and want for standards. I do question this implementation. This is will not enforce any standards whatsoever and I feel it serves the need for existing frameworks and standardizing them. I have to say, I've never heard the argument, "man, if there's one thing I'd like to standardize in PHP, it'd definitely be autoloading classes." No, indeed I believe this is a group of frameworks trying to implement a standard into PHP that will help them all stay on the same page. After all, what better outcome than to implement the feature into PHP rather than deciding which framework gets to "call the shots." Oh well. I'm drunk and needed to get that off my chest. Sent from my iPhone On Nov 8, 2011, at 8:26 PM, "guilhermeblanco@gmail.com" wrote: > PS: I'd love to point you all this article... this is something that > motivates me to push this RFC forward. =3D) > > http://phpmaster.com/the-importance-of-standards/ > > Cheers, > > On Tue, Nov 8, 2011 at 11:23 PM, guilhermeblanco@gmail.com > wrote: >> For all those interested, I implemented what I referred as item 2: >> >> >> After some thought it makes sense, but only if interface then defines >> the setMode prototype. >> The background for this is supported if the user decides to enhance >> the base class to support caching, requiring the injection of the >> Cache layer in constructor. If it's part of the interface, it cannot >> be changed. >> >> >> RFC is now updated covering this. If you have more questions or >> suggestions, feel free to tell me. >> >> On Tue, Nov 8, 2011 at 3:55 PM, guilhermeblanco@gmail.com >> wrote: >>> Hi Nikita, >>> >>> Thanks. >>> It's your option and I won't fight. But it seems my proposal is not yet= 100%. >>> Some things I have either identified or people have reported. >>> >>> 1- Remove ->register() and ->unregister(), and make the >>> spl_autoload_register to support SplClassLoader. >>> >>> I'm really +0 on this one. >>> But since the proposal covers an OO approach for autoload, it makes >>> sense the registering is pat of the available API, not in procedural >>> land. >>> >>> 2- Remove constructor prototype in interface >>> >>> After some thought it makes sense, but only if interface then defines >>> the setMode prototype. >>> The background for this is supported if the user decides to enhance >>> the base class to support caching, requiring the injection of the >>> Cache layer in constructor. If it's part of the interface, it cannot >>> be changed. >>> I took this example after looking at Symfony's >>> ApcUniversalClassLoader: >>> https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Cl= assLoader/ApcUniversalClassLoader.php >>> >>> >>> What do you think about these 2 points? >>> >>> Even if you're against the proposal, for sure you can still help to >>> make it consistent. >>> >>> >>> Cheers, >>> >>> On Tue, Nov 8, 2011 at 3:39 PM, Nikita Popov wrote: >>>> On Tue, Nov 8, 2011 at 6:28 PM, guilhermeblanco@gmail.com >>>> wrote: >>>>> Because there's no need to bring to C a single foreach. >>>>> Also, if you re-read the RFC, you'll see that SplClassLoader is >>>>> extendable for personalized developer needs, such as an addAll or an >>>>> APC lookup. >>>> After your changes the RFC looks much more decent. I am still opposed >>>> to the idea of this going into core (mainly because there is no >>>> necessity), but now the implementation is somewhat more useful. >>>> >>>> Nikita >>>> >>> >>> >>> >>> -- >>> Guilherme Blanco >>> Mobile: +55 (11) 8118-4422 >>> MSN: guilhermeblanco@hotmail.com >>> S=C3=A3o Paulo - SP/Brazil >>> >> >> >> >> -- >> Guilherme Blanco >> Mobile: +55 (11) 8118-4422 >> MSN: guilhermeblanco@hotmail.com >> S=C3=A3o Paulo - SP/Brazil >> > > > > -- > Guilherme Blanco > Mobile: +55 (11) 8118-4422 > MSN: guilhermeblanco@hotmail.com > S=C3=A3o Paulo - SP/Brazil > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php >