This looks okay but it's very possible this will need to change again soon once a home for the Version class is found.

-Alan

On 18/02/2016 13:42, Chris Hegarty wrote:
With the introduction of a JDK specific Version API, the console AboutDialog
should no longer use sun.misc.Version. It should be be updated to use the
new JDK specific API.

diff --git a/src/jdk.jconsole/share/classes/sun/tools/jconsole/AboutDialog.java 
b/src/jdk.jconsole/share/classes/sun/tools/jconsole/AboutDialog.java
--- a/src/jdk.jconsole/share/classes/sun/tools/jconsole/AboutDialog.java
+++ b/src/jdk.jconsole/share/classes/sun/tools/jconsole/AboutDialog.java
@@ -34,8 +34,6 @@
  import javax.swing.border.*;
  import javax.swing.event.*;
-import static sun.misc.Version.jdkMinorVersion;
-
  import static java.awt.BorderLayout.*;
  import static sun.tools.jconsole.Utilities.*;
@@ -183,7 +181,7 @@
      }
private static String getOnlineDocUrl() {
- String version = Integer.toString(jdkMinorVersion());
+ String version = Integer.toString(jdk.Version.current().major());
          return 
Resources.format(Messages.HELP_ABOUT_DIALOG_USER_GUIDE_LINK_URL,
                                  version);
      }

-Chris.

https://bugs.openjdk.java.net/browse/JDK-8150168

Reply via email to