Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:115514
MIME-Version: 1.0
References: <6a2ef792-9624-4355-ad08-dcc7bfa5e19f@www.fastmail.com>
 <CADyq6sJFmNiVSJPTGnjQOXGMFeezRLUCPbM6rJ3jooAVKPfXxA@mail.gmail.com> <83bdd8d9-9962-4995-9aff-4bd9b6b7e9b9@www.fastmail.com>
In-Reply-To: <83bdd8d9-9962-4995-9aff-4bd9b6b7e9b9@www.fastmail.com>
Date: Tue, 20 Jul 2021 11:12:47 +0200
Message-ID: <CADyq6sJtRNETNtkPKnwNmVtSA3Xnugn1hTqU0FK37fvndizu8A@mail.gmail.com>
To: Larry Garfield <larry@garfieldtech.com>
Cc: php internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000c6deae05c78a75e7"
Subject: Re: [PHP-DEV] [Vote] Pipe operator v2
From: ocramius@gmail.com (Marco Pivetta)

--000000000000c6deae05c78a75e7
Content-Type: text/plain; charset="UTF-8"

Hey Larry,

On Sat, Jul 17, 2021 at 6:00 PM Larry Garfield <larry@garfieldtech.com>
wrote:

> Hi Marco.  Thank you for your explanation, even if I naturally disagree.
>
> Out of curiosity, what sort of additional
> power/capability/flexibility/etc. would, in your mind, justify pipe or
> similar being a native feature?  PHP has a *ton* of native features that
> *could* be done in user space, or are simply syntax sugar, but still wildly
> popular and useful as native syntax.  What is your heuristic for that?
>
> (The fact that there are 3-4 user space implementations of pipe-like
> behavior, all incompatible, is one of the reasons why I think standardizing
> it into a common core syntax *is* a good idea, though I know others
> disagree.)
>

I think the pipe operator as a **custom** operator made sense when we had
the entire discussion around placeholder parameters (`$$`) to be used in
the pipeline.

As a plain "chain of functions with one input parameter and one output", it
makes little sense to have it as a custom construct, as it only adds
complexity to the AST.

Greets,

Marco Pivetta

http://twitter.com/Ocramius

http://ocramius.github.com/

--000000000000c6deae05c78a75e7--