-
Notifications
You must be signed in to change notification settings - Fork 77
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
cxx-qt-gen: allow for unused unsafe blocks and resolve no-exception issues #1131
cxx-qt-gen: allow for unused unsafe blocks and resolve no-exception issues #1131
Conversation
Otherwise we get build warnings as CXX has removed this attribute with changes in CXX 1.0.130
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1131 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 71 71
Lines 11927 11962 +35
=========================================
+ Hits 11927 11962 +35 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
fc55a25
to
bae305e
Compare
This allows for no exceptions to be generated for wasm.
deb5947
to
c2a71a0
Compare
c2a71a0
to
2013a01
Compare
2013a01
to
ba8a355
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pretty much looks good, just a minor nit left.
ba8a355
to
e83c0ec
Compare
Just met this warning and can not fix. Thank you. |
No description provided.