Skip to content

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Oct 9, 2017

bpo-31701: On Windows, faulthandler.enable() now registers the
exception handler as the last handler to be called, rather than the
first to be called. It prevents handled C++ exceptions to be logged
by faulthandler.

https://bugs.python.org/issue31701

bpo-31701: On Windows, faulthandler.enable() now registers the
exception handler as the last handler to be called, rather than the
first to be called. It prevents handled C++ exceptions to be logged
by faulthandler.
@vstinner vstinner requested a review from zooba October 9, 2017 10:26
@vstinner vstinner changed the title faulthandler.enable() registers exc handler as last bpo-31701:faulthandler.enable() registers exc handler as last Oct 9, 2017
@vstinner vstinner changed the title bpo-31701:faulthandler.enable() registers exc handler as last bpo-31701: faulthandler.enable() registers exc handler as last Oct 9, 2017
@vstinner
Copy link
Member Author

vstinner commented Oct 9, 2017

Oops, my PR doesn't work as expected.

@vstinner vstinner closed this Oct 9, 2017
@vstinner vstinner deleted the faulthandler branch October 9, 2017 10:28
@vstinner vstinner restored the faulthandler branch October 9, 2017 10:50
@vstinner
Copy link
Member Author

vstinner commented Oct 9, 2017

I misunderstood how Windows works. UNIX signals handlers and Windows exception handlers are unrelated. Exception handlers are not called to handle a SIGSEGV signal.

@vstinner vstinner reopened this Oct 9, 2017
@vstinner
Copy link
Member Author

vstinner commented Oct 9, 2017

I tested this PR with a C++ exception: the change has no effect, the exception is still logged by faulthandler. So I abandon this useless change.

@vstinner vstinner closed this Oct 9, 2017
@vstinner vstinner deleted the faulthandler branch October 9, 2017 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants