Hi Naoto,
Bundle.java:
- 28: Explicit import are preferred, (if the IDE make the change, fix
the settings in the IDE).
- 763, the indentation of the nested getOrDefault calls doesn't look
conventional.
(It might be interesting to have an API that allows one or more
defaults to be tried in turn.
It will call each of the methods so there is no shortcutting if the
value is found early in the list).
CLDRConverter:
- 513, 1169...: Debugging code can be removed
Otherwise looks ok.
Thanks, Roger
On 9/6/19 1:59 PM, naoto.s...@oracle.com wrote:
Hello,
Please review the fix to the following issue:
https://bugs.openjdk.java.net/browse/JDK-8230284
The proposed changeset is located at:
https://cr.openjdk.java.net/~naoto/8230284/webrev.00/
The original enhancement for the accounting currency format support
(https://bugs.openjdk.java.net/browse/JDK-8215181) did not account for
the explicit/implicit numbering script. The above change made it to
share the same code with NumberElements which properly deals with the
numbering script.
Naoto
