On Tue, 24 Sep 2024 16:04:58 GMT, Artur Barashev <d...@openjdk.org> wrote:
> https://bugs.openjdk.org/browse/JDK-8340670 I think this fix is trivial and obvious enough that you don't need a test; however if you don't plan to add one, please add a noreg-trivial label to the bug so that it doesn't get flagged as missing a test. src/java.base/share/classes/java/security/Policy.java line 907: > 905: @Override > 906: public boolean isReadOnly() { > 907: return perms.isReadOnly(); You could just return true here. Either way is ok with me. ------------- PR Review: https://git.openjdk.org/jdk/pull/21165#pullrequestreview-2325849471 PR Review Comment: https://git.openjdk.org/jdk/pull/21165#discussion_r1773707174
