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