Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:93374
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.47 as permitted sender)
To: Jesse Schalken <me@jesseschalken.com>
References: <CAESVnVpa5j=fagr-wm5+DzKKX=wzTN1p8+Kv3-KtGpuyH51b0A@mail.gmail.com>
 <CAF1yvARCKbLx6QCFnHNzzMSJhGk_vr7kdi1Uook1Z=xKhJhu_w@mail.gmail.com>
 <66d7fe0f-6c0b-c6c0-f30a-a54ccd10e949@gmail.com>
 <CAF1yvASqq5Ycy4n36oDSNf6c+EcOxMCCbts26n6AOuqjEB5tGg@mail.gmail.com>
Cc: PHP internals <internals@lists.php.net>
Message-ID: <b6c435d1-c5ca-14cf-8d8e-d25cabce1fd0@gmail.com>
Date: Tue, 17 May 2016 14:08:08 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101
 Thunderbird/45.1.0
MIME-Version: 1.0
In-Reply-To: <CAF1yvASqq5Ycy4n36oDSNf6c+EcOxMCCbts26n6AOuqjEB5tGg@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] [RFC] Pipe Operator
From: rowan.collins@gmail.com (Rowan Collins)

On 17/05/2016 13:48, Jesse Schalken wrote:
>
> On Tue, May 17, 2016 at 9:14 PM, Rowan Collins <rowan.collins@gmail.com
> <mailto:rowan.collins@gmail.com>> wrote:
>
>     To reiterate my comments from earlier in the thread, I think the
>     "$id = $id" looks really weird here, and spoils the step-by-step
>     layout - and, in this case, the symmetry.
>
>
> We have the same thing already when chaining methods on value/immutable
> objects, like with PSR-7:
>
>     $message = $message
>         ->withHeader('foo', 'bar')
>         ->withAddedHeader('foo', 'baz');
>
>
> So depending on often that pattern is used, many are probably already
> used to seeing "$var = $var   ....", myself included.

A fair point. Although you could of course combine OO chaining with 
syntax chaining:

$message
     ->withHeader('foo', 'bar')
     ->withAddedHeader('foo', 'baz')
     |=> $message;

The longer the chain, the more awkward it is to rewind to the beginning 
to find where the result is being used / saved.

See also my earlier post where I went into why a tail assignment would 
improve the ability to debug pipes, and mix them with non-piped code: 
http://marc.info/?l=php-internals&m=146205812523026&w=2

Again, all of this applies to mixing with existing OO pipes like PSR-7.

Regards,
-- 
Rowan Collins
[IMSoP]