RFR: 8251438: Issues with our POSIX set_signal_handler()
Gerard Ziemski
gziemski at openjdk.java.net
Thu Dec 10 20:16:57 UTC 2020
On Thu, 10 Dec 2020 20:03:47 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:
>> Hi Gerard,
>> Functionally this is fine. Stylistically I would have just done simply:
>>
>> return (act.sa_flags & SA_SIGINFO) != 0 ? CAST_FROM_FN_PTR(address, act.sa_sigaction) : CAST_FROM_FN_PTR(address, act.sa_handler);
>>
>> Thanks,
>> David
>
>> Hi Gerard,
>> Functionally this is fine. Stylistically I would have just done simply:
>>
>> return (act.sa_flags & SA_SIGINFO) != 0 ? CAST_FROM_FN_PTR(address, act.sa_sigaction) : CAST_FROM_FN_PTR(address, act.sa_handler);
>>
>> Thanks,
>> David
>
> Thank you David for the review, but I need to pull it out of JDK16 and redo for JDK17. I think I need to withdraw and close it.
I was told that all I need to do is to wait till official JDK16->JDK17 switch over, and can use this PR for JDK17.
-------------
PR: https://git.openjdk.java.net/jdk/pull/1680
More information about the hotspot-runtime-dev
mailing list