RFR: 8253742: POSIX signal code cleanup [v3]

Gerard Ziemski gziemski at openjdk.java.net
Fri Nov 6 16:57:58 UTC 2020


On Fri, 6 Nov 2020 16:43:33 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:

>> src/hotspot/os/posix/signals_posix.cpp line 1286:
>> 
>>> 1284: void PosixSignals::print_signal_handlers(outputStream* st, char* buf, size_t buflen) {
>>> 1285:   st->print_cr("Signal Handlers:");
>>> 1286:   PosixSignals::print_signal_handler(st, SIGSEGV, buf, buflen);
>> 
>> You shouldn't need the PosixSignals:: prefix in this method.
>
> Goo catch, fixed.

Actually now that we are changing `PosixSignals::print_signal_handlers()` to `os::print_signal_handlers()` we do need the `PosixSignals::` prefix after all.

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

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


More information about the hotspot-dev mailing list