On Thu, 15 Jun 2023 06:29:09 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> Sean Coffey has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Incorporate review comments from Roger and tweak some code > > src/java.base/share/classes/sun/launcher/LauncherHelper.java line 183: > >> 181: case "locale": >> 182: printLocale(); >> 183: break; > > printLocale might need the same treatment to avoid printing the list of all > locales by default. Have you tried putting the new code into say sun.launcher.SecuritySettings to avoid adding so much security conf code into LauncherHelper? Maybe eventually this will need to a more pluggable as -XshowSettings has grown a bit beyond what it was originally for. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/14394#discussion_r1230499915