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