New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Win32 Perl's signal emulation permanently disables clib signals, then sets them, and excessive win32_signal_context() calls #12679
Comments
From @bulk88Created by @bulk88I noticed a redundant call to win32_signal_context from Perl_csighandler ________________________________________________________________________________
Yet TLS is guarenteed to have an interp by now because of
but, from my research, a CRT sig handler will never be called, except relevant code is Line 4570 in 1feab43
I feel something is redundant here or should be removed. I am not sure Trying to synthesis a ctrl-c with kill didn't work ( Line 1231 in 1feab43
Also Perl_sighandler calls win32_signal_context, yet can only be called Perl's Win32 emulated signals will never be posix compatible, but they Perl Info
|
From @tonycozOn Mon Dec 31 00:53:30 2012, bulk88 wrote:
________________________________________________________________________________
________________________________________________________________________________
I'm not sure that's true, do_raise() is also called from Now, perl does
http://perl5.git.perl.org/perl.git/blob/1feab43956f936b2526693e4394a9653fcea6079:/win32/win32.c#l4570
Perhaps it's to allow interoperability with XS modules that call raise(). I can't think of any other reason. Perhaps that's why Perl_sys_inter_init() calls signal() - so XS calling
http://perl5.git.perl.org/perl.git/blob/1feab43956f936b2526693e4394a9653fcea6079:/win32/win32.c#l1231
What brought me to this ticket was this issue. I was attemptting to write a signal handler test for #85104, but kill I did end up managing to generate a signal caught by Supplying the pid of the process to GenerateConsoleCtrlEvent() never This appears to be a Win32 issue (ie. Windows is strange...) Testing done on Windows 7.
See my note about win32_process_message() above.
Tests are probably the hard part. Tony |
The RT System itself - Status changed from 'new' to 'open' |
From @jkeenanbulk88: If I read the comments in this ticket correctly -- and I concede I might not, because the way it's displaying in RT is somewhat confusing -- TonyC responded in Aug 2013 to your initial post and there has been no correspondence since. Is this still an ongoing concern? If not, then I think we should close the ticket. Thank you very much. -- |
@bulk88 given no response in 4 years, I'm closing this. |
Migrated from rt.perl.org#116254 (status was 'open')
Searchable as RT116254$
The text was updated successfully, but these errors were encountered: