<i18n dev> RFR: 8344252: SM cleanup in java.util classes [v3]

Eirik Bjørsnøs eirbjo at openjdk.org
Mon Dec 9 09:02:47 UTC 2024


On Mon, 9 Dec 2024 07:58:35 GMT, Alan Bateman <alanb at openjdk.org> wrote:

> If the former, and it was never documented, there is a lot more flexibility to change.

Fair enough. However, this SM change seems to accidentally have introduced a bug where the system property is read twice, first calling System.getProperty to look up the name of the system property which is then looked up by  Boolean.getBoolean. That behavior just seems broken. 

@RogerRiggs may have intended to use `Boolean::parseBoolean` instead:

`Boolean.parseBoolean(System.getProperty("resource.bundle.debug", "false"))`

which is equivalent to:

`Boolean.getBoolean("resource.bundle.debug")`

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22122#discussion_r1875603019


More information about the i18n-dev mailing list