Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:111479
MIME-Version: 1.0
References: <alpine.DEB.2.23.453.2008100935570.7489@singlemalt.home.derickrethans.nl>
 <CAESVnVqvVG4dXDbj25YyA4tK=7v0HiEuNbMDZTBCDuAwqSGH0g@mail.gmail.com>
 <CAFp73XsjYXrPO3NCGdNb_8f1EzoWQYT1kJavCgTPA=ZtxRGsmA@mail.gmail.com>
 <CAESVnVo3z0oKkbVF0nvcAgUgAoFyWeF_3v0a402_Ffw3pp3N5g@mail.gmail.com> <59019606-a3fd-3c19-e962-a847689244a9@gmx.net>
In-Reply-To: <59019606-a3fd-3c19-e962-a847689244a9@gmx.net>
Date: Tue, 11 Aug 2020 11:38:10 -0500
Message-ID: <CAESVnVpLyny1L41SrM0t-0V8=akR6N8kytgOdyEqhJPSVTMUCQ@mail.gmail.com>
To: Andreas Leathley <a.leathley@gmx.net>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000fed2c305ac9cb248"
Subject: Re: [PHP-DEV] [VOTE] Shorter Attribute Syntax Change
From: pollita@php.net (Sara Golemon)

--000000000000fed2c305ac9cb248
Content-Type: text/plain; charset="UTF-8"

On Tue, Aug 11, 2020 at 11:25 AM Andreas Leathley <a.leathley@gmx.net>
wrote:

> This would be a feature if libraries start parsing PHP token by token
> and start supporting the #[] syntax for the 7.x versions of PHP - then
> early adoption would be possible and it would be a real feature. But it
> seems unclear if that will happen, as it will probably have a heavy
> price on performance (and might have some complexity). And if some
> libraries implement it but others don't, then it might get confusing for
> users about why the new attribute syntax sometimes works and sometimes
> silently does nothing.
>
>
The perf penalty on 7 and earlier would probably be similar to existing
state of the world.
Parsing a docblock is easier to fetch from the runtime (as we actually
store it), but docblocks contain more than just annotations, so some plus
some minus.
PHP 8+ performance on theses would certainly be between though, and that's
an extra carrot to push users to upgrade.  If annotations can't be written
until after users upgrade, then that carrot vanishes.


> It is also not perfect as multi-line attributes with #[ still break, or
> code with #[] followed by more code instead of a newline and then more
> code.
>
>
That's true. It'll take some care (and possible reduction in readability)
to make annotations work well across verisons, but it's better to have an
option than not, no?
-Sara

--000000000000fed2c305ac9cb248--