Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:121179
MIME-Version: 1.0
References: <1FF8038C-908B-456F-A73D-81363111C724@sakiot.com>
In-Reply-To: <1FF8038C-908B-456F-A73D-81363111C724@sakiot.com>
Date: Fri, 29 Sep 2023 18:35:13 +0100
Message-ID: <CAFPFaM+mf2oheTgLD7atWjr+oZsbWg3X3zHqO53=o1OS5-NLaw@mail.gmail.com>
To: Saki Takamachi <saki@sakiot.com>
Cc: internals@lists.php.net
Content-Type: multipart/alternative; boundary="00000000000083ed67060682d9f8"
Subject: Re: [PHP-DEV] Add bcround, bcfloor, bcceil to BCMath.
From: george.banyard@gmail.com ("G. P. B.")

--00000000000083ed67060682d9f8
Content-Type: text/plain; charset="UTF-8"

On Tue, 26 Sept 2023 at 11:10, Saki Takamachi <saki@sakiot.com> wrote:

> Hi, internals.
>
> I'm currently working on renovating the `round()` function.
>
> The current `round()` function rounds a value like 0.285 (0.28499....) to
> 0.29 using pre-round.
> This is a mistake for FP, so discussions are underway to abolish such
> pre-round behavior.
>
> However, there is definitely a user request to round 0.285 to 0.29, so I
> thought why not add a function similar to the round function to BCMath.
>
> Similarly, I'm considering adding similar functionality to `floor()` and
> `ceil()`. In rare cases, when dealing with very large values, these will
> return a different value than expected as a decimal number.
>
> I would love to hear everyone's opinions.
>

I think regardless of the issues around round(), ceil(), floor(), etc.
having the equivalent functions for BCMath makes a lot of sense to me.

Best regards,

George P. Banyard

--00000000000083ed67060682d9f8--