Hi Alex,
Thanks for doing this. Not having output from a spawned process that
failed is an issue with more than just LingeredApp tests. This is a good
start in getting those fixed.
I'm a little unclear on one part of your fix. Why did you move the
"appProcess != null" into finishApp(). You already make that check in
stopApp(). If anything it looks like that check should have been there
before your changes, but is no longer needed after your changes.
thanks,
Chris
On 2/12/20 1:28 PM, Alex Menkov wrote:
Hi all,
Please review small fix for
https://bugs.openjdk.java.net/browse/JDK-8238710
webrev:
http://cr.openjdk.java.net/~amenkov/jdk15/LingeredApp_log_error/webrev/
--alex
