Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:103402
MIME-Version: 1.0
References: <CAGU04mJf7MdrOkRy37ii=ZmgXtDWBvkG=11gvwWCPfpw88EWYw@mail.gmail.com>
 <2370262.6EcsR2toYx@vulcan> <CAJW__o02dXw4B-DwuCH=-F2=i+OwHvZ5KLTm+Nu_J_mOpbXGQA@mail.gmail.com>
 <CAFMT4NpoK7vdHrFzbFUTTSts0F5koo=B4qcFv8xv0L_C0d3mDQ@mail.gmail.com>
In-Reply-To: <CAFMT4NpoK7vdHrFzbFUTTSts0F5koo=B4qcFv8xv0L_C0d3mDQ@mail.gmail.com>
Date: Fri, 26 Oct 2018 23:09:41 +0200
Message-ID: <CAJW__o11Q4ypxH3OoqXes-Lv+O2g86qVCvcf2zEZkkGd5CyB0Q@mail.gmail.com>
To: Levi Morrison <levim@php.net>
Cc: Larry Garfield <larry@garfieldtech.com>, Internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [PHP-DEV] Idea for better function callbacks (another syntactic sugar)
From: kalle@php.net (Kalle Sommer Nielsen)

Den fre. 26. okt. 2018 kl. 18.03 skrev Levi Morrison <levim@php.net>:
> In my opinion "ideal" here is that our symbol tables are merged, so
> referring to "trim" in any context will resolve to at most one symbol,
> *and* that we also have a short-closure syntax for times when there
> isn't a *perfect* function to use.
>
> In the next version of PHP I would really like to see some warnings
> for when symbol names get re-used so that in the future we can merge
> them with less of a BC break.

I like that idea a lot tbh, but I do worry a lot about the lookup time
if we end up with a gigantic hashtable to hold all the symbols and the
implications it has. In most contexts we know (based on the syntax)
what we are looking for, however as it currently stands then it would
not be clear wether to look for the constant "trim" or the procedural
function "trim" in the example given, and if thats all the same, I can
see that many things not utilizing this could be slower for no good
reason (at least in my honest opinion). However I don't know if there
is something sneaky we can do here.



-- 
regards,

Kalle Sommer Nielsen
kalle@php.net