Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:68288
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.182 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <403E6D88-8057-48F1-A742-73D026CB1760@wiedler.ch>
References: <60BF8DD5-FEEA-47D9-834F-6C7FDEF3B879@wiedler.ch>
 <CALtuQzB9Jm=h9-UNAJ5qGzjKd2XVhVL_uxWC1ssi+ycpRixzzQ@mail.gmail.com> <403E6D88-8057-48F1-A742-73D026CB1760@wiedler.ch>
Date: Tue, 23 Jul 2013 22:42:53 +0200
Message-ID: <CALtuQzBNiFYUuzKFjQnTDOuXqn1SrE26dStAfXgQhoayDVvJdw@mail.gmail.com>
To: Igor Wiedler <igor@wiedler.ch>
Cc: PHP internals list <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=f46d040fa1d4b220b104e233d870
Subject: Re: [PHP-DEV] [RFC] Importing namespaced functions
From: krebs.seb@gmail.com (Sebastian Krebs)

--f46d040fa1d4b220b104e233d870
Content-Type: text/plain; charset=ISO-8859-1

Hi,

Thanks for that.

But actually I don't see: Why is "use" not enough? As far as I can see your
example, why it would introduce a BC, doesn't really match to original
question, why "use function" is used. Especially I don't see any ambiguity:

foo(); // Always a function. You cannot call classes this way
new foo(); // always a class
foo::bar(); // Always a class

But the example I mentioned before

namespace {
    function bar() {}}
 namespace foo {
    function bar() {}}
 namespace {
    use foo\bar;
    bar();}

points to a different problem, which I didn't see solved by adding the
"function"-keyword to "use". Can you clarify this? Because I'd try to avoid
new syntax wherever possible.

Regards,
Sebastian


2013/7/23 Igor Wiedler <igor@wiedler.ch>

> Hi,
>
> Based on Sebastian's feedback I have updated the RFC and the patch to
> include a `use const` sequence that works just like `use function`, but for
> namespaced constants.
>
> Example usage:
>
>     namespace foo\bar {
>         const baz = 42;
>     }
>
>     namespace {
>         use const foo\bar\baz;
>         var_dump(baz);
>     }
>
> I also fixed some other issues that the original patch had in the process
> (see commit history). Please keep the feedback coming.
>
> Thanks,
>
> Igor
>
>


-- 
github.com/KingCrunch

--f46d040fa1d4b220b104e233d870--