On Mon, 26 Jan 2026 13:21:03 GMT, Jaikiran Pai <[email protected]> wrote:

> I don't know if it was intentional, but this comment and some others added in 
> this PR are missing a `.` after the sentence.

Not intentional, I added periods as separators for the field descriptors where 
this PR adds additional sentences.

> I think we should name this `manifestClassPath`.

Renamed, along with `addManifestClassPaths` and related comments.

I think @liach wanted a name suggesting the DFS search order these URL where 
"expanded" in, while this better descibes the origin of the URL. I have a 
slight preference to the latter.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/29288#discussion_r2727673188
PR Review Comment: https://git.openjdk.org/jdk/pull/29288#discussion_r2727670018

Reply via email to