Hi Robin,
It looks like the status argument to BeforeHalt is discarded in
JVM_BeforeHalt
and is not inserted into the event.
That suggests it should be removed all the way back to Shutdown.beforeHalt.
Roger
On 2/13/2018 9:59 AM, Robin Westberg wrote:
Hi Alan,
On 12 Feb 2018, at 09:02, Alan Bateman <alan.bate...@oracle.com> wrote:
On 12/02/2018 07:07, David Holmes wrote:
Okay, I’ve removed the code related to the status field, certainly makes the
change a bit less intrusive.
Updated webrev: http://cr.openjdk.java.net/~rwestberg/8041626/webrev.01/
Incremental: http://cr.openjdk.java.net/~rwestberg/8041626/webrev.00-01/
This looks much cleaner/neater to me - thanks.
The updates to Runtime/Shutdown seems okay.
Thanks for reviewing!
Best regards,
Robin
-Alan
