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

[Eirik Bjørsnøs]

On Mon, 26 Jan 2026 13:55:54 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

> This wasn't a product bug, so the `@bug` isn't necessary for this new test.

Yes, thanks! I must have thought this was a "product change", but the requirement is indeed a product bug. Tag removed.

> test/jdk/jdk/internal/loader/URLClassPath/JarClassPathDfsOrder.java line 50:
> 
>> 48:  * @run junit JarClassPathDfsOrder
>> 49:  */
>> 50: public class JarClassPathDfsOrder {
> 
> The JAR specification for the `Class-Path` attribute states:
> 
>> The resulting URLs are inserted into the class path, immediately following the URL of the context JAR. For example, given the following class path ...
> 
> so it's good to introduce a test for this (I haven't verified we don't have any). However, I think the use of DFS in the test name and comments can be removed. I think we should name the test `JarManifestClassPathOrder`.

Thanks, I have renamed the test and deemphasized the "DFS" aspect in comments.

The long-standing behavior that we want to prevent regressions of is to find these files in DFS order though, so I left some minimum mentioning of it. 

Take a look at the updated test and let me know what you think.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29288#discussion_r2727740672
PR Review Comment: https://git.openjdk.org/jdk/pull/29288#discussion_r2727737183