[jdk8u-dev] Request for Review: 8129533: ISO 4217 amendment 160

Naoto Sato naoto.sato at oracle.com
Mon Aug 29 20:13:10 UTC 2016


Hi Leo,

The change looks good to me. Also I approve to backport this to 
jdk8u-dev. Please note that you need to use the *original* bug id for 
both the review and the changeset comment.

Naoto

On 8/29/16 10:24 AM, Leo Jiang wrote:
> Hi,
>
> Request a review for JDK8 as well as approval.
>
> Webrev: http://cr.openjdk.java.net/~ljiang/8129533/webrev.00/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8129533
>
> This is a backport of the original fix to support ISO 4127 amendment #160.
>
> Original bug: https://bugs.openjdk.java.net/browse/JDK-8129361
>
> I had tried the unshuffle_patch tool but failed as the paths of
> CurrencyData.properties and CurrencyNames.properties are not managed by
> the tool. I had to modify the patch and edit manually to apply to 8u repo.
>
> I had run the builds on all platforms and test with jprt.
>
> Thanks,
> Leo


More information about the jdk8u-dev mailing list