On Tue, 27 Feb 2024 08:22:21 GMT, Martin Doerr <mdo...@openjdk.org> wrote:
>> Suchismith Roy has updated the pull request incrementally with one >> additional commit since the last revision: >> >> remove redundant logic > > src/hotspot/os/aix/os_aix.cpp line 1176: > >> 1174: if (result == nullptr && pointer_to_dot != nullptr && >> strcmp(pointer_to_dot, old_extension) == 0) { >> 1175: snprintf(pointer_to_dot, sizeof(old_extension), "%s", >> new_extension); >> 1176: result = dll_load_library(file_path, ebuf, ebuflen); > > You should have adapted the indentation, too. Which indentation ? @TheRealMDoerr adapted ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16604#discussion_r1503820212 PR Review Comment: https://git.openjdk.org/jdk/pull/16604#discussion_r1503824233