Code Review Request: 7127771: (macosx)test/java/net/Socket/TrafficClass.java fails on Mac OS X

[Kurchi Hazra]

I updated the comment: http://cr.openjdk.java.net/~khazra/7127771/webrev.02/

- Kurchi



On 1/16/2012 2:43 AM, Michael McMahon wrote:
> Yes, looks fine to me too. I would just update the comment above this
> code to add Mac OS to the Solaris case.
>
> Thanks
> Michael
>
> On 13/01/12 21:02, Kurchi Hazra wrote:
>> How does this look: 
>> http://cr.openjdk.java.net/~khazra/7127771/webrev.01/
>>
>> - Kurchi
>>
>>
>>
>> On 1/13/2012 12:14 PM, Alan Bateman wrote:
>>>
>>>>> Bug :  http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7127771
>>>>> Webrev : http://cr.openjdk.java.net/~khazra/7127771/webrev.00/
>>> What you have is fine although you could combine with the Solaris 
>>> code? Should the __ALLBSD_SOURCE XXX be removed while you are there?
>>>
>>> -Alan
>>
>

-- 
-Kurchi