JDK 9 RFR of 8172547: (se) Selector.select(Long.MAX_VALUE) fires repeatedly

[Langer, Christoph]

Yep, +1.

> -----Original Message-----
> From: nio-dev [mailto:nio-dev-bounces at openjdk.java.net] On Behalf Of Roger
> Riggs
> Sent: Dienstag, 17. Januar 2017 21:17
> To: nio-dev at openjdk.java.net
> Subject: Re: JDK 9 RFR of 8172547: (se) Selector.select(Long.MAX_VALUE) fires
> repeatedly
> 
> Looks good Brian.
> 
> +1
> 
> Roger
> 
> On 1/17/2017 3:11 PM, Brian Burkhalter wrote:
> >
> > On Jan 16, 2017, at 12:12 PM, Alan Bateman <Alan.Bateman at oracle.com
> > <mailto:Alan.Bateman at oracle.com>> wrote:
> >
> >> On 13/01/2017 22:57, Brian Burkhalter wrote:
> >>> Hi Roger,
> >>>
> >>> Thanks for the comments. Here is an update which takes them into
> >>> account:
> >>>
> >>> http://cr.openjdk.java.net/~bpb/8172547/webrev.01/
> >>> <http://cr.openjdk.java.net/%7Ebpb/8172547/webrev.01/>
> >>>
> >>>
> >> Aside from the typo in the comment that Christophpointed then then
> >> this looks good to me.
> >
> > Here is an updated patch with the typo in the implementation fixed and
> > line 77 of the test changed to t.join(SLEEP_MILLIS).
> >
> > http://cr.openjdk.java.net/~bpb/8172547/webrev.02/
> > <http://cr.openjdk.java.net/%7Ebpb/8172547/webrev.02/>
> >
> > Thanks,
> >
> > Brian