Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:127376
Feedback-ID: i8414410d:Fastmail
Precedence: bulk
MIME-Version: 1.0
Date: Thu, 15 May 2025 08:45:32 -0500
To: "php internals" <internals@lists.php.net>
Message-ID: <03a18250-860c-4a62-90b0-57dc084f176b@app.fastmail.com>
In-Reply-To: 
 <DS7P223MB13654D98D660863F7221EF03DE90A@DS7P223MB1365.NAMP223.PROD.OUTLOOK.COM>
References: <c77374f9-d638-418e-8151-d4fdfbbd4561@app.fastmail.com>
 <CAPyj-LCx6iNBruKCsieejYuszOoj5s30d3hQfpzwHcN8zZLnDA@mail.gmail.com>
 <9a1ece99-3528-4084-8aa8-d41a07b46a3f@app.fastmail.com>
 <DS7P223MB13654D98D660863F7221EF03DE90A@DS7P223MB1365.NAMP223.PROD.OUTLOOK.COM>
Subject: Re: [PHP-DEV] [Vote] Pipe operator
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
From: larry@garfieldtech.com ("Larry Garfield")

On Wed, May 14, 2025, at 10:36 PM, Theodore Brown wrote:
> On Wed, May 14, 2025 at 09:08 Larry Garfield wrote:
>
>> On Tue, May 13, 2025, at 5:55 PM, Ilija Tovilo wrote:
>>> On Tue, May 13, 2025 at 3:58=E2=80=AFAM Larry Garfield <larry@garfie=
ldtech.com> wrote:
>>>>
>>>> I hereby open the vote on the Pipe operator RFC:
>>>>
>>>> https://wiki.php.net/rfc/pipe-operator-v3
>>>
>>> It looks like the example under "Rejected Features" is wrong.
>>>
>>> // With Elixir style
>>> $foo
>>>   |> bar(...)
>>>
>>> should be
>>>
>>> $foo
>>>   |> bar()
>>>
>>> Also a nit: The RFC says "There was discussion off-list of
>>> =E2=80=9Cauto-partialling=E2=80=9D functions after a pipe", but this=
 idea was since
>>> discussed on-list as well.
>>>
>>> Ilija
>>
>> Hm, good catch.  Normally you're not supposed to edit an RFC in votin=
g,
>> but as this is a typo fix in a Rejected Features section (ie, it has
>> zero impact on the thing being voted on), I think it's probably OK to=
 fix.
>>
>> I'll give a day or so for someone to object that it still shouldn't b=
e allowed,
>> and if no one does then I'll make those edits.
>>
>> --Larry Garfield
>
> I think it's fine to fix typos, misspellings, and grammar mistakes=20
> which don't affect what's being voted on.
>
> While you're at it, please also fix the `match()` example, which shoul=
d=20
> not contain `case` keywords.
>
> Theodore

I have made the above corrections.  No substantive changes were made to =
the RFC.

For verification, here's the diff: https://wiki.php.net/rfc/pipe-operato=
r-v3?do=3Ddiff&rev2%5B0%5D=3D1747101303&rev2%5B1%5D=3D1747316653&difftyp=
e=3Dsidebyside

--Larry Garfield