Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112046
MIME-Version: 1.0
References: <CAJmy8YGSWkzEu1g2c9o=+5jSHEXGb=vW0dKcEwrHtKGPyC+eCA@mail.gmail.com>
 <ffa2774b-757e-879d-7bfa-dc554e4cc04a@gmx.de> <CADq_CxrGRb27DCH-jPDNOQaO7z+ka0bO-WfwhpgTQMp4aeys7A@mail.gmail.com>
 <CAFPFaMJ0GddrCXRtP1UT_s9myEeQvZ6RnNhL4Ov9cN5DdEBbKQ@mail.gmail.com>
 <CALKiJKpXM=ObDQ3uP0se_0qcy=_zRJMM9Crgjg-qgEKOJ2KcuQ@mail.gmail.com>
 <CAJmy8YEu+N3NB6WAkvTbY7LG8x59c0VB0_rr8iHwTzhpYeDr_Q@mail.gmail.com>
 <CALKiJKo=M3NkQJ3dYs0UCAmh0sCj+bg7DjOMbvSw_8u8peXiGA@mail.gmail.com> <CAJmy8YF=nQ4w3k=Gr9q6voh8L0=RYz3LfT7cyCyyeUdfma_6YA@mail.gmail.com>
In-Reply-To: <CAJmy8YF=nQ4w3k=Gr9q6voh8L0=RYz3LfT7cyCyyeUdfma_6YA@mail.gmail.com>
Date: Tue, 13 Oct 2020 16:43:39 +0100
Message-ID: <CALKiJKqJ9jO561YeObs2NBgbbLHD4kYwjKswTsZk-q2=bcTfKg@mail.gmail.com>
To: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000ffaf3405b18f47c8"
Subject: Re: [PHP-DEV] Re: want an Object-oriented interface for HashContext
From: rowan.collins@gmail.com (Rowan Tommins)

--000000000000ffaf3405b18f47c8
Content-Type: text/plain; charset="UTF-8"

On Tue, 13 Oct 2020 at 16:33, Hans Henrik Bergan <divinity76@gmail.com>
wrote:

> also the sample class in the 3v4l link has an issue i hadn't thought
> of: i wanted to support method chaining, so i made nearly everything
> return `:self` , but hash_update_stream() returns int, the number of
> bytes actually read from the stream. that could easily be fixed with a
> `updateStream($stream, int $length = -1, int &$bytesRead = null):self`
> or something tho
>


I think I'd prefer to drop the fluent interface (returning void on most
things, and relevant values on others) than have to use reference
parameters. Since PHP doesn't have actual "out" parameters, references like
that are awkward for static analysis; and simple things like using them in
conditions becomes much more verbose:

$result  = null;
$foo->updateStream($bar, bytesRead: $result)
if ( $result > 42 ) ...

vs

if ( $foo->updateStream($bar) > 42 ) ...

Regards,
-- 
Rowan Tommins
[IMSoP]

--000000000000ffaf3405b18f47c8--