-
-
Notifications
You must be signed in to change notification settings - Fork 51
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
Improve C++ exceptions stacktrace on iOS #1655
Conversation
That does |
All native plugin files in the Unity Project (in the Assets directory, as well as in Packages) will be linked to UnityFramework target, and will be properly handled. |
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.
So this gets copied to the generated Xcode project by Unity. Any chance this causes issues with Sentry disabled? The includes look fine to me.
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.
Just checked. And we also copy the no-op bridge over. So it's fine.
Fixes #1628
There are still some cases that are not covered by this fix.
If the exception is thrown not from UnityFramework, but from another library, the hook will not be called.
The details can be found in the conversation of the issue.