RFR: 8072379: Implement jdk.Version and jdk.OracleVersion
Mandy Chung
mandy.chung at oracle.com
Tue Jan 12 22:25:53 UTC 2016
> On Jan 11, 2016, at 1:44 PM, Iris Clark <iris.clark at oracle.com> wrote:
>
> Hi, Joe, Roger, Alan, Magnus, and Mandy.
>
> At the end of December (shortly before the Christmas/Winter
> break and my vacation), I provided responses to your messages
> and an updated webrev:
>
> http://cr.openjdk.java.net/~iris/verona/8072379/webrev.2/
It’s good to see that jdk.OracleVersion has been removed and use jdk.Version to obtain the fourth element of the version number.
This patch looks good in general. Minor comment:
273 current = parse(System.getProperty("java.version"));
System.getProperty will do a permission check and it needs to be called within a doPrivileged block.
154 * @see <a href="http://openjdk.java.net/jeps/223">JEP 223: New Version-String Scheme</a>
Does the javadoc have the essential information from this JEP? Wonder if this @see is necessary.
Alan already comment on the “jdk” package that needs to find the proper module to export it (that’s a future RFE) and modules.xml should be updated.
Mandy
More information about the core-libs-dev
mailing list