Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:121617
MIME-Version: 1.0
References: <CAEKnhAGXnotsxHaAWxrkm42VdEg3XWHtJqKicG=F2CSS4P84dQ@mail.gmail.com>
 <F792AB1C-39F9-4760-9183-279A48C9C693@sakiot.com> <51BC0083-B54F-4601-AD2A-4FDE7003ADF7@sakiot.com>
In-Reply-To: <51BC0083-B54F-4601-AD2A-4FDE7003ADF7@sakiot.com>
Date: Wed, 8 Nov 2023 09:50:54 +0000
Message-ID: <CAEKnhAGNG-B5Sf4cFCnMVXcFUaP=LjhaY2uJjg9UMm7_ARjGfQ@mail.gmail.com>
To: Saki Takamachi <saki@sakiot.com>
Cc: Michael Wallner <mike@php.net>, internals@lists.php.net
Content-Type: multipart/alternative; boundary="0000000000009e1b150609a106d1"
Subject: Re: [PHP-DEV] [RFC] [Discussion] Change the edge case of round()
From: bukka@php.net (Jakub Zelenka)

--0000000000009e1b150609a106d1
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Wed, Nov 8, 2023 at 9:40=E2=80=AFAM Saki Takamachi <saki@sakiot.com> wro=
te:

> I added and modified into the RFC. I would be happy if you could check it
> out.
>

It looks much better.

It might make sense to also put some info what other languages / dbs do.
For example Python round(0.285, 2) result is 0.28 and it's got also decimal
parameter. If you can find other implementations that have got such
function, that would be great!

Cheers

Jakub

--0000000000009e1b150609a106d1--