> Can I please get a review of this change that proposes to address the issue > noted in https://bugs.openjdk.org/browse/JDK-8344908? > > With this change, the URLClassPath will no longer propagate the > `IllegalArgumentException` throwing when constructing a resource loader. The > URL which caused the issue will continue to not be able to serve resources > through that URLClassPath but the URLClassPath itself will now be > functionally usable for locating resources through rest of the URLs. > > While at it, the internal class `FileURLMapper` which gets used in this code > flow has been updated to be more confined by making it package private. > > For addressing the underlying issue with ParseUtil we have > https://bugs.openjdk.org/browse/JDK-8258246 which I plan to look into > separately. There are also a few other usages of `ParseUtil.decode()` in > various other places outside of this code flow which I plan to look into > separately. > > A new jtreg test has been introduced which reproduces the issue and verifies > the fix. This test and other existing tests in tier1, tier2 and tier3 > continue to pass with this change.
Jaikiran Pai has updated the pull request incrementally with two additional commits since the last revision: - Alan's review - rename variable and drop final from test - Alan's review - no need for throws in method signature ------------- Changes: - all: https://git.openjdk.org/jdk/pull/22351/files - new: https://git.openjdk.org/jdk/pull/22351/files/9f9a40d3..20674bdc Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=22351&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=22351&range=01-02 Stats: 7 lines in 2 files changed: 1 ins; 0 del; 6 mod Patch: https://git.openjdk.org/jdk/pull/22351.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/22351/head:pull/22351 PR: https://git.openjdk.org/jdk/pull/22351
