Re: Re: want an Object-oriented interface for HashContext

From: Date: Wed, 21 Oct 2020 18:49:16 +0000
Subject: Re: Re: want an Object-oriented interface for HashContext
References: 1 2 3 4 5 6  Groups: php.internals 
Request: Send a blank email to internals+get-112091@lists.php.net to get a copy of this message
@Rowan Tommins >I think I'd prefer to drop the fluent interface (returning void on most > things, and relevant values on others) well both approaches has pros and cons, that said, the sample implementation Sara posted returns $this, if you (or anyone) feels strongly about it one way or the other, i guess the php devs can vote on it @Sara Golemon thanks! (and you're the original pecl hash author, neat) given the "PHP8 release manager" thing, i guess you have a lot of other PHP8 stuff to focus on right now > Happy to co-author an RFC with you if you'd like to get involved directly, or I can just > pick up the ball and run with it if you'd rather make the feature request then step back. stepping back sounds good to me, i have no idea how to do any of that stuff ^^ > No reply from Hans, so I've thrown together an implementation and will attach an RFC to it > in the coming days. > https://github.com/php/php-src/pull/6347 looks great to me, and you even took care of the hash_update_stream return value issue ^^ On Sat, 17 Oct 2020 at 04:52, Sara Golemon <pollita@php.net> wrote: > > On Tue, Oct 13, 2020 at 12:04 PM Sara Golemon <pollita@php.net> wrote: >> >> On Tue, Oct 13, 2020 at 11:42 AM Christoph M. Becker <cmbecker69@gmx.de> wrote: >>> >>> >>> > Happy to co-author an RFC with you if you'd like to get involved directly, >>> > or I can just pick up the ball and run with it if you'd rather make the >>> > feature request then step back. >>> >>> Please go with this, if you like. Note that I didn't make the feature >>> request; I just support it. :) >>> >> >> Sorry for confusion. That part of my reply was aimed at Hans, but I didn't have his >> original message in my thread to reply to. :p >> > > No reply from Hans, so I've thrown together an implementation and will attach an RFC to it > in the coming days. > https://github.com/php/php-src/pull/6347 > > -Sara

« previous php.internals (#112091) next »