Thank Neil for committed this patch. And also thank Sean and Masayoshi for your review.

On 11/01/2012 01:00 AM, Neil Richards wrote:
Both changesets now pushed to jdk7u-dev/jdk [1] [2].

Regards,
Neil

[1] http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/87abc0c4804d
[2] http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/5341e088e9a3

On Wed, 2012-10-31 at 11:00 +0000, Seán Coffey wrote:
Good point Masayoshi.

Deven, to keep in sync with jdk8 you might consider importing the
7172177 changeset also:
http://hg.openjdk.java.net/jdk8/tl/jdk/rev/60033ab79213
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7172177

Consider that approved for jdk7u-dev push.

regards,
Sean.


On 31/10/12 10:02, Masayoshi Okutsu wrote:
Hi Deven,

The fix itself looks good, but I believe the test was removed in JDK 8
later.

Thanks,
Masayoshi

On 10/31/2012 4:54 PM, Deven You wrote:
Hi All,

This is  a request for approval to push the fix of 7094176 from jdk8
to jdk7u, could anyone take a look?

Link to the bug:
http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7094176

Changeset in jdk8:
http://hg.openjdk.java.net/jdk8/tl/jdk/rev/71cf74329a9e

Webrev for jdk7u:
http://cr.openjdk.java.net/~youdwei/ojdk-557/webrev/
<http://cr.openjdk.java.net/%7Eyoudwei/ojdk-557/webrev/>

The webrev for jdk7u is the same as of jdk8.

Review thread for jdk8 starts at:

http://mail.openjdk.java.net/pipermail/i18n-dev/2012-May/000613.html

Thanks a lot!



Reply via email to