On 8/14/2012 10:42 PM, Chris Hegarty wrote:
On 14/08/12 13:11, Alan Bateman wrote:
On 14/08/2012 04:11, Frank Ding wrote:
On 8/7/2012 1:46 PM, Frank Ding wrote:
:
Could anybody take a look at my patch below and make comment?
http://cr.openjdk.java.net/~youdwei/ojdk-533/webrev.00/
Thanks & Best regards,
Frank
Hi all,
Is there anybody who is interested in the patch and who can take a
look and comment?
It looks okay to me but I don't have time at the moment to sponsor it.
Can you confirm that you've run the tests with this change?
I filed 7191275: "Cleanup OS specific blocks in
PlainDatagramSocketImpl.c to support more unix-like platforms",for
this issue.
I can sponsor this patch and help get it in. Can answer Alan's
question about testing? And confirm that it builds on all platforms?
Thanks,
-Chris.
-Alan
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).
Best regards,
Frank