On Mon, 1 Apr 2024 18:24:53 GMT, Mandy Chung <mch...@openjdk.org> wrote:

>> Suchismith Roy has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   spaces
>
> src/java.base/aix/classes/jdk/internal/loader/ClassLoaderHelper.java line 48:
> 
>> 46:     * file may be located at the alternate location.
>> 47:     * For most platforms, this behavior is not supported and returns 
>> null.
>> 48:     */
> 
> Suggestion:
> 
>     /**
>     * AIX implementation of JVM_LoadLibrary handles the alternate path name 
> mapping.
>     * If loading of the given library name with ".so" suffix fails, it will 
> attempt to load the library of
>     * the same name with ".a" suffix as the alternate name.
>     * 
>     * This method simply returns null.  It could implement the alternate name 
> converting ".so" with ".a" suffix but redundant.

Nit: please add a new line above line 43.

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

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

Reply via email to