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.


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,


Reply via email to