RFR 8004547: Extend JavaFX launcher support...
david.dehaven at oracle.com
Fri Dec 21 17:09:15 UTC 2012
> It looks great here are few items I noticed:
> 1. The error defined by:
> + 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.
Error: The JavaFX launchApplication method has the wrong signature, it\n\
must be declared static and return a value of type void
> 2. Nit: extraneous new lines, maybe ?
What lines? I removed a few from FXHelper to tighten it up a bit, and the extra one at the end.
> 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
> -System.out.println("Main-Class: "+mainClassEntry);
> +System.out.println("Main-Class: " + mainClassEntry);
Fixed.. again, not sure how it reverted. Maybe I went too crazy hitting Undo :)
More information about the core-libs-dev