On Thu, 11 Sep 2025 03:12:33 GMT, Chen Liang <[email protected]> wrote:
>> While you are at it, both findNode and BuildModulesNode should have comments >> indicating that the node can be found in one of two places. > > If something shouldn't happen, I would prefer an explicit exception or at > least an assert. Not 100% sure it's impossible to tickle this with current callers (e.g. "jrt:/module//path" leaving the trailing path starting with a '/') so I'd rather not risk triggering IAE in existing code. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27203#discussion_r2339884131
