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

Expose the inner rust side error type via the exception on the c++ side #1063

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

weiznich
Copy link

@weiznich weiznich commented Jul 6, 2022

This is an attempt to address #870 and similar issues.

I attempt to expose more information on the underlying rust error by including the the original error into the c++ side exception. This allows accessing any error related information from the c++ side as well, assuming that the error type is exposed as part of the cxx bridge definition.

I deliberately open this PR in this state, to gather some feedback on the design idea before spending to much time on details.

@weiznich
Copy link
Author

Gentle ping to @dtolnay to get some input on the general design and whether this feature is wanted or not.

@dtolnay
Copy link
Owner

dtolnay commented Jul 17, 2022

I am interested for this crate to support this, but I will not be able to prioritize iterating on the implementation in the near term.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants