Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:77091
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.177 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAEZPtU5qsvpAY5i6AFpgAsiFgjKmtOQ+Fzk=hx01r9+3QfOuFw@mail.gmail.com>
References: <F3528124-17C4-4DB1-B11A-E9132775A144@ajf.me>
	<CAEZPtU5qsvpAY5i6AFpgAsiFgjKmtOQ+Fzk=hx01r9+3QfOuFw@mail.gmail.com>
Date: Sun, 7 Sep 2014 08:22:14 -0400
Message-ID: <CAL+t6cHp8syWG660dMD_3D9c0wiAWxqzv1V8qdtKh7D3iGrruQ@mail.gmail.com>
To: Pierre Joye <pierre.php@gmail.com>
Cc: Andrea Faulds <ajf@ajf.me>, PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=047d7bae49569df8b8050278c0d1
Subject: Re: [PHP-DEV] [RFC] Implicit isset() in Shorthand Ternary Operator
From: theanomaly.is@gmail.com (Sherif Ramadan)

--047d7bae49569df8b8050278c0d1
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

I've played around with this branch for a bit and seems reasonable, passes
the tests, and doesn't seem to have any serious issues/memories leaks
AFAICT.

I can't stress enough, however, how important it is for us to avoid
inconsistencies in behavior between various forms of syntax. This is often
something users complain of as often they complain about not getting such
features. So let's make ternary consistent before we implement yet another
oddity in PHP (half-assed and ill designed).

I don't actually agree with the premise that ternary should wrap the first
operand in empty() to begin with, but if this is what people want, fine.
Let's just do it right.

And as Rasmus mentioned in the PR, make the documentation very _cleary_ if
it gets accepted.


On Sun, Sep 7, 2014 at 6:30 AM, Pierre Joye <pierre.php@gmail.com> wrote:

> Hi Andrea,
>
> On Sep 6, 2014 2:03 AM, "Andrea Faulds" <ajf@ajf.me> wrote:
> >
> > Good evening,
> >
> > I=E2=80=99ve written an RFC and working patch which attempt to add this=
 feature
> which has been often requested: https://wiki.php.net/rfc/isset_ternary
>
> I did not look at the patch yet but I like it from a feature point of vie=
w.
> We should have had this since long :)
>
> Cheers,
> Pierre
>

--047d7bae49569df8b8050278c0d1--