On Mon, 9 Dec 2024 07:58:35 GMT, Alan Bateman <al...@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
