On 23/01/15 14:05, Alan Bateman wrote:
On 23/01/2015 13:51, Chris Hegarty wrote:
This is a fix for a minor oversight when refactoring the initializing
of Inet[4|6]Address IDs, used in many places in the networking native
code.

It would appear that there is an initInetAddressIDs() call missing
from the Windows Dual Stacks Socket implementation,
Java_java_net_DualStackPlainSocketImpl_initIDs.  An existing test has
been amended to cover this.

http://cr.openjdk.java.net/~chegar/8071424/webrev.00/webrev/

-Chris.
This looks okay to me. So has this always been missing or did it get
deleted by something else?

It has been missing from the original refactoring :-( just unnoticed as other code typically cause the initialization. Good news is that we now have better test coverage to catch this.

-Chris.

Reply via email to