8198753: (dc) DatagramChannel throws unspecified exceptions

[Alan Bateman]

Looks good.

On 22/03/2018 17:19, Brian Burkhalter wrote:
> On Mar 21, 2018, at 2:06 PM, Alan Bateman <Alan.Bateman at oracle.com 
> <mailto:Alan.Bateman at oracle.com>> wrote:
>
>> On 21/03/2018 19:21, Brian Burkhalter wrote:
>>> Updated tests to use expectedExceptions = ….
>>>
>>> http://cr.openjdk.java.net/~bpb/8198753/webrev.03/ 
>>> <http://cr.openjdk.java.net/%7Ebpb/8198753/webrev.03/><http://cr.openjdk.java.net/%7Ebpb/8198753/webrev.03/>
>>>
>> This is much neater, thumbs up from me.
>
> I caught a failure in Connect.java due to the IAE to ACE change. 
> Webrev.03 is updated in place; the diff versus the previous version of 
> the webrev is included below.
>
> Brian
>
> 22 10:14:22.000000000 -0700
> > +++ 
> new/test/jdk/java/nio/channels/DatagramChannel/Connect.java2018-03-22 
> 10:14:22.000000000 -0700
> > @@ -1,5 +1,5 @@
> > /*
> > - * Copyright (c) 2001, 2013, Oracle and/or its affiliates. All 
> rights reserved.
> > + * Copyright (c) 2001, 2018, Oracle and/or its affiliates. All 
> rights reserved.
> >   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
> >   *
> >   * This code is free software; you can redistribute it and/or modify it
> > @@ -102,7 +102,7 @@
> >                 try {
> >                     dc.send(bb, bogus);
> >                     throw new RuntimeException("Allowed bogus send 
> while connected");
> > -                } catch (IllegalArgumentException iae) {
> > +                } catch (AlreadyConnectedException ace) {
> >                     // Correct behavior
> >                 }
>

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