Changes looks good. They match changes in 9 and 8u.

Thanks,
Vladimir

On 1/5/15 8:57 AM, Roland Westrelin wrote:
Hi Vladimir,

Can you review this?

Thanks,
Roland.

Begin forwarded message:

Date: December 22, 2014 at 4:55:50 PM GMT+1
From: Seán Coffey <sean.cof...@oracle.com>
To: Roland Westrelin <roland.westre...@oracle.com>, jdk7u-dev@openjdk.java.net
Subject: Re: [7u-dev] Request for approval for 8048170: Test 
closed/java/text/Normalizer/ConformanceTest.java failed

Roland,

Please get a peer review before pushing the jdk7u changes.

Approved.

regards,
Sean.

On 22/12/14 14:46, Roland Westrelin wrote:
Backport to 7u request. Changes were pushed to jdk9 a bit more than 2 weeks ago 
and to 8u a bit less than 2 weeks ago. Nightly testing is fine. The change 
doesn’t apply cleanly to 7u sources but the conflict is straightforward to 
solve.

https://bugs.openjdk.java.net/browse/JDK-8048170

http://cr.openjdk.java.net/~roland/8048170/webrev.7u/

Review thread:
http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-December/016406.html

jdk9 changeset:
http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/e78fa38ad3a3

jdk8u changeset:
http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/b6585ac86988

Thanks,
Roland.


Reply via email to