> On 2 Oct 2019, at 16:54, Patrick Concannon <patrick.concan...@oracle.com> > wrote: > > Hi, > > I've updated the webrev following feedback received here, and from the CSR on > this issue. > > webrev: http://cr.openjdk.java.net/~pconcannon/8222829/webrevs/webrev.02/ > Mainly looks good. Trivially, there are superfluous parentheses around the exception message string: throw new IllegalArgumentException(("timeout < 0")); No need to re-generate the webrev. -Chris.
- RFR[8222829]: DatagramSocket.setSoTimeout does not speci... Patrick Concannon
- Re: RFR[8222829]: DatagramSocket.setSoTimeout does ... Chris Hegarty
- Re: RFR[8222829]: DatagramSocket.setSoTimeout d... Patrick Concannon
- Re: RFR[8222829]: DatagramSocket.setSoTimeo... Alan Bateman
- Re: RFR[8222829]: DatagramSocket.setSoT... Patrick Concannon
- Re: RFR[8222829]: DatagramSocket.s... Alan Bateman
- Re: RFR[8222829]: DatagramSocket.setSoTimeo... Daniel Fuchs
- Re: RFR[8222829]: DatagramSocket.setSoT... Patrick Concannon
- Re: RFR[8222829]: DatagramSocket.s... Chris Hegarty