> On 20 Oct 2016, at 13:47, Pavel Rappo wrote:
>
> Hello,
>
> Could you please review the following change for [1]?
>
> http://cr.openjdk.java.net/~prappo/8168405/webrev/
Thank you Pavel, this looks good.
-Chris.
> This change addresses some code paths in the native networking code for
>
Hi Pavel,
> > On 20 Oct 2016, at 14:03, Langer, Christoph
> wrote:
> >
> > Hi Pavel,
> >
> > overall this looks good. I've got a few minor remarks:
> >
> > 1. What about using the macro CHECK_NULL_RETURN in
> NetworkInterface_winXP.c?
> >
> > 2. in Java_java_net_TwoStacksPlainDatagramSocketImpl_p
> On 20 Oct 2016, at 14:03, Langer, Christoph wrote:
>
> Hi Pavel,
>
> overall this looks good. I've got a few minor remarks:
>
> 1. What about using the macro CHECK_NULL_RETURN in NetworkInterface_winXP.c?
>
> 2. in Java_java_net_TwoStacksPlainDatagramSocketImpl_peekData:
>
> You could move
egards
Christoph
> -Original Message-
> From: Pavel Rappo [mailto:pavel.ra...@oracle.com]
> Sent: Donnerstag, 20. Oktober 2016 14:47
> To: OpenJDK Network Dev list
> Subject: RFR 8168405: Pending exceptions in java.base/windows/native
>
> Hello,
>
> Could yo
Hello,
Could you please review the following change for [1]?
http://cr.openjdk.java.net/~prappo/8168405/webrev/
This change addresses some code paths in the native networking code for Windows
operating system where thrown exceptions might be left unnoticed.
Thanks,
-Pavel
--
