RFR: 8319516: AIX System::loadLibrary needs support to load a shared library from an archive object [v25]

Magnus Ihse Bursie ihse at openjdk.org
Mon Apr 15 14:52:04 UTC 2024


On Mon, 15 Apr 2024 13:23:22 GMT, Suchismith Roy <sroy at openjdk.org> wrote:

>> Allow support for both .a and .so files in AIX.
>> If .so file is not found, allow fallback to .a extension.
>> JBS Issue: [JDK-8319516](https://bugs.openjdk.org/browse/JDK-8319516)
>
> Suchismith Roy has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Files copy

src/java.base/aix/classes/jdk/internal/loader/ClassLoaderHelper.java line 42:

> 40:     */
> 41:     static boolean loadLibraryOnlyIfPresent() {
> 42:         return true;

I've been following this PR to and fro for a while, so I might have missed something. But it looks like the code and the documentation is completely at odds -- the documentation claims the function should return `false`, but in reality it returns `true`. If the code is correct, the documentation needs to be fixed or clarified.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17945#discussion_r1565925292


More information about the core-libs-dev mailing list