RFR[11] of JDK-8207833, java/nio/channels/Selector/SelectWithConsumer.java testCancel() fails intermittently
Alan Bateman
Alan.Bateman at oracle.com
Thu Jul 19 11:33:53 UTC 2018
On 19/07/2018 10:07, Hamlin Li wrote:
> Would you please review the following patch?
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8207833
>
> webrev: http://cr.openjdk.java.net/~mli/8207833/webrev.00/
Can you swap key1 and key2 so that key1 is created first? It may confuse
further maintainers to create key2 first.
Also can be group the creation of key1 with select, and then key2 with
selectNow. The changes mean there's a blank line after creating key1 and
it's no longer obvious that it needs the selectNow to register the key.
-Alan
More information about the nio-dev
mailing list