>>>>>>> Cmd line                   FAC             LAUNCH_MODE 
>>>>>>> JAVAFX_LAUNCH_MODE
>>>>>>> java -jar fxapp.jar        Present         LM_JAR       LM_JAR
>>>>>>> java -jar fxapp.jar        Not present     LM_JAR       [LM_JAR]
>>>>>>> java -cp fxapp.jar ...     Not present LM_CLASS            LM_CLASS
>>>>>>> java -cp somedir ...       Not present LM_CLASS            LM_CLASS
>> I didn't see any reliable means of testing the last case without further 
>> changes to TestHelper.java, is it that important? The code path through 
>> LauncherHelper.java should be the same for either -cp case. I can add the 
>> test case if it's important, but that will require changes to 
>> TestHelper.java to allow compiling to a directory without creating a jar 
>> file.
> 
> -cp fxapp.jar should be sufficient for this, this is more sensitive to the 
> launcher itself,
> and is tested thoroughly in Arrrghs.java and elsewhere.

In that case, the only changes I have are the changes you requested earlier. If 
there are no further objections can we call this done?

Latest webrev:
http://cr.openjdk.java.net/~ddehaven/8004547/webrev.3/

-DrD-

Reply via email to