March 2016 - core-libs-dev

RFR 9: 8152005: sun/misc/SunMiscSignalTest.java failed intermittently
by Roger Riggs 30 Mar '16

30 Mar '16

Please review this improvement to an intermittently failing test of sun.misc.Signal. The timeout value is increased in case of a busy system and for additional debug value the raising of the signal is retried. Webrev: http://cr.openjdk.java.net/~rriggs/webrev-signal-8152005/ Issue: https://bugs.openjdk.java.net/browse/JDK-8152005 Thanks, Roger

2 3

review of changes to the @Deprecated annotation
by Stuart Marks 29 Mar '16

29 Mar '16

FYI for denizens of core-libs-dev: I've posted a set of proposed changes to the @Deprecated annotation to jdk9-dev. Please see the following review thread: http://mail.openjdk.java.net/pipermail/jdk9-dev/2016-March/003917.html Thanks, s'makrs

1 0

Review request JDK-8153027: Exclude tools/jimage/JImageTest.java
by Mandy Chung 29 Mar '16

29 Mar '16

This test should be excluded until JDK-8150975 is resolved. It’s currently commented out in ProblemList which was unintentional. diff --git a/test/ProblemList.txt b/test/ProblemList.txt --- a/test/ProblemList.txt +++ b/test/ProblemList.txt @@ -443,7 +443,7 @@ # core_tools # 8150975 -# tools/jimage/JImageTest.java linux-i586,windows-i586 +tools/jimage/JImageTest.java linux-i586,windows-i586

2 1

Please review 8148491: Revisit jlink --genbom
by Sundararajan Athijegannathan 29 Mar '16

29 Mar '16

Hi, Please review http://cr.openjdk.java.net/~sundar/8148491/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8148491 Thanks, -Sundar

3 2

RFR: 8152951: Avoid calculating the reverse of StringConcatFactory$Recipe elements
by Claes Redestad 29 Mar '16

29 Mar '16

Hi, the reverse of Recipe.elements is calculated eagerly but only used once and only by some strategies. Removing this might also help a bit with footprint if run with caching enabled. Bug: https://bugs.openjdk.java.net/browse/JDK-8152951 Webrev: http://cr.openjdk.java.net/~redestad/8152951/webrev.01/ Thanks! /Claes

3 4

RFR:JDK-8030864:Add an efficient getDateTimeMillis method to java.time
by nadeesh tv 29 Mar '16

29 Mar '16

Hi all, Please review an enhancement for a garbage free epochSecond method. Bug ID: https://bugs.openjdk.java.net/browse/JDK-8030864 <https://bugs.openjdk.java.net/browse/JDK-8030864> webrev: http://cr.openjdk.java.net/~ntv/8030864/webrev.01 <http://cr.openjdk.java.net/%7Entv/8030864/webrev.01> -- Thanks and Regards, Nadeesh TV

3 13

RFR:JDK-8148950 :Enhance ChronoField Javadoc
by nadeesh tv 29 Mar '16

29 Mar '16

Hi all, Please see the doc changes http://cr.openjdk.java.net/~ntv/8148950/webrev.00/ Bug ID https://bugs.openjdk.java.net/browse/JDK-8148950 -- Thanks and Regards, Nadeesh TV

3 2

RFR [9] 8152190: Move sun.misc.JarIndex and InvalidJarIndexException to an internal package
by Chris Hegarty 29 Mar '16

29 Mar '16

sun.misc.JarIndex, and its accompanying InvalidJarIndexException, are not "Critical APIs", as defined by JEP 260, so they should be moved out of sun.misc and placed into a more appropriate package where they can be encapsulated. http://cr.openjdk.java.net/~chegar/8152190/00/ https://bugs.openjdk.java.net/browse/JDK-8152190 Since the Exception type is only ever statically referenced where it is thrown, I’ve moved it to a static nested class of JarIndex, so as to keep the namespace cleaner. -Chris.

2 4

Instruction to test webrev
by Thanh Hong Dai 29 Mar '16

29 Mar '16

Hi, I'm interested in testing some of the webrev, esp. regex webrev, but I don't want to recompile the whole JDK just for this. Is there any way to test only the regex package? Best regards, Hong Dai Thanh

3 4

RFR: 8151876: (tz) Support tzdata2016b
by Ramanand Patil 29 Mar '16

29 Mar '16

Hi all, Please review the latest TZDATA (tzdata2016b) integration to JDK9. Bug: https://bugs.openjdk.java.net/browse/JDK-8151876 Webrev: http://cr.openjdk.java.net/~rpatil/8151876/webrev.00/ All the TimeZone related tests are passed after integration. Please note that, as per the release notes: As a trial of a new system that needs less information to be made up, the new zones use numeric time zone abbreviations like "+04" instead of invented abbreviations like "ASTT". Since this is a trial system, I have ignored this in the current patch which adds 3 new timezones. But I think it's worth discussing this new system along with this bug and get the experts opinion on this. Do you think that JDK should also follow these IANA timezone naming system for zone abbreviations in future ? Will it be convenient to use such numeric time zone abbreviations ? (Also, it looks like the abbreviations similar to "+03/+04" will be used for the zones linked to the rule set with changing letters like: EST/EDT for US, MSK/MSD for RU, etc..). Regards, Ramanand.

2 4