On Thu, 23 Jan 2025 02:50:46 GMT, Justin Lu <j...@openjdk.org> wrote:
>> Please review this PR and CSR which adds a method to _java.util.Currency_ >> which returns a stream of the available Currencies. >> >> The motivation can be found in the CSR. > > 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 eight additional commits since > the last revision: > > - make 'available' un-volatile > - Merge branch 'master' into 8347949-Currency-Stream > - reflect feedback > - provide message for test assertion > - reflect Roger's review > - init hashset w/ accurate size via HashSet.newHashSet(int) > - synchronize init method over block > - init We could add @Stable to `available` for now, but forseeing the stable values, not that significant. So fine by me as in its current form ------------- Marked as reviewed by naoto (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/23165#pullrequestreview-2578935131
