> This PR addresses a JCK test failure related to `DecimalFormatSymbols` > de-serialization. While the current public API of DFS disallows a null > locale, it was possible to set in the past. Thus, the > `loadNumberData(locale)` call currently throws NPE when locale is null in the > stream. The call should be guarded with a null check, such that if locale is > null, then `lenientMinusSigns` defaults to `minusSignText`. > > Defaulting the locale field when `null` to Locale.ROOT is also a reasonable > solution, but I think that the current one is preferable as a user would not > expect locale data related logic to occur if locale is `null`.
Justin Lu has updated the pull request incrementally with two additional commits since the last revision: - Nested class rename - Add JDK regression test ------------- Changes: - all: https://git.openjdk.org/jdk/pull/27008/files - new: https://git.openjdk.org/jdk/pull/27008/files/0ace8f6a..751d00be Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=27008&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=27008&range=00-01 Stats: 262 lines in 1 file changed: 262 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk/pull/27008.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/27008/head:pull/27008 PR: https://git.openjdk.org/jdk/pull/27008
