Re: RFR: 8271602: Add Math.ceilDiv() family parallel to Math.floorDiv() family [v2]

2021-09-10 Thread Brian Burkhalter
On Fri, 10 Sep 2021 18:42:07 GMT, Raffaello Giulietti wrote: >> This PR ideally continues #5285, which has been closed as a consequence of >> inadvertently removing the branch on my repo. See there for initial >> discussion. >> >> Sorry for the mess. > > Raffaello Giulietti has updated the

Re: RFR: 8271602: Add Math.ceilDiv() family parallel to Math.floorDiv() family [v2]

2021-09-10 Thread Raffaello Giulietti
On Thu, 9 Sep 2021 23:44:49 GMT, Brian Burkhalter wrote: >> Raffaello Giulietti has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8271602: Add Math.ceilDiv() family parallel to Math.floorDiv() family > >

Re: RFR: 8271602: Add Math.ceilDiv() family parallel to Math.floorDiv() family [v2]

2021-09-10 Thread Raffaello Giulietti
> This PR ideally continues #5285, which has been closed as a consequence of > inadvertently removing the branch on my repo. See there for initial > discussion. > > Sorry for the mess. Raffaello Giulietti has updated the pull request incrementally with one additional commit since the last

Re: RFR: 8271602: Add Math.ceilDiv() family parallel to Math.floorDiv() family

2021-09-09 Thread Brian Burkhalter
On Wed, 1 Sep 2021 20:13:38 GMT, Raffaello Giulietti wrote: > This PR ideally continues #5285, which has been closed as a consequence of > inadvertently removing the branch on my repo. See there for initial > discussion. > > Sorry for the mess.

RFR: 8271602: Add Math.ceilDiv() family parallel to Math.floorDiv() family

2021-09-01 Thread Raffaello Giulietti
This PR ideally continues #5285, which has been closed as a consequence of inadvertently removing the branch on my repo. See there for initial discussion. Sorry for the mess. - Commit messages: - 8271602: Add Math.ceilDiv() family parallel to Math.floorDiv() family Changes:

Re: RFR: 8271602: Add Math.ceilDiv() family parallel to Math.floorDiv() family

2021-08-31 Thread Brian Burkhalter
On Tue, 31 Aug 2021 07:22:17 GMT, Raffaello Giulietti wrote: >> src/java.base/share/classes/java/lang/Math.java line 1501: >> >>> 1499: // if the signs are the same and modulo not zero, round up >>> 1500: if ((x ^ y) >= 0 && (q * y != x)) { >>> 1501: return q + 1;

Re: RFR: 8271602: Add Math.ceilDiv() family parallel to Math.floorDiv() family

2021-08-31 Thread Raffaello Giulietti
On Mon, 30 Aug 2021 22:25:09 GMT, Brian Burkhalter wrote: >> Please review this PR to add officially endorsed `ceilDiv()` and `ceilMod()` >> methods do `j.l.[Strict]Math`. >> >> Beside adding fresh tests to `test/jdk/java/lang/Math/DivModTests.java`, >> this PR also corrects small typos in it

Re: RFR: 8271602: Add Math.ceilDiv() family parallel to Math.floorDiv() family

2021-08-30 Thread Brian Burkhalter
On Fri, 27 Aug 2021 18:53:23 GMT, Raffaello Giulietti wrote: > Please review this PR to add officially endorsed `ceilDiv()` and `ceilMod()` > methods do `j.l.[Strict]Math`. > > Beside adding fresh tests to `test/jdk/java/lang/Math/DivModTests.java`, this > PR also corrects small typos in it

Re: RFR: 8271602: Add Math.ceilDiv() family parallel to Math.floorDiv() family

2021-08-30 Thread Brian Burkhalter
On Fri, 27 Aug 2021 18:53:23 GMT, Raffaello Giulietti wrote: > Please review this PR to add officially endorsed `ceilDiv()` and `ceilMod()` > methods do `j.l.[Strict]Math`. > > Beside adding fresh tests to `test/jdk/java/lang/Math/DivModTests.java`, this > PR also corrects small typos in it

RFR: 8271602: Add Math.ceilDiv() family parallel to Math.floorDiv() family

2021-08-27 Thread Raffaello Giulietti
Please review this PR to add officially endorsed `ceilDiv()` and `ceilMod()` methods do `j.l.[Strict]Math`. Beside adding fresh tests to `test/jdk/java/lang/Math/DivModTests.java`, this PR also corrects small typos in it and exercises tests that were already present but which were never