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

[Roger Riggs]

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