On Wed, 27 Mar 2024 17:18:10 GMT, Mandy Chung <mch...@openjdk.org> wrote:
>> I think we both mean that the `if (name.contains("("))` block should get >> removed. > >> We are not supporting that. Are you referring to the comment in the code ? >> Yeah it should be resconstruction of libname(member_name).so , which is the >> first filename the classLoader constructs. > > Note that `System.mapLibraryName` and `mapAlternativeName` are called for > `System.loadLibrary` (i.e. prepending `lib` and appending `.so`). > "libname(member_name)" is not a valid name and no reason for > `System.loadLibrary` to support it. > I think we both mean that the if (name.contains("(")) block should get > removed. Because of the VM support, we can remove `mapAlternativeName` completely. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/17945#discussion_r1541531589