On Sat, 8 Mar 2025 00:28:15 GMT, Henry Jen <[email protected]> wrote:
> JDK-8349989: jlink can use String.replace instead of String.replaceAll
src/jdk.jlink/share/classes/jdk/tools/jlink/internal/ImageResourcesTree.java
line 303:
> 301: }
> 302: String pkg = removeRadical(path, module + "/");
> 303: return pkg.replace("/", ".");
With the conversion to `replace`, we can use the char literals `'/'` and `'.'`
instead of the string literals, which gives better performance as well.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23954#discussion_r1986384292