[jdk21u-dev] RFR: 8320890: [AIX] Find a better way to mimic dl handle equality
Martin Doerr
mdoerr at openjdk.org
Sat Feb 10 07:43:05 UTC 2024
On Fri, 9 Feb 2024 16:42:01 GMT, Joachim Kern <jkern at openjdk.org> wrote:
> Hi all,
>
> This pull request contains a backport of commit [b8ae4a8c](https://github.com/openjdk/jdk/commit/b8ae4a8c0985d1763ac48ba78943d8b992d7be77) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository.
>
> The commit being backported was authored by Joachim Kern on 11 Jan 2024 and was reviewed by Thomas Stuefe and Martin Doerr.
>
> Thanks!
Looks correct, but please minimize diffs to upstream version!
src/hotspot/os/aix/os_aix.cpp line 1132:
> 1130:
> 1131: const char* error_report = nullptr;
> 1132: void * result = Aix_dlopen(filename, dflags, &error_report);
Diff to upstream version should get minimized: `void* result;` is in a separate line here: https://github.com/openjdk/jdk/commit/b8ae4a8c0985d1763ac48ba78943d8b992d7be77
src/hotspot/os/aix/porting_aix.cpp line 1170:
> 1168:
> 1169: return res;
> 1170: } // end: os::pd_dll_unload()
Newline diff.
src/hotspot/os/posix/os_posix.cpp line 740:
> 738: char ebuf[1024];
> 739: bool res = os::pd_dll_unload(lib, ebuf, sizeof(ebuf));
> 740:
Empty lines are different than in upstream version.
-------------
PR Review: https://git.openjdk.org/jdk21u-dev/pull/252#pullrequestreview-1873491689
PR Review Comment: https://git.openjdk.org/jdk21u-dev/pull/252#discussion_r1484998854
PR Review Comment: https://git.openjdk.org/jdk21u-dev/pull/252#discussion_r1484999021
PR Review Comment: https://git.openjdk.org/jdk21u-dev/pull/252#discussion_r1484999527
More information about the jdk-updates-dev
mailing list