RFR: 8281658: Add a security category to the java -XshowSettings option [v8]
Mandy Chung
mchung at openjdk.org
Thu Jul 6 17:17:57 UTC 2023
On Thu, 29 Jun 2023 08:27:43 GMT, Alan Bateman <alanb at openjdk.org> wrote:
> The changes make me wonder if `-XshowSetting:aardvark` should be an error rather than default to print all settings. Something we should look at again. Same thing for `-XshowSettings:system` on non-Linux, probably should have been a bit more discussion when that was added as its more about the container environment.
I think throwing an error for an invalid suboption makes sense. `-XshowSetting` without the suboption is default to print all settings, i.e. equivalent to `-XshowSetting:all`. But the switch statment is missing the `all` case. I think we should fix this and error for any invalid option.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/14394#issuecomment-1624030798
More information about the core-libs-dev
mailing list