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