On Tue, 29 Jul 2025 17:30:54 GMT, Alan Bateman <al...@openjdk.org> wrote:
>> David Beaumont has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Removing direct references to JrtFS. > > src/java.base/share/classes/jdk/internal/jimage/ImageReader.java line 686: > >> 684: /// to child nodes, while retaining efficiency when child nodes are >> accessed >> 685: /// (since any incomplete nodes will be created and placed in the >> node cache >> 686: /// when the parent was first returned to the user). > > The new class descriptions for Node, Directory and Resource are very useful > for future maintainers. Can you convert these to /* */ style as we've ended > up with a mix in this update. As discussed offline, I converted all the markdown comments for non-visible elements to HTML/JavaDoc. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/26054#discussion_r2242217898