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.