8202252: (aio) Closed AsynchronousSocketChannel keeps completion handler alive

[Brian Burkhalter]

On Jul 2, 2018, at 10:41 AM, Alan Bateman <Alan.Bateman at oracle.com> wrote:

>> The ugly thing about this version, if it matters, is that when run against the un-patched source the test just hangs instead of failing outright. I can look into the ReferenceQueue if need be.
>> 
> This looks much better. Are you going to move the System.gc() into the loop in waitForRefToClear?

I changed it to use a ReferenceQueue per Roger’s suggestion:

http://cr.openjdk.java.net/~bpb/8202252/webrev.03/

> Also should the @require to exclude Windows be dropped?

This problem does not exist on Windows so I was thinking it might be better not to add another test to the Windows run. I can drop the @requires however if you think it better.

Thanks,

Brian
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20180702/cb2db36f/attachment.html>