On Fri, 15 Dec 2023 07:27:14 GMT, Thomas Stuefe <stu...@openjdk.org> wrote:
>> Joachim Kern has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> followed the proposals
>
> src/hotspot/os/aix/os_aix.cpp line 1234:
>
>> 1232:
>> 1233: stringStream Libpath;
>> 1234: if (env == nullptr) {
>
> Proposal for shorter version not needing string assembly:
>
> const char* paths [2] = { env, rtv_linkedin_libpath() }:
> for (int i = 0; i < 2; i ++) {
> const char* this_libpath = paths[i];
> if (this_libpath == nullptr) {
> continue;
> }
> ... do the token thing...
> }
> }
Sorry, I did not clearly understand how this should work. The mystery must be
in _... do the token thing ..._
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16920#discussion_r1427877337
