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
