Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:108312
MIME-Version: 1.0
References: <DM5PR07MB30670BE1BAF74AAC2B4486DBF9050@DM5PR07MB3067.namprd07.prod.outlook.com>
 <CAEYWF=5hEwx8C=FKzmRS=va0rNtv1Wxx6fMW8g554v-TH6vO7g@mail.gmail.com>
In-Reply-To: <CAEYWF=5hEwx8C=FKzmRS=va0rNtv1Wxx6fMW8g554v-TH6vO7g@mail.gmail.com>
Date: Wed, 29 Jan 2020 10:45:19 +0100
Message-ID: <CANS-=pd_RmaFzYHcmUdmhn6QSf_GnB2ZonpeYK9Hcbx7wWcQCQ@mail.gmail.com>
To: bishop@php.net
Cc: tyson andre <tysonandre775@hotmail.com>, 
	"internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000005a0dcc059d443430"
Subject: Re: [PHP-DEV] [VOTE] declare(function_and_const_lookup='global')
From: kjarli@gmail.com (Lynn)

--0000000000005a0dcc059d443430
Content-Type: text/plain; charset="UTF-8"

On Wed, Jan 29, 2020 at 9:55 AM Bishop Bettini <bishop@php.net> wrote:

> Notwithstanding the excellent work done in the PR, I lean toward a No vote.
>
> Suppose I'm presented with this diff (and only this diff):
>
> $a = array_diff($x, $y);
> + if (count($a)) die('Unexpected differences');
>
> Where does count come from? I can't tell from this diff. It could be local
> to this namespace, or it could be global. In the current implementation, I
> have to scroll to the top of the file, check for what namespace I am in,
> and what use are made. In the proposed implementation, I have also to check
> the top of the file, but now its for
> "declare(function_and_cost_lookup='global')" and the presence (or not) of
> "use function count".
>
> It's not clear to me this is an improvement over status quo.
>
> [...]
>
> What do you think?
>

Hi,

I think that when we're at a point where developers start using existing
core function names for custom functions and they start 'overriding'
things, you already have this problem, regardless of this RFC. You already
have to check whether or not the function was defined via a `use` statement
on top. I don't think your example of the diff would have a difference
between `use` and `declare` when it comes to reviewing.

I would advice developers to avoid using core function names in namespaces
for custom functions, as doing this would open up a whole different can of
worms when it comes to developer experience.

Regards,
Lynn

--0000000000005a0dcc059d443430--