On 27/01/2017 12:52, Ramanand Patil wrote:

Thank you Alan and Kumar for your review.
Here is the updated webrev: http://cr.openjdk.java.net/~rpatil/8167063/webrev.03

Changes done:
1. LinkageError in method loadMainClass is handled separately and with a new 
message in launcher.properties
2. Test case updated to print the error message(even when test passes) and to 
print the final test result message as suggested by Kumar.

The updated LauncherHelper looks okay although you can drop the catching of LinkageError at L641 because it will be caught by the outer catch at L647.

-Alan.

Reply via email to