RFR 8004547: Extend JavaFX launcher support...

Kumar Srinivasan kumar.x.srinivasan at oracle.com
Fri Dec 21 16:42:28 UTC 2012


David,

It looks great here are few items I noticed:
1. The error defined by:

+java.launcher.javafx.error1=\
+    Error: The JavaFX runtime is incompatible with the Java launcher

  is used for a signature mismatch, I suggest changing the
  message to reflect the real reason.

2. Nit:  extraneous new lines, maybe ?

3. typo:

525     // Check the existance and signature of main and abort if it's incorrect
525     // Check the existence and signature of main and abort if incorrect

FXLauncherTest

Nit:

-System.out.println("Main-Class: "+mainClassEntry);
+System.out.println("Main-Class: " + mainClassEntry);



Kumar

> Request for review for extending the launcher support to allow the JavaFX runtime to fully support all of it's launch features, including preloaders, classpath, etc..
>
> Webrev:
> http://cr.openjdk.java.net/~ddehaven/8004547/webrev.1/
>
> Corresponding JavaFX JIRA issue that these changes depend on:
> http://javafx-jira.kenai.com/browse/RT-26751
>
>
> This should be the final step in adding launcher support for JavaFX applications. These changes should allow any future changes to be done entirely in the JavaFX runtime. The changes for RT-26751 are in this weeks promotion of JavaFX, so should be available in next weeks JRE build (I think.. I'm never sure about promotion timing..).
>
> -DrD-
>




More information about the core-libs-dev mailing list