Re: RFR: 8155939: sun.launcher.LauncherHelper INSTANCE is unused

[Claes Redestad]

Thanks Roger!

/Claes

On 2016-05-03 16:56, Roger Riggs wrote:

Looks fine, Roger


On 5/3/2016 10:46 AM, Claes Redestad wrote:

Hi,

sun.launcher.LauncherHelper is currently a singleton enum class, but the INSTANCE is never used. With encapsulation in place we shouldn't worry about cleaning this up.

jdeps data show no usage of this helper class outside of the JDK.

Bug: https://bugs.openjdk.java.net/browse/JDK-8155939

Patch:

diff -r 930d3aef37ee src/java.base/share/classes/sun/launcher/LauncherHelper.java --- a/src/java.base/share/classes/sun/launcher/LauncherHelper.java Tue May 03 15:50:54 2016 +0200 +++ b/src/java.base/share/classes/sun/launcher/LauncherHelper.java Tue May 03 16:23:26 2016 +0200

@@ -84,8 +84,10 @@
 import jdk.internal.misc.VM;


-public enum LauncherHelper {
-    INSTANCE;
+public final class LauncherHelper {
+
+    // No instantiation
+    private LauncherHelper() {}

     // used to identify JavaFX applications
     private static final String JAVAFX_APPLICATION_MARKER =

Thanks!

/Claes