RFR: 8253742: POSIX signal code cleanup [v3]

Gerard Ziemski gziemski at openjdk.java.net
Tue Nov 3 21:48:02 UTC 2020


On Tue, 3 Nov 2020 21:42:07 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:

>> src/hotspot/os/aix/os_aix.cpp line 1442:
>> 
>>> 1440: }
>>> 1441: 
>>> 1442: void os::print_signal_handlers(outputStream* st, char* buf, size_t buflen) {
>> 
>> This makes sense. But then, it would make sense to move this to os_posix.cpp completely. Or to even completely replace calls to os::print_signal_handlers with PosixSignals::print_signal_handlers() and remove the former.
>
> print_signal_handlers() is declared in os.h, so at first glance I thought I couldn't collapse it further. I'll have a second look.

void os::print_signal_handlers(outputStream* st, char* buf, size_t buflen) is declared in os.h and is used on Windows platform as well.

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

PR: https://git.openjdk.java.net/jdk/pull/636


More information about the hotspot-dev mailing list