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

Field_error improved, Boxed error kind, explicit naming also for DecodeErrorKind #197

Merged
merged 2 commits into from
Nov 10, 2023

Conversation

Nicceboy
Copy link
Contributor

@Nicceboy Nicceboy commented Nov 8, 2023

Boxed Kind type for error variants.
Also some other error improvements.

Parser errors from Incomplete and MismatchedTag does not include backtrace anymore.
Should restore the performance.

@XAMPPRocky
Copy link
Collaborator

Thank you for your PR! I'm unsure about making the backtrace optional. Since I'm not sure this is less complex than fixing the decode_sequence_of and decode_set_of

@Nicceboy
Copy link
Contributor Author

Nicceboy commented Nov 9, 2023

I was also thinking cases if someone builds logic for 'Incomplete' errors, not just the internal handling.

But I can try to find another solution.

@Nicceboy Nicceboy changed the title Optional Backtrace for decoding, field_error improved Field_error improved, Boxed errror kind, explicit naming Nov 9, 2023
@Nicceboy
Copy link
Contributor Author

Nicceboy commented Nov 9, 2023

I reverted the optional backtrace, maybe it is better to check other changes first

@Nicceboy Nicceboy changed the title Field_error improved, Boxed errror kind, explicit naming Field_error improved, Boxed error kind, explicit naming Nov 9, 2023
@Nicceboy Nicceboy changed the title Field_error improved, Boxed error kind, explicit naming Field_error improved, Boxed error kind, explicit naming also for DecodeErrorKind Nov 9, 2023
@XAMPPRocky XAMPPRocky merged commit a75b26b into librasn:main Nov 10, 2023
65 checks passed
@Nicceboy Nicceboy deleted the error-impr branch November 10, 2023 18:03
@github-actions github-actions bot mentioned this pull request Nov 10, 2023
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