Aleksey Shipilëv
Aleksey Shipilëv
Thank you! /integrate
Ah, labels. Let's pretend this is the same as [JDK-8290059](https://bugs.openjdk.org/browse/JDK-8290059). /label add core-libs
All right, thanks! If there are no other suggestions/complaints, I am going to integrate it.
> > Fix looks good as far as it goes (can't believe I didn't see what was going on here!) - but `os::signal` is still broken as it uses `sa_handler`...
> For POSIX we expect the handler argument for os::signal to be a sigaction handler (taking 3 arguments). For Windows we expect os::signal to take a 1 argument handler, since...
> ...unless @shipilev really wants it? :) No, I do NOT want it :) I'll be happy to test it, though.
Build system changes now merged with upstream (look how nice they are). Ready to integrate.
I think we want to ask @magicus about this as well, maybe he will discover some unusual quirks in this code. Otherwise, I'll sponsor.