[7u4-osx] Request for approval for 7127660 (macosx) *Socket Async close not working (original) (raw)

Michael McMahon michael.x.mcmahon at oracle.com
Mon Jan 9 14:13:26 PST 2012


This is actually the same code that's always been there on Linux. It's an interesting observation though because on the face of it, it is dead code that shouldn't be. The reason it has always worked is that the value passed in is never different to the value returned. And we don't even check the returned value anywhere. Socket addresses always have the same size at runtime. I think I will remove the dead code at least, and add a comment.

On 09/01/12 21:56, Kelly O'Hair wrote:

Don't think so.

I assume this socklen is to get around the different types needed. -kto On Jan 9, 2012, at 1:42 PM, Paul Hohensee wrote:

In NETRecvFrom(), isn't the last line

*fromlen = socklen; dead code that really shouldn't be dead? Paul On 1/9/12 4:27 PM, Michael McMahon wrote: This is a request to push the following fix to jdk7u-osx:

CR: http://bugs.sun.com/bugdatabase/viewbug.do?bugid=7127660 Webrev: http://cr.openjdk.java.net/~michaelm/7127660/webrev.1/ Reviewed by: alanb Thanks, Michael.



More information about the macosx-port-dev mailing list