Skip to content

Report crash messages on Windows to the CRT #369

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

Merged
merged 1 commit into from
Jun 20, 2018
Merged

Report crash messages on Windows to the CRT #369

merged 1 commit into from
Jun 20, 2018

Conversation

adierking
Copy link
Contributor

Provide a Windows implementation of the _dispatch_set_crash_log_* macros which
pass the crash message to the CRT using _RPTF0. If libdispatch is linked against
a debug version of msvcrt, this will display a dialog with detailed information
about the crash and provide an option to attach a debugger. On release versions
of msvcrt, this reporting is a no-op.

Example of what this dialog looks like (from an actual issue I encountered):

Screenshot

Provide a Windows implementation of the _dispatch_set_crash_log_* macros which
pass the crash message to the CRT using _RPTF0. If libdispatch is linked against
a debug version of msvcrt, this will display a dialog with detailed information
about the crash and provide an option to attach a debugger. On release versions
of msvcrt, this reporting is a no-op.
@adierking
Copy link
Contributor Author

cc @MadCoder @compnerd

@compnerd
Copy link
Member

I like this. It definitely would improve the developer experience. I am in favor of this, but will let @parkera or @MadCoder make the final call on this.

@MadCoder
Copy link
Contributor

this is exactly how it's meant to be used, excellent!

@MadCoder
Copy link
Contributor

@swift-ci please test

@MadCoder MadCoder merged commit 12ff819 into swiftlang:master Jun 20, 2018
@adierking adierking deleted the crtassert branch December 2, 2018 00:33
ktopley-apple pushed a commit that referenced this pull request Dec 6, 2018
Report crash messages on Windows to the CRT

Signed-off-by: Kim Topley <ktopley@apple.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants