RFR 8004547: Extend JavaFX launcher support... (original) (raw)
Kumar Srinivasan kumar.x.srinivasan at oracle.com
Fri Dec 21 16:42:28 UTC 2012
- Previous message: RFR 8004547: Extend JavaFX launcher support...
- Next message: RFR 8004547: Extend JavaFX launcher support...
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
David,
It looks great here are few items I noticed:
- 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.
Nit: extraneous new lines, maybe ?
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-
- Previous message: RFR 8004547: Extend JavaFX launcher support...
- Next message: RFR 8004547: Extend JavaFX launcher support...
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]