Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:104760
MIME-Version: 1.0
References: <OF2EDD3BDE.BAC08076-ON432583BE.004A746C-432583BE.004B009B@alfabank.ru>
 <CAF+90c8Xn7yZEkER_dPq9Dv6w_p1dsg6DtF7goo6+4Tb5GwG_Q@mail.gmail.com>
 <CAJW__o3w8sdxbrjF7PFDAXP2b31-LMad0_1BQGfvcgFvRqEh9g@mail.gmail.com> <CAFMT4NqNQuM2Xdfap=kVakF9aLJpv+eK1==iE+gKHiTV1bKcPA@mail.gmail.com>
In-Reply-To: <CAFMT4NqNQuM2Xdfap=kVakF9aLJpv+eK1==iE+gKHiTV1bKcPA@mail.gmail.com>
Date: Fri, 15 Mar 2019 20:23:32 +0200
Message-ID: <CAJW__o1wtNhzB8d1mEAzvJwUT0F935gB8Bk_HB+HADPtxoqzyA@mail.gmail.com>
To: Levi Morrison <levim@php.net>
Cc: Nikita Popov <nikita.ppv@gmail.com>, Andrey O Gromov <AGromov@alfabank.ru>, 
	PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [PHP-DEV] Proposal for deprecate alternate syntax(curly braces)
 for array and string access
From: kalle@php.net (Kalle Sommer Nielsen)

Den fre. 15. mar. 2019 kl. 19.53 skrev Levi Morrison <levim@php.net>:
> Four thoughts:
>
>   1. I cannot think of *any* reason to separate them. If you want to
> make sure you are indexing a string, *do a type check*, not bake `{}`
> into only working on strings.
>   2. While Kalle says pretty much every codebase they've seen lately
> has used `{}` for indexing, I have *never* seen a codebase that used
> it.

Well maybe that happens to be just me, not a huge amount of usages but
there is usually some.

>   3. In any case, usage of `{}`  can be migrated to `[]` by a style
> fixer, right?
>   4. Even if we deprecate `{}`, I don't think we'd be in any hurry to
> remove it, though with an automatic fixer this seems doable, if we
> care to do it at all.

The thing is, I just don't understand the rationale to remove this
feature, what the full gain from doing so is?

-- 
regards,

Kalle Sommer Nielsen
kalle@php.net