<i18n dev> RFR: 8321480: ISO 4217 Amendment 176 Update

Naoto Sato naoto at openjdk.org
Thu Dec 7 22:31:16 UTC 2023


On Thu, 7 Dec 2023 19:43:14 GMT, Justin Lu <jlu at openjdk.org> wrote:

> Please review this PR which incorporates the ISO 4217 Amendment 176 Update. As the replacement of `ANG` to `XCG` won't occur until 2025, this change does not need to go into JDK22. `HR` was also updated to remove the past cutover dates.
> 
> An existing test in _ValidateISO4217.java_ checked that `Currency::getAvailableCurrencies` had all the expected currencies. This method returns all currencies, including ones to take place in the future (e.g. `XCG`). The expected currencies `Set` the method was test against had to be updated to also include future currencies as well.
> 
> Additionally, this change also converted a parameterized test to a normal JUnit test, due to output overflow.

src/java.base/share/classes/sun/util/resources/CurrencyNames.properties line 497:

> 495: xbd=European Unit of Account (XBD)
> 496: xcd=East Caribbean Dollar
> 497: xcg=Caribbean Guilder

I think `XCG=XCG` is also needed for not throwing `MissingResourceException` for `getSymbol()`

test/jdk/java/util/Currency/ValidateISO4217.java line 181:

> 179:                     // without updating ISO4217Codes
> 180:                     String futureCurr = tokens.nextToken();
> 181:                     testCurrencies.add(Currency.getInstance(futureCurr));

I'd not add the future currency, and fix it in the code not to add future currency in available currencies.

test/jdk/java/util/Currency/ValidateISO4217.java line 289:

> 287:                 assertNull(Currency.getInstance(Locale.of("", country)),
> 288:                         "Error: Currency.getInstance() for this locale should return null: " + country);
> 289:             }

What is this change for?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/17023#discussion_r1419721244
PR Review Comment: https://git.openjdk.org/jdk/pull/17023#discussion_r1419722063
PR Review Comment: https://git.openjdk.org/jdk/pull/17023#discussion_r1419722636


More information about the i18n-dev mailing list