mirror of
https://github.com/qmlnet/qmlnet.git
synced 2026-05-15 14:15:54 -06:00
[GH-ISSUE #37] Wrap all native callbacks with try/catch to give more details about errors. #17
Labels
No labels
bug
enhancement
help wanted
help wanted
pull-request
question
todo
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/qmlnet#17
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @pauldotknopf on GitHub (Jul 24, 2018).
Original GitHub issue: https://github.com/qmlnet/qmlnet/issues/37
.NET exceptions don't travel along PInvoke boundaries. The entire app just SIG quits.
We should
try{}catch(Exception ex){}all .NET callbacks that are passed to native world, write the exceptions details to stdout, and then rethrow.@pauldotknopf commented on GitHub (Jul 24, 2018):
Even better yet, our callbacks should support an error return code. That way, native code can handle it properly.
@pauldotknopf commented on GitHub (Feb 6, 2019):
Closing in favor of #104