Clean-room implementation of Double::toString(double) and Float::toString(float)
thomas.stuefe at gmail.com
Fri Apr 13 09:31:44 UTC 2018
On Sat, Mar 31, 2018 at 12:17 AM, Brian Burkhalter
<brian.burkhalter at oracle.com> wrote:
> Hi Raffaello,
> On Mar 30, 2018, at 2:57 PM, raffaello.giulietti at gmail.com wrote:
>>>> The new code also has a better specification than the current one, while
>>>> being mostly compatible. Indeed, the current specification leaves room
>>>> for interpretation and thus cannot ensure that an implementation
>>>> produces consistent and unique results from one release to the next. The
>>>> newer spec ensures a unique result.
>>> Any specification change would need to go through the Compatibility and
>>> Specification Review process. 
>> OK, as you will see, as soon as the code will be uploaded, the only
>> thing that formally affects output is the "1.0E23" versus "9.99....E22"
>> issue. Everything else is worded in such a way to remain compatible but
>> is simply a little bit more rigorous.
> Sounds good.
>> My wording was misleading: I already got the confirmation that my OCA
>> application has been accepted, so I'm formally ready to contribute.
> That’s good as it gives more time.
>>> Per the JDK 11 schedule  there could well be sufficient time to run
>>> this submission through the review processes. I suggest, once your OCA
>>> has been processed, to proceed by posting your proposed changes for
>>> review on this mailing list. Note that in general attachments are
>>> scrubbed, so the patch would need either to be included inline or
>>> published as a webrev .
>> OK, I'll take a look on how the mechanics works.
>> I'm usually on Windows. Are there technical issues with it as far as
>> Webrev is concerned? I mean, I could setup a Linux VM in VirtualBox if
>> this simplifies my life, but I'd prefer continuing my main work in Win.
> As seen in Jon’s posting there are some attachment types which will work. As to webrev, I think it should work on Windows at least in cygwin but I’ve not used it there myself. If it’s just a matter of creating a webrev I could do that on your behalf based on your supplied patch.
Just to chime in here, producing webrevs on windows is easy:
- in cygwin, install Korn shell (ksh)
- and, of course, mercurial
the rest is standard:
- clone the code-tools/webrev repo:
- run webrev: ksh <path to webrev.ksh>/webrev.ksh -o <output dir>
Best Regards, Thomas
More information about the core-libs-dev