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

[Michael McMahon]

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
>