RFR: JDK-8255711: Fix and unify hotspot signal handlers [v2]
David Holmes
david.holmes at oracle.com
Wed Nov 4 22:41:54 UTC 2020
On 5/11/2020 12:40 am, Thomas Stuefe wrote:
> On Wed, 4 Nov 2020 05:48:14 GMT, David Holmes <dholmes at openjdk.org> wrote:
>
>>> Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
>>>
>>> Remove unnecessary call-once guard from PosixSignals::install_signal_handlers()
>>
>> src/hotspot/os/posix/signals_posix.cpp line 443:
>>
>>> 441: extern "C" JNIEXPORT int
>>> 442: #if defined(BSD)
>>> 443: JVM_handle_bsd_signal
>>
>> Can we define this using token pasting e.g.
>>
>> PASTE_TOKENS(JVM_handle, PASTE_TOKENS(INCLUDE_SUFFIX_OS, _signal))
>>
>> ?
>
> I find that actually less readable than what we have now. Plus, right now its easier to grep for the function name.
True ... I just hate the cascading ifdefs for this kind of thing. I
actually expected to see JVM_handle_xxx_signal still defined in the
platform specific os_xxx.cpp file.
David
-----
> -------------
>
> PR: https://git.openjdk.java.net/jdk/pull/1034
>
More information about the hotspot-dev
mailing list