Suggestion of combining some macros of processing solaris, macosx with other UNIX

[Frank Ding]

Hi Chris and Jonathan,
   Thank you all.  The change set is OK.

Best regards,
Frank

On 8/17/2012 5:20 PM, Jonathan Lu wrote:
> On 08/17/2012 04:14 PM, Chris Hegarty wrote:
>> On 16/08/12 10:21, Frank Ding wrote:
>>> ....
>>> Hi Chris and Alan,
>>>    Thank you for taking time to help this issue.  I have built using
>>> latest openjdk 8 repo on Windows 64 and Linux 32/64.  Since it's a 
>>> macro
>>> change in path "src/solaris", I only did jtreg tests for Linux 32 
>>> and 64
>>> build.  The jtreg tests I ran are restricted to package "java/net".
>>> Please let me know if you need me to do more tests or on more platforms
>>> (such as Solaris).
>>
>> I ran some builds and tests on all ( Solaris, Linux & Mac ) 
>> platforms. All looks good.
>>
>> You can list me as a reviewer. I can push this for you, or can have 
>> someone else from IBM do the push, just let me know.
>>
>> Thanks for the contribution,
>> -Chris.
>>
>>>
>>> Best regards,
>>> Frank
>>>
>>
> Hello Chris,
>
> Thanks for review, I've pushed the change @ 
> http://hg.openjdk.java.net/jdk8/tl/jdk/rev/4993f8aa7f2e
>
> changeset:   5704:4993f8aa7f2e
> tag:         tip
> user:        dingxmin
> date:        Fri Aug 17 17:10:56 2012 +0800
> files:       src/solaris/native/java/net/PlainDatagramSocketImpl.c
> description:
> 7191275: Cleanup OS specific blocks in PlainDatagramSocketImpl.c to 
> support more unix-like platforms
> Reviewed-by: chegar
>
> And to Frank, pls verify the change set.
>
> Thanks
> Jonathan