On Fri, 14 May 2021 05:39:10 GMT, Athijegannathan Sundararajan <sun...@openjdk.org> wrote:
> Problematic paths that start with /modules/modules prefix are handled properly Marked as reviewed by alanb (Reviewer). src/java.base/share/classes/jdk/internal/jimage/ImageReader.java line 475: > 473: } > 474: // Make sure that the thing that follows "/modules/" is a > module name. > 475: // This will rule out paths that start as > "/modules/modules/". The code change looks fine. I guess I would probably change the comment to say that it make sure that there is something ending with "/" after "/modules/" (as the code that it comments doesn't actually change for a module name). ------------- PR: https://git.openjdk.java.net/jdk/pull/4022
