Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:59999
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain googlemail.com designates 209.85.214.170 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <4F8C16B5.7000103@ralphschindler.com>
References: <4F89D4F1.8070009@ralphschindler.com>
	<CAJWj5+EhPkmpWmAz4Yekh794SO4x4sM27VZLk-X9qsDzMP4aNg@mail.gmail.com>
	<4F8B766F.3000702@ralphschindler.com>
	<CAJWj5+Gm0xxuxHdvxt0wEuu9Dyw0CvM3_bejRH6msAdXqz7ueg@mail.gmail.com>
	<4F8C16B5.7000103@ralphschindler.com>
Date: Mon, 16 Apr 2012 15:15:58 +0200
Message-ID: <CAJWj5+Hp0bELN7xhFbPS=u69qC6tdNXvSCgddFFwj-Rubcg7VA@mail.gmail.com>
To: Ralph Schindler <ralph@ralphschindler.com>
Cc: internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] New Feature: Fully qualified class name resolution as
 scalar with class keyword
From: simonsimcity@googlemail.com (Simon Schick)

2012/4/16 Ralph Schindler <ralph@ralphschindler.com>
>
> ... PHP does not invoke the autoloader to determine if the class name
> actually exists as a declaration somewhere, it simply resolves it accordi=
ng
> to some very specific rules (in the case of this patch, carried out by
> zend_resolve_class_name()).
>
> If I am within a namespace and am using a short name without a preceding
> "\", it means the class I am referencing exists in the current namespace.
> =C2=A0Whether or not that class exists is irrelevant as my short name can=
 only
> mean "a class by this name in the current namespace". =C2=A0In this code:
>
> =C2=A0namespace Foo\Bar {
> =C2=A0 =C2=A0global $foo;
> =C2=A0 =C2=A0if ($foo && $foo instance Bar) {
> =C2=A0 =C2=A0 =C2=A0 =C2=A0// ...
> =C2=A0 =C2=A0}
> =C2=A0}
>
> ... that you, the developer, intend that your reference to Baz actually
> means "Foo\Bar\Baz", it can never mean anything else.
>
> So, in a nutshell, there is no guessing going on ;)
> -ralph

Hi, Ralph

Agree on that. I'd like to have a comment in the test, specially for
this line where you're checking for non-existing classes. It does not
seem to be obvious what should be tested here (at least it was not
obvious for me). This may make it easier to fix stuff if this test
should fail in future.
I don't know if this here is the right test to put the comment in, but
it's at least relying on something global that may change.

Thanks for giving me a deeper knowledge here :)

Bye
Simon