Request for approval: fix 4428022, System.out.println(0.001) outputs 0.0010

Andrew Haley aph at redhat.com
Thu May 28 19:29:21 UTC 2009


Joseph D. Darcy wrote:
> Andrew Haley wrote:
>> Joseph D. Darcy wrote:
>>  
>>> Alan Bateman wrote:
>>>    
>>>> Andrew Haley wrote:
>>>>      
>>>>> :
>>>>> OK, here it is.  I put the test in java/lang/Double/ToString.java
>>>>> rather than sun/misc/FloatingDecimal because the bug manifested
>>>>> itself in Double.toString although the fix was in
>>>>> FloatingDecimal.dtoa.
>>>>>           
>>>> I agree that test/java/lang/Double is the best place for this. All
>>>> looks good to me (except minor nit, "String argv[]" -> "String[]
>>>> args").  Joe, are you okay with this?
>>>>
>>>> -Alan.
>>>>       
>>> Yes; the changes look fine for both releases.
>>
>> To which openjdk7 repo do you want me to push this?
>> My best guess is jdk7/jdk7-gate/jdk.
> 
> Please push this through TL (tools and libraries):
> ssh://hg.openjdk.java.net/jdk7/tl-gate/jdk

Ta.  A request:

I think that patch approvals had better start to include the repo.
External contributors need this information, even if it's obvious
from inside Sun.

Thanks,
Andrew.




More information about the core-libs-dev mailing list