Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:78888
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain adamharvey.name designates 209.85.223.171 as permitted sender)
MIME-Version: 1.0
Sender: adam@adamharvey.name
In-Reply-To: <001101cffda8$9f56e980$de04bc80$@tutteli.ch>
References: <000201cff3b2$37615e50$a6241af0$@tutteli.ch> <001101cffda8$9f56e980$de04bc80$@tutteli.ch>
Date: Wed, 12 Nov 2014 12:45:08 -0800
Message-ID: <CADajX4B-mFu=ETAkHGYD4quDu+Ddegi8gPkHiX5Lmqv9Sr7daA@mail.gmail.com>
To: Robert Stoll <php@tutteli.ch>
Cc: PHP Internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [PHP-DEV] forbid use declaration outside of a namespace in PHP 7
From: aharvey@php.net (Adam Harvey)

On 11 November 2014 04:11, Robert Stoll <php@tutteli.ch> wrote:
>> I always found it very ugly that it is possible to define a use outside of a namespace. Consider the following:
>>
>> namespace{ //default namespace
>> }
>>
>> use foo\Bar;
>>
>> namespace test{
>>   new Bar(); //error, test\Bar not found }
>>
>> After some thoughts it is quite clear that Bar is test\Bar and not foo\Bar inside of the namespace test. But consider
> the
>> following example which is not so obvious:
>>
>> use foo\Bar;
>> namespace test;
>> new Bar(); //error, test\Bar not found
>>
>> The use declaration looks like a normal use declaration at first glance.
>> I do not see why we should actually support this "feature" any longer and thus suggest to remove it in PHP 7.
> Although, it is
>> not a bug (the use declaration is simply ignored as far as I can tell) I suppose it confuses the user.
>> Nevertheless, even if we declare it as a "feature" I think it should at least not be a "feature" of the specification
> of PHP 7.

Sorry, I apparently missed this the first time. Would this mean that
this sort of script (where we're using use statements without a
namespace) would no longer work?

<?php
use GuzzleHttp\Client;

include __DIR__.'/vendor/autoload.php';

$client = new Client;
// $client is a GuzzleHttp\Client object
?>

Because that strikes me as an irritating and unnecessary BC break for
people who are writing throwaway scripts (with no need to live in
namespaces) that pull in namespaced libraries to me.

Adam