Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112502
MIME-Version: 1.0
References: <b6e84a3b-012e-4ef3-8c2b-14c7fcf2d45d@www.fastmail.com>
 <CADyq6s+pe0NDvL22HTCinAcCFNxJc6ZozMRn_b0mhJT15tPPxA@mail.gmail.com>
 <CAEiu4gzw8Q__VZADPbacNz4N4ccoqEvJVfqr37s0+FKZPVo05A@mail.gmail.com> <CADyq6s+Jvw+0K4qU+EQLaO8V-t0vFHa8xTr3Mnn5QerrQesaSg@mail.gmail.com>
In-Reply-To: <CADyq6s+Jvw+0K4qU+EQLaO8V-t0vFHa8xTr3Mnn5QerrQesaSg@mail.gmail.com>
Date: Mon, 14 Dec 2020 22:15:51 +0100
Message-ID: <CAJpLVh3HAdwBQdA+HKuZrao6OoZoxHLPo+=rUxsAymL1K=u_vw@mail.gmail.com>
To: Marco Pivetta <Ocramius@gmail.com>
Cc: Doug Nelson <dougnelson@silktide.com>, Larry Garfield <larry@garfieldtech.com>, 
	php internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="00000000000057480d05b67326db"
Subject: Re: [PHP-DEV] [RFC] Short-match
From: olleharstedt@gmail.com (=?UTF-8?Q?Olle_H=C3=A4rstedt?=)

--00000000000057480d05b67326db
Content-Type: text/plain; charset="UTF-8"

Another situation in which _ might be useful. match (_) { ... }

On Mon, 14 Dec 2020, 21:43 Marco Pivetta, <ocramius@gmail.com> wrote:

> Hey Doug,
>
>
>
>
> On Mon, Dec 14, 2020, 21:24 Doug Nelson <dougnelson@silktide.com> wrote:
>
> > 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
> >
>
>
> It's effectively yet another way to write something that already exists,
> which means more syntax to learn, more syntax to deal with (in downstream
> tooling), and that for an edge case being a fall-through list of boolean
> expressions, which is what already happens by using `if` with early return
> statements.
>
> Let me rephrase it then:
>
>  * It brings little to no advantage to the table for something seen very
> rarely (`match` already being super-rare, since adoption didn't even start
> yet)
>  * It increases language complexity
>
> Every time new AST is to be added, that complexity needs to be weighed
> against the benefits, and I don't see any compelling ones here.
>

--00000000000057480d05b67326db--