On Thu, 17 Apr 2025 17:32:03 GMT, Koushik Muthukrishnan Thirupattur <d...@openjdk.org> wrote:
> The private marshal() method in sun.security.util.Debug still contains code > to parse "permission=<classname>" and "codebase=<URL>" options. These > sub-options were part of the "access" option which was removed in JDK 24 as > part of JEP 486, so this code can be removed. src/java.base/share/classes/sun/security/util/Debug.java line 359: > 357: > 358: // convert to lower-case characters > 359: return String.valueOf(args.toLowerCase(Locale.ENGLISH)); You can just return `args.toLowerCase(Locale.ENGLISH)`. Or better yet, you don't need the `marshal` method anymore, and can just replace the one call to marshal with this line. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/24731#discussion_r2067191328
