stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

7021568: Double.parseDouble() returns architecture dependent results

Browse Source 
Reviewed-by: alanb
This commit is contained in:
Joe Darcy 2011-04-17 22:52:26 -07:00
parent b99716e22a
commit 04d347dde1
3 changed files with 30 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
jdk/src/share/classes/sun/misc/FloatingDecimal.java
View File

@ -30,7 +30,7 @@ import sun.misc.DoubleConsts;
import sun.misc.FloatConsts;
import java.util.regex.*;
public class FloatingDecimal{
public strictfp class FloatingDecimal{
boolean isExceptional;
boolean isNegative;
int decExponent;

4
jdk/src/share/classes/sun/misc/FormattedFloatingDecimal.java
View File

@ -1,5 +1,5 @@
/*
* Copyright (c) 2003, 2004, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2003, 2011, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@ -30,7 +30,7 @@ import sun.misc.DoubleConsts;
import sun.misc.FloatConsts;
import java.util.regex.*;
public class FormattedFloatingDecimal{
public strictfp class FormattedFloatingDecimal{
boolean isExceptional;
boolean isNegative;
int decExponent; // value set at construction, then immutable

28
jdk/test/java/lang/Double/ParseDouble.java
View File

@ -23,7 +23,7 @@
/*
* @test
* @bug 4160406 4705734 4707389 4826774 4895911 4421494
* @bug 4160406 4705734 4707389 4826774 4895911 4421494 7021568
* @summary Test for Double.parseDouble method and acceptance regex
*/
@ -581,6 +581,31 @@ public class ParseDouble {
}
}
private static void testStrictness() {
final double expected = 0x0.0000008000001p-1022;
boolean failed = false;
double conversion = 0.0;
double sum = 0.0; // Prevent conversion from being optimized away
//2^-1047 + 2^-1075
String decimal = "6.631236871469758276785396630275967243399099947355303144249971758736286630139265439618068200788048744105960420552601852889715006376325666595539603330361800519107591783233358492337208057849499360899425128640718856616503093444922854759159988160304439909868291973931426625698663157749836252274523485312442358651207051292453083278116143932569727918709786004497872322193856150225415211997283078496319412124640111777216148110752815101775295719811974338451936095907419622417538473679495148632480391435931767981122396703443803335529756003353209830071832230689201383015598792184172909927924176339315507402234836120730914783168400715462440053817592702766213559042115986763819482654128770595766806872783349146967171293949598850675682115696218943412532098591327667236328125E-316";
for(int i = 0; i <= 12_000; i++) {
conversion = Double.parseDouble(decimal);
sum += conversion;
if (conversion != expected) {
failed = true;
System.out.printf("Iteration %d converts as %a%n",
i, conversion);
}
}
System.out.println("Sum = " + sum);
if (failed)
throw new RuntimeException("Inconsistent conversion");
}
public static void main(String[] args) throws Exception {
rudimentaryTest();
@ -595,5 +620,6 @@ public class ParseDouble {
testRegex(paddedBadStrings, true);
testSubnormalPowers();
testStrictness();
}
}