[7u4-osx] Request for approval for 7127660 (macosx) *Socket Async close not working
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.
- Michael
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 NET_RecvFrom(), 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/view_bug.do?bug_id=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