On Mon, 26 Feb 2024 11:24:13 GMT, Suchismith Roy <s...@openjdk.org> wrote:

>> J2SE agent does not start and throws error when it tries to find the shared 
>> library ibm_16_am.
>> After searching for ibm_16_am.so ,the jvm agent throws and error as dll_load 
>> fails.It fails to identify the shared library ibm_16_am.a shared archive 
>> file on AIX.
>> Hence we are providing a function which will additionally search for .a file 
>> on AIX ,when the search for .so file fails.
>
> Suchismith Roy has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>    Address review comments

src/hotspot/os/aix/os_aix.cpp line 1175:

> 1173:   // Shared object in .so format dont have braces, hence they get 
> removed for archives with members.
> 1174:   if (result == nullptr && pointer_to_dot != nullptr && 
> strcmp(pointer_to_dot, old_extension) == 0) {
> 1175:     if (strcmp(pointer_to_dot, old_extension) == 0) {

Can you please remove this redundancy?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16604#discussion_r1503728978

Reply via email to