Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:115193
MIME-Version: 1.0
In-Reply-To: <b256ac45-e893-90b9-70ba-89fd326c1cf7@gmail.com>
References: <d50eaa78-21c7-46e7-9da6-629ba5694d3f@www.fastmail.com>
 <CAJpLVh0Fo9odHeO_CoSxCKA+2dELhG7-H9BvY--tfd59q0iXhQ@mail.gmail.com> <b256ac45-e893-90b9-70ba-89fd326c1cf7@gmail.com>
Date: Mon, 28 Jun 2021 22:28:35 +0200
Message-ID: <CAJpLVh0O1i_2OKG-uvVehbvPpk++n1-W+fNrBpa7_3v_arjhpA@mail.gmail.com>
To: Rowan Tommins <rowan.collins@gmail.com>
Cc: internals@lists.php.net
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] [RFC] Pipe Operator, take 2
From: olleharstedt@gmail.com (=?UTF-8?Q?Olle_H=C3=A4rstedt?=)

2021-06-28 22:04 GMT+02:00, Rowan Tommins <rowan.collins@gmail.com>:
> On 28/06/2021 20:25, Olle H=C3=A4rstedt wrote:
>> Usage (ignoring the pesky undefined constant warnings ><):
>
>
> Unfortunately, you can't ignore those warnings; not least because
> they're fatal errors in PHP 8, as they frankly should have been in PHP 3.
>
> You can use our current ugly callable syntax (strings and two-element
> arrays); you can tackle the complexity of unifying functions with
> constants, and methods with properties (as Levi explained in the other
> thread); or you can add a dedicated callable syntax, which the PFA
> proposal gets us with bells on.

Sorry for hijacking the thread, but are there no other alternatives,
really? Just brainstorming:

1) Setting to silence the warning. Probably frowned upon, AND requires
that functions/methods are case sensitive, which they are not right
now. BUT, constants are always case sensitive since 7.3, so the risk
of collision is not that high...? Assuming constants are by convention
always UPPER_CASE, and functions either snake_case or camelCase. You
can argue that the setting together with a CI to check cases would be
reasonably safe.

2) Setting to silence the warning IF and only if argument expects a
callable, like in array_map (won't work with a pipe() function,
though, since pipe() would take any number of arguments and that can't
be typed)

3) Silence the warning if you type-case explicitly, like in
`pipe($start, (callable) htmlentities);`. Another alternative is
`(fn)` as a type-cast. Not much better than `pipe($start,
htmlentities(?))`, I guess. Just different style.

4) Silence the warning ONLY when it's the right-hand argument to pipe
operator, like `$start |> str_len`.

All stupid?