Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112500
MIME-Version: 1.0
References: <b6e84a3b-012e-4ef3-8c2b-14c7fcf2d45d@www.fastmail.com> <CADyq6s+pe0NDvL22HTCinAcCFNxJc6ZozMRn_b0mhJT15tPPxA@mail.gmail.com>
In-Reply-To: <CADyq6s+pe0NDvL22HTCinAcCFNxJc6ZozMRn_b0mhJT15tPPxA@mail.gmail.com>
Date: Mon, 14 Dec 2020 20:24:15 +0000
Message-ID: <CAEiu4gzw8Q__VZADPbacNz4N4ccoqEvJVfqr37s0+FKZPVo05A@mail.gmail.com>
To: Marco Pivetta <ocramius@gmail.com>
Cc: Larry Garfield <larry@garfieldtech.com>, php internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000b0197605b6726d35"
Subject: Re: [PHP-DEV] [RFC] Short-match
From: dougnelson@silktide.com (Doug Nelson)

--000000000000b0197605b6726d35
Content-Type: text/plain; charset="UTF-8"

Hi Marco,

Is there any chance you can elaborate on why you feel it's a bad idea?

Both you and Sara at different points have talked about thinking was bad
practice, but I've not read anything compelling about why it should be
considered as such.

Kind regards,
Doug Nelson

On Mon, 14 Dec 2020 at 18:14, Marco Pivetta <ocramius@gmail.com> wrote:

> Hey Larry,
>
> On Mon, Dec 14, 2020 at 6:34 PM Larry Garfield <larry@garfieldtech.com>
> wrote:
>
> > I present to Internals this tiny RFC to follow up on the match()
> > expression RFC from earlier in the year.  There was solidly positive
> > support for this shortcut previously but it was removed for simplicity at
> > the time, with the intent to bring it back later.  It's now later.
> >
> > https://wiki.php.net/rfc/short-match
> >
>
> Overall back to the `switch (true)` case, which IMO is not a good idea in
> first place :|
>
> Nothing wrong with using a set of conditional that are put in a sequence
> through `if ()`, for these rare cases.
>
> Marco Pivetta
>
> http://twitter.com/Ocramius
>
> http://ocramius.github.com/
>


-- 
Doug Nelson
Senior Software Engineer

--000000000000b0197605b6726d35--