> 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.

Justin Lu has updated the pull request with a new target base due to a merge or 
a rebase. The incremental webrev excludes the unrelated changes brought in by 
the merge/rebase. The pull request contains five additional commits since the 
last revision:

 - Merge branch 'master' into 8321480-ISO4217-176
 - add comment
 - reflect review comments
 - add xcg to CurrencyNames
 - init

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17023/files
  - new: https://git.openjdk.org/jdk/pull/17023/files/249fbc7a..94f5870c

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17023&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17023&range=00-01

  Stats: 21626 lines in 351 files changed: 17767 ins; 3052 del; 807 mod
  Patch: https://git.openjdk.org/jdk/pull/17023.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17023/head:pull/17023

PR: https://git.openjdk.org/jdk/pull/17023

Reply via email to