On Mon, 26 Jun 2023 05:06:13 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> Tom Rodriguez has updated the pull request incrementally with three
>> additional commits since the last revision:
>>
>> - Flatten nested ifs
>> - Adjust ordering of message
>> - Adjust printing
>
> src/jdk.hotspot.agent/macosx/native/libsaproc/ps_core.c line 302:
>
>> 300: // The base of the library is offset by a random amount which
>> ends up as a load command with a
>> 301: // filesize of 0. This must be ignored otherwise the base
>> address of the library is wrong.
>> 302: if (segcmd.filesize != 0 && add_map_info(ph, fd, segcmd.fileoff,
>> segcmd.vmaddr, segcmd.vmsize, segcmd.flags) == NULL) {
>
> I actually preferred this part with two `if` statements. Seems easier to
> read that way.
Ok. I reverted it to 2 ifs.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14569#discussion_r1242483550
