Code Review for jdk

Prepared by:aph on Wed Nov 3 13:26:45 GMT 2010
Workspace:/local/openjdk/jdk6/jdk
Compare against: http://hg.openjdk.java.net/jdk6/jdk6//jdk
Compare against version:396
Summary of changes: 2079 lines changed: 976 ins; 472 del; 631 mod; 8351 unchg
Patch of changes: jdk.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/classes/java/math/BigDecimal.java

rev 397 : 6622432: RFE: Performance improvements to java.math.BigDecimal
Reviewed-by: darcy
rev 398 : 6850606: Regression from JDK 1.6.0_12
Summary: The returned result from multiply should be constructed by using valueOf to take care of the INFLATED case.
Reviewed-by: darcy
rev 399 : 6876282: BigDecimal's divide(BigDecimal bd, RoundingFormat r) produces incorrect result
Reviewed-by: darcy
1152 lines changed: 543 ins; 240 del; 369 mod; 2948 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/classes/java/math/BigInteger.java

rev 397 : 6622432: RFE: Performance improvements to java.math.BigDecimal
Reviewed-by: darcy
432 lines changed: 118 ins; 152 del; 162 mod; 2888 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/classes/java/math/BitSieve.java

rev 397 : 6622432: RFE: Performance improvements to java.math.BigDecimal
Reviewed-by: darcy
4 lines changed: 0 ins; 2 del; 2 mod; 210 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/classes/java/math/MathContext.java

rev 397 : 6622432: RFE: Performance improvements to java.math.BigDecimal
Reviewed-by: darcy
27 lines changed: 0 ins; 27 del; 0 mod; 326 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/classes/java/math/MutableBigInteger.java

rev 397 : 6622432: RFE: Performance improvements to java.math.BigDecimal
Reviewed-by: darcy
300 lines changed: 155 ins; 49 del; 96 mod; 1226 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/classes/java/math/SignedMutableBigInteger.java

rev 397 : 6622432: RFE: Performance improvements to java.math.BigDecimal
Reviewed-by: darcy
3 lines changed: 0 ins; 2 del; 1 mod; 134 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/math/BigDecimal/AddTests.java

rev 397 : 6622432: RFE: Performance improvements to java.math.BigDecimal
Reviewed-by: darcy
25 lines changed: 25 ins; 0 del; 0 mod; 280 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/java/math/BigDecimal/DivideTests.java

rev 397 : 6622432: RFE: Performance improvements to java.math.BigDecimal
Reviewed-by: darcy
rev 399 : 6876282: BigDecimal's divide(BigDecimal bd, RoundingFormat r) produces incorrect result
Reviewed-by: darcy
38 lines changed: 37 ins; 0 del; 1 mod; 339 unchg

------ ------ ------ ------ --- New Patch Raw test/java/math/BigDecimal/MultiplyTests.java

rev 398 : 6850606: Regression from JDK 1.6.0_12
Summary: The returned result from multiply should be constructed by using valueOf to take care of the INFLATED case.
Reviewed-by: darcy
98 lines changed: 98 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /home/aph/bin/webrev (vers 23.18-hg).