RFR 8004547: Extend JavaFX launcher support...

[Mandy Chung]

On 12/21/2012 5:52 PM, Kevin Rushforth wrote:
>
> David DeHaven wrote:
>>
>>> This explains what caused the confusion - I didn't expect that "java -jar fxapp.jar" will be passed to fxlauncher with the same launch mode as "java -cp fxapp.jar SomeFXAppClass" (i.e. LM_CLASS).   I think the semantics there was not obvious why it is not LM_JAR mode but JAVAFX_LAUNCH_MODE_xxx has a different semantics than LM_JAR and LM_CLASS that are defined in java.c and also LauncherHelper.   Is there any reason why "java -cp fxapp.jar" can't be passed to fxlauncher with LM_JAR mode?  That'll be consistent how launch mode is used in java.c and the non-FX app.  I'll leave it for you and Kumar/Kevin to decide if this is important to address.
>>>      
>>
>>
>> I *think* what you're asking is why can't we always use "LM_JAR" if java was launched with "java -jar fxapp.jar", correct? Even if there are no JavaFX-* attributes in the manifest.

To be specific - when the fx app is launched by "java -jar fxapp.jar", 
it's a JAR file and shouldn't it be passed to fxlauncher as in JAR file 
mode?   I had assumed that the fxlauncher may need to read the manifest 
if it's launched as a JAR file regardless of the presence of 
JavaFX-Application-Class attribute (but maybe not with the understanding 
I have in this discussion).

Reading your reply and Kevin's reply, the launchMode parameter of the 
launchApplication method (i.e. fxlauncher) specifies whether fxapp.jar 
has a JavaFX-Application-Class attribute or a FX app entry point (that 
could be a JAR file or specified in the classpath) and also indicate the 
name parameter is a class name or a JAR file name - that is not what 
LM_JAR and LM_CLASS  mean in java.c and LauncherHelper.   Strictly 
speaking the parameter is not launch mode to me.

I don't object to what you have.  Aside from this issue, your change 
looks really good.  Additional comments for the launchApplication method 
would really help.  Thanks for the explanation.

Happy Holidays and Happy New Year.
Mandy