Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:121333
MIME-Version: 1.0
References: <CAL-Ucw7H4zcbnhUJHHPi9=KX9NtPEVKiCOyNz+TTJfdwkzLRTw@mail.gmail.com>
 <c4b40b7a-aa8b-470f-ae07-1485c010d907@bastelstu.be> <A14E0D76-9EB2-4881-8F9C-4A2848EFCCFF@cschneid.com>
In-Reply-To: <A14E0D76-9EB2-4881-8F9C-4A2848EFCCFF@cschneid.com>
Date: Mon, 16 Oct 2023 13:26:26 +0100
Message-ID: <CAFPFaMLFQ+xfeNs5Y=rr9R7Bhp-XFc6S99hq1qAR4etuqHLqMQ@mail.gmail.com>
To: Christian Schneider <cschneid@cschneid.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000008758c40607d484b0"
Subject: Re: [PHP-DEV] Better name for method Randomizer::nextFloat()
From: george.banyard@gmail.com ("G. P. B.")

--0000000000008758c40607d484b0
Content-Type: text/plain; charset="UTF-8"

On Mon, 16 Oct 2023 at 09:34, Christian Schneider <cschneid@cschneid.com>
wrote:

> > Side note: The implementation of nextFloat() is much more efficient than
> that of getFloat(0.0, 1.0) and the output will differ even for the same
> seeded engine. The domain of legal return values is identical.
>
> This sounds like an implementation detail as I'm pretty sure you could
> switch to that fast path easily enough if neither of the arguments were
> given.
>

This is not how the engine works, it would require an if-clause on every
single call of getFloat() that would probably defeat the point of it.

Best regards,

George P. Banyard

--0000000000008758c40607d484b0--