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

Suchismith Roy sroy at openjdk.org
Mon Apr 15 16:01:36 UTC 2024


On Mon, 15 Apr 2024 14:48:54 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:

>> 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.

It was a wrong push. Sorry. The autocomplete command was causing some troubles. I had changed it in local to see if the jtreg test fails.

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

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


More information about the core-libs-dev mailing list