Hi Alex,
Overall looks good to me. I'll be real happy if I never see
"WaitForEvent Failed!" again in our CI runs. Just one nit:
402 // see JDK-8204994: sometimes WaitForEvent fails with E_ACCESSDENIED,
403 // but succeeded on 2nd call.
I think "succeeds" would be better than "succeeded".
thanks,
Chris
On 5/27/20 1:54 PM, Alex Menkov wrote:
Hi all,
please review the fix for
https://bugs.openjdk.java.net/browse/JDK-8204994
webrev:
http://cr.openjdk.java.net/~amenkov/jdk15/WinDbg_WaitForEvent/webrev/
--alex
