Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:91235
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.215.53 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAGa2bXYv0GLweN10Y1w4oGqFwE_6-w-Dh6+yusGUs85GRfUMzg@mail.gmail.com>
References: <56A3A01F.1020500@php.net> <56BC988F.7080101@php.net>
 <D9.BC.25203.806BCB65@pb1.pair.com> <56BD1CD4.8070206@php.net>
 <CAF+90c89n8pDCh4O3J5DBTao_8+1ON0TnJTxg-GbbROPorFyGg@mail.gmail.com>
 <56C14AC6.7060307@gmail.com> <CAGa2bXYv0GLweN10Y1w4oGqFwE_6-w-Dh6+yusGUs85GRfUMzg@mail.gmail.com>
Date: Mon, 15 Feb 2016 11:26:42 +0200
Message-ID: <CAL0xaBGtPG6Qg0ELxE3JjHKwZgRSFFu94rPSJNYqjmsLcLur3w@mail.gmail.com>
To: Yasuo Ohgaki <yohgaki@ohgaki.net>, Stanislav Malyshev <smalyshev@gmail.com>
Cc: Andrea Faulds <ajf@ajf.me>, =?UTF-8?Q?Fran=C3=A7ois_Laupretre?= <francois@php.net>, 
	Nikita Popov <nikita.ppv@gmail.com>, PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a114022089207a9052bcb9e79
Subject: Re: [PHP-DEV] [RFC] Generalize support of negative string offsets
From: arvids.godjuks@gmail.com (Arvids Godjuks)

--001a114022089207a9052bcb9e79
Content-Type: text/plain; charset=UTF-8

On Mon, 15 Feb 2016 08:16 Yasuo Ohgaki <yohgaki@ohgaki.net> wrote:

> Hi all,
>
> On Mon, Feb 15, 2016 at 12:49 PM, Stanislav Malyshev
> <smalyshev@gmail.com> wrote:
> >> Another issue mentioned in this thread is the spurious array conversion
> >> that happens for empty strings. We have an existing bug report for this:
> >> https://bugs.php.net/bug.php?id=53432 I've created a PR to fix this
> >> issue: https://github.com/php/php-src/pull/1764 If there are no
> >> objections, I'll merge this for 7.1 as well.
> >
> > This one looks like a bug, converting string to array is really weird.
> > And one more reason to never use [] for string offsets - clearly, the
> > bug stems out of confusion between arrays operation (which expects
> > conversion to array) and string operation (which expects string).
> > In any case, this needs a note in UPGRADING as behavior change.
>
> I fully agree {} and [] usage.
>  - {} only for string
>  - [] only for array
> Simple is better if it satisfies all of our needs.
>
> Since PHP allowed [] for strings, how about allow it, but deprecate it
> by doc in 7.1, deprecate(Raise error) it in 7.2, then remove it by
> 7.3?
>
> Regards,
>
> --
> Yasuo Ohgaki
> yohgaki@ohgaki.net
>
>
 Hello,

As a user-land developer I approve this message (with correction from
Pierre that final removal should be in PHP 8 and keep it deprecated in PHP
7.1 by docs, 7.2 with a error message).

Regards,
Arvids Godjuks,
Areto Development.

--001a114022089207a9052bcb9e79--