Re: [VOTE] Shorter Attribute Syntax Change
| From: | Peter Bowyer | Date: | Mon, 17 Aug 2020 08:17:34 +0000 |
| Subject: | Re: [VOTE] Shorter Attribute Syntax Change | ||
| References: | 1 2 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-111571@lists.php.net to get a copy of this message | ||
On Sun, 16 Aug 2020 at 10:29, Benjamin Eberlei <kontakt@beberlei.de> wrote:
> We have updated the RFC at
> https://wiki.php.net/rfc/shorter_attribute_syntax_change with
> what we
> think
> covers all the discussion and arguments made in this and the previous
> mailing list threads.
>
Thank you for putting in the work it took to revise the RFC. It's good. I
now understand what the delimiters achieve.
While I don't agree with the first point in "Forcing @@ Attributes to end
with parenthesis does not solve issues" (in this new syntax I'd ban
whitespace) I appreciate the point you are making, and it is sensible for
consistency.
I feel grep'ability has been played down, as unless
@@
MyProject\FooAttr
is allowed (which isn't shown in the codeblock), then it's easier to grep
for @@.+?Foo and know you have a chance of an accurate match (assuming
renaming is not used) than it is with the delimiter syntax.
> Sorry to everyone for causing this hazzle.
>
These things happen. Thank you for taking on-board the feedback and working
on the RFC.
Peter