Hi Chris,

On 05/28/2020 15:22, Chris Plummer wrote:
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:

I hope so.


402   // see JDK-8204994: sometimes WaitForEvent fails with E_ACCESSDENIED,
403   // but succeeded on 2nd call.

I think "succeeds" would be better than "succeeded".

fixed locally.

--alex


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


Reply via email to