Sorry about that. Obviously I still need to learn some things about the process, e.g., review the prescribed workflow.
Thanks, Brian On Oct 16, 2013, at 7:48 PM, David Holmes wrote: > Brian, > > On 17/10/2013 9:21 AM, Brian Burkhalter wrote: >> Dmitry, >> >> I think you are correct: that slipped by both me and the reviewers. I have >> reopened the issue and posted an amendment to the original webrev here: > > You can not reopen a bug once it has been fixed! You need to create a new bug > for this issue and re-close the original ensuring that you restore all the > correct field values ie "Fixed in build". > > David > >> http://cr.openjdk.java.net/~bpb/8010371/webrev.4-amendment/ >> >> Thanks for catching this. >> >> Brian >> >> On Oct 16, 2013, at 3:06 PM, Dmitry Samersoff wrote: >> >>> You have to check error != 0 before call to WSAGetLastError() at >>> ll. 134 Inet6AddressImpl.c >>> >>> Besides that - the fix looks good for me. >>>> >>>> Here's the patch updated for this option: >>>> >>>> http://cr.openjdk.java.net/~bpb/8010371/webrev.4/ >>
