RFR: 8375580: Avoid using ArrayDeque in jdk.internal.loader.URLClassPath [v2]

Eirik Bjørsnøs eirbjo at openjdk.org
Mon Jan 26 13:49:53 UTC 2026


On Mon, 26 Jan 2026 13:21:03 GMT, Jaikiran Pai <jpai at openjdk.org> 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


More information about the core-libs-dev mailing list