Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v10]

2023-11-17 Thread Joe Darcy
> Add discussion of some of the common pitfalls related to decimal <-> binary > conversion. Joe Darcy has updated the pull request incrementally with one additional commit since the last revision: Fix link. - Changes: - all: https://git.openjdk.org/jdk/pull/16566/files - new

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v9]

2023-11-17 Thread Joe Darcy
> Add discussion of some of the common pitfalls related to decimal <-> binary > conversion. Joe Darcy has updated the pull request incrementally with one additional commit since the last revision: Minor tweaks. - Changes: - all: https://git.openjdk.org/jdk/pull/16566/files -

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v8]

2023-11-17 Thread Joe Darcy
On Wed, 8 Nov 2023 18:41:53 GMT, Eirik Bjorsnos wrote: >> Joe Darcy has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Small clean ups. > > src/java.base/share/classes/java/lang/Double.java line 209: > >> 207: * decimal conversion. While

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v8]

2023-11-17 Thread Joe Darcy
> Add discussion of some of the common pitfalls related to decimal <-> binary > conversion. Joe Darcy has updated the pull request incrementally with one additional commit since the last revision: Small clean ups. - Changes: - all: https://git.openjdk.org/jdk/pull/16566/files

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v7]

2023-11-17 Thread Joe Darcy
> Add discussion of some of the common pitfalls related to decimal <-> binary > conversion. Joe Darcy has updated the pull request incrementally with one additional commit since the last revision: Minor rewordings. - Changes: - all: https://git.openjdk.org/jdk/pull/16566/files

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v6]

2023-11-17 Thread Joe Darcy
> Add discussion of some of the common pitfalls related to decimal <-> binary > conversion. Joe Darcy has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains e

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v5]

2023-11-13 Thread Joe Darcy
> Add discussion of some of the common pitfalls related to decimal <-> binary > conversion. Joe Darcy has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains s

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v4]

2023-11-08 Thread Joe Darcy
> Add discussion of some of the common pitfalls related to decimal <-> binary > conversion. Joe Darcy has updated the pull request incrementally with one additional commit since the last revision: Fix typo. - Changes: - all: https://git.openjdk.org/jdk/pull/16566/files - new

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v3]

2023-11-08 Thread Joe Darcy
On Wed, 8 Nov 2023 19:40:11 GMT, Brian Burkhalter wrote: >> Joe Darcy has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Switch to parseFloat from valueOf, add links. > > src/java.base/share/classes/java/lang/Double.java line 752: > >> 750

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v3]

2023-11-08 Thread Brian Burkhalter
On Wed, 8 Nov 2023 19:24:24 GMT, Joe Darcy wrote: >> Add discussion of some of the common pitfalls related to decimal <-> binary >> conversion. > > Joe Darcy has updated the pull request incrementally with one additional > commit since the last revision: > > Switch to parseFloat from valueOf

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v3]

2023-11-08 Thread Joe Darcy
> Add discussion of some of the common pitfalls related to decimal <-> binary > conversion. Joe Darcy has updated the pull request incrementally with one additional commit since the last revision: Switch to parseFloat from valueOf, add links. - Changes: - all: https://git.open

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v3]

2023-11-08 Thread Eirik Bjorsnos
On Wed, 8 Nov 2023 19:24:24 GMT, Joe Darcy wrote: >> Add discussion of some of the common pitfalls related to decimal <-> binary >> conversion. > > Joe Darcy has updated the pull request incrementally with one additional > commit since the last revision: > > Switch to parseFloat from valueOf

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v2]

2023-11-08 Thread Brian Burkhalter
On Wed, 8 Nov 2023 18:41:15 GMT, Joe Darcy wrote: >> Add discussion of some of the common pitfalls related to decimal <-> binary >> conversion. > > Joe Darcy has updated the pull request incrementally with one additional > commit since the last revision: > > Address issues found by reviewers

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v2]

2023-11-08 Thread Raffaello Giulietti
On Wed, 8 Nov 2023 18:41:15 GMT, Joe Darcy wrote: >> Add discussion of some of the common pitfalls related to decimal <-> binary >> conversion. > > Joe Darcy has updated the pull request incrementally with one additional > commit since the last revision: > > Address issues found by reviewers

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v2]

2023-11-08 Thread Joe Darcy
On Wed, 8 Nov 2023 18:17:00 GMT, Raffaello Giulietti wrote: >> Joe Darcy has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Address issues found by reviewers. > > src/java.base/share/classes/java/lang/Double.java line 287: > >> 285: * le

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v2]

2023-11-08 Thread Joe Darcy
On Wed, 8 Nov 2023 18:09:52 GMT, Raffaello Giulietti wrote: >> Joe Darcy has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Address issues found by reviewers. > > src/java.base/share/classes/java/lang/Double.java line 257: > >> 255: * //

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v2]

2023-11-08 Thread Joe Darcy
On Wed, 8 Nov 2023 17:54:50 GMT, Brian Burkhalter wrote: >> Joe Darcy has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Address issues found by reviewers. > > src/java.base/share/classes/java/lang/Double.java line 212: > >> 210: * fracti

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues [v2]

2023-11-08 Thread Joe Darcy
> Add discussion of some of the common pitfalls related to decimal <-> binary > conversion. Joe Darcy has updated the pull request incrementally with one additional commit since the last revision: Address issues found by reviewers. - Changes: - all: https://git.openjdk.org/jdk

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues

2023-11-08 Thread Raffaello Giulietti
On Wed, 8 Nov 2023 17:29:28 GMT, Joe Darcy wrote: > Add discussion of some of the common pitfalls related to decimal <-> binary > conversion. src/java.base/share/classes/java/lang/Double.java line 257: > 255: * // Numeric values listed are exact values > 256: * oneTenthApproxAsFloat = 0.1000

Re: RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues

2023-11-08 Thread Brian Burkhalter
On Wed, 8 Nov 2023 17:29:28 GMT, Joe Darcy wrote: > Add discussion of some of the common pitfalls related to decimal <-> binary > conversion. src/java.base/share/classes/java/lang/Double.java line 212: > 210: * fraction (0.3); but in base 3, 1/3 is exactly > 211: * 0.1(3), that is 1 

RFR: JDK-8295391: Add discussion of binary <-> decimal conversion issues

2023-11-08 Thread Joe Darcy
Add discussion of some of the common pitfalls related to decimal <-> binary conversion. - Commit messages: - Appease jcheck. - JDK-8295391: Add discussion of binary <-> decimal conversion issues Changes: https://git.openjdk.org/jdk/pull/16566/files Webrev: https://webrevs.openjdk