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

Martin Doerr mdoerr at openjdk.org
Mon Apr 15 14:11:50 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

I think it's good. I only found minor nits.

test/jdk/java/lang/RuntimeTests/loadLibrary/aix/LoadAIXLibraryFromArchiveObject.java line 57:

> 55:         // // launch a java application which calls System.loadLibrary and is passed
> 56:         // // the directory containing the native library archive file, through
> 57:         // // -Djava.library.path

Please avoid `// //`.

test/jdk/java/lang/RuntimeTests/loadLibrary/aix/LoadAIXLibraryFromArchiveObject.java line 71:

> 69:         }
> 70:     }
> 71: }

I think files typically have a newline at the end.

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

PR Review: https://git.openjdk.org/jdk/pull/17945#pullrequestreview-2001218422
PR Review Comment: https://git.openjdk.org/jdk/pull/17945#discussion_r1565859302
PR Review Comment: https://git.openjdk.org/jdk/pull/17945#discussion_r1565860272


More information about the core-libs-dev mailing list