RFR: 8253742: POSIX signal code cleanup [v7]

[Gerard Ziemski]

On Mon, 16 Nov 2020 20:15:24 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

>> Gerard Ziemski has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix missing os::print_signal_handlers on Windows, white space on Linux
>
> src/hotspot/os/posix/signals_posix.hpp line 31:
> 
>> 29: #include "utilities/globalDefinitions.hpp"
>> 30: 
>> 31: // Forward declarations to be independent of the include structure.
> 
> I don't think you have to write this, this is clear from the context (we do this in a zillion other places too).

Fixed.

> src/hotspot/os/posix/signals_posix.cpp line 1389:
> 
>> 1387: }
>> 1388: 
>> 1389: int PosixSignals::unblock_thread_signal_mask(const sigset_t *set) {
> 
> I don't think we need this (nor the prototype in the header)

Fixed.

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

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