Skip to content
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

Optionally disable printing of diagnostics during terminate. #3188

Merged
merged 1 commit into from Feb 23, 2018

Conversation

hkaiser
Copy link
Member

@hkaiser hkaiser commented Feb 22, 2018

This also removes the call to std::abort in the termination handler as the system is aborting anyways.

This also removes the call to std::abort in the termination handler as the system is aborting anyways.
Copy link
Contributor

@parsa parsa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as intended.

@sithhell
Copy link
Member

Out of curiosity, what's the original problem this PR is addressing?

@hkaiser
Copy link
Member Author

hkaiser commented Feb 22, 2018

Out of curiosity, what's the original problem this PR is addressing?

If you work with Python loading modules that depend on HPX, then forcefully existing the python interpreter causes HPX to print the full debug info trace, which is just confusing. This allows to disable printing things, if required.

@hkaiser hkaiser merged commit a4c18bd into master Feb 23, 2018
@hkaiser hkaiser deleted the termination_handler branch February 23, 2018 13:16
sithhell pushed a commit that referenced this pull request Mar 13, 2018
This abort was removed with #3188.
sithhell pushed a commit that referenced this pull request Mar 15, 2018
This abort was removed with #3188.
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.

None yet

3 participants