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

Add flag to get standard-compliant exception handling #68

Merged
merged 2 commits into from
Oct 26, 2021

Conversation

azeey
Copy link
Contributor

@azeey azeey commented Oct 26, 2021

🦟 Bug fix

Summary

This should fix the following compiler warning with MSVC

 warning C4530: C++ exception handler used, but unwind semantics are not enabled. Specify /EHsc

Checklist

Note to maintainers: Remember to use Squash-Merge

@azeey azeey requested a review from scpeters October 26, 2021 15:38
@azeey azeey requested a review from caguero as a code owner October 26, 2021 15:38
@github-actions github-actions bot added 🌱 garden Ignition Garden 🏢 edifice Ignition Edifice 🏯 fortress Ignition Fortress 🏰 citadel Ignition Citadel 🔮 dome Ignition Dome labels Oct 26, 2021
Copy link

@Blast545 Blast545 left a comment

Choose a reason for hiding this comment

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

LGTM

@azeey
Copy link
Contributor Author

azeey commented Oct 26, 2021

I'm going to test this on a release build before merging.

Build Status

@azeey azeey merged commit b764a7c into ign-tools1 Oct 26, 2021
@azeey azeey deleted the azeey/msvc_ehsc branch October 26, 2021 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏰 citadel Ignition Citadel 🔮 dome Ignition Dome 🏢 edifice Ignition Edifice 🏯 fortress Ignition Fortress 🌱 garden Ignition Garden
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants