RFR 8004547: Extend JavaFX launcher support...
David DeHaven
david.dehaven at oracle.com
Fri Dec 21 17:09:15 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.
How about:
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
Fixed.
> FXLauncherTest
>
> Nit:
>
> -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 :)
-DrD-
More information about the core-libs-dev
mailing list