Re: [RFC] Generalize support of negative string offsets

From: Date: Mon, 15 Feb 2016 09:26:42 +0000
Subject: Re: [RFC] Generalize support of negative string offsets
References: 1 2 3 4 5 6 7  Groups: php.internals 
Request: Send a blank email to internals+get-91235@lists.php.net to get a copy of this message
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.

« previous php.internals (#91235) next »