RFR(XS): 8166604: nio: remove unneeded locals variables and correct NPE

Langer, Christoph christoph.langer at sap.com
Fri Sep 23 07:18:47 UTC 2016


Hi,

please review a small cleanup in nio socket and datagram channel that I have spotted.

I would even suggest to completely remove the null check throwing NPE in DatagramChannelImpl.receive(ByteBuffer dst) as an NPE would implicitly be thrown when dst is null and dst.isReadOnly() is called. The current behavior would not change then.

Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8166604.0/
Bug: https://bugs.openjdk.java.net/browse/JDK-8166604

Best regards
Christoph

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


More information about the nio-dev mailing list