Integrated: 8300228: ModuleReader.find on exploded module throws if resource name maps to invalid file path
Alan Bateman
alanb at openjdk.org
Tue Jan 17 16:29:02 UTC 2023
On Tue, 17 Jan 2023 11:34:52 GMT, Alan Bateman <alanb at openjdk.org> wrote:
> The ModuleReader implementation for exploded modules maps resource names to file paths. A small oversight is that it doesn't handle InvalidPathException which is thrown when the resource name maps to something that can't be parsed as a file path. This has a knock on impact to Class/ClassLoader getResourceXXX and other usages of ModuleReader.
>
> The existing ModuleReaderTest is updated to include additional resource names to test. I've also moved `@Test` annotation to the test methods so that it's a bit clearer which are the test methods in this source file.
This pull request has now been integrated.
Changeset: fb147aae
Author: Alan Bateman <alanb at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/fb147aaea1593e8a13d562d15994f67cdde3eb35
Stats: 55 lines in 2 files changed: 29 ins; 12 del; 14 mod
8300228: ModuleReader.find on exploded module throws if resource name maps to invalid file path
Reviewed-by: jpai, chegar, cstein
-------------
PR: https://git.openjdk.org/jdk/pull/12035
More information about the core-libs-dev
mailing list