Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:94624
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: softfail (pb1.pair.com: domain golemon.com does not designate 209.85.214.49 as permitted sender)
MIME-Version: 1.0
Sender: php@golemon.com
In-Reply-To: <5024e8b2-4a4d-4a12-99f3-a539c9930337@gmail.com>
References: <CAESVnVoBrtfWGF-Be+uXWwLNO3=JzuLBXyhW0Dpo-NvoskofQw@mail.gmail.com>
 <71557808-5718-0406-3df9-830bc2e03006@gmail.com> <CAESVnVosKc=-yfF3oAP3Oo9Z4F9_Fcv9Nu12S6SOPWykO6ZX5w@mail.gmail.com>
 <5024e8b2-4a4d-4a12-99f3-a539c9930337@gmail.com>
Date: Thu, 21 Jul 2016 22:52:26 -0700
Message-ID: <CAESVnVp9JeHjODR6vTiYAOsOOs7dw6D95zgxQ3yVhCvKGDr7xw@mail.gmail.com>
To: Rowan Collins <rowan.collins@gmail.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [PHP-DEV] Pipe Operator v2
From: pollita@php.net (Sara Golemon)

On Thu, Jul 21, 2016 at 2:56 PM, Rowan Collins <rowan.collins@gmail.com> wrote:
> On 21/07/2016 22:28, Sara Golemon wrote:
>>
>> Are you picturing the return statement returning from the current
>> function? Or "returning" from the current pipe chain?
>>
>> I think you mean the former, in which case I'd ask how that'd better/worse
>> than:
>>
>> return foo() |> bar($$);
>
> I do indeed mean returning from the current function. I went into more
> detail (perhaps too much!) on the previous thread, but the basic premise is
> the pipe operator gives you code that reads left to right. Adding an element
> where you have to jump back to the beginning of the expression feels like it
> defeats that.
>
Ah! I understand your point now.  Indeed, in terms of code-scan,
there's something of a whiplash effect going on. Now I understand the
motivation behind $> assignment and/or using a terminal like `return
$$` at the end.

Adding in support for some terminals (like return) could make sense,
it's an exception to a rule, but it's a reasonable looking one (at
first glance, anyway).

The assign to var at the end however, is something I'd rather handle
with the following:

foo() |> bar($$) |> $baz = qux($$);

Which is both legal with the syntax as-is, and entirely readable
without adding an extra operator.

-Sara