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

Feature request: Have DekuError give position of errors when possible #347

Open
typedrat opened this issue Jun 22, 2023 · 0 comments
Open
Labels
enhancement New feature or request

Comments

@typedrat
Copy link

Unfortunately, the smallest example I have of the binary format I am currently trying to parse is several megabytes in size, and so it is very hard to find the exact location of an error like Parse error: Missing magic value [80, 71] when this occurs at the start of each of several thousand records.

I haven't looked into the implementation, but based on my experience working with parsers it shouldn't be especially hard to add at the very least a byte offset into the input at which the error occurred.

@sharksforarms sharksforarms added the enhancement New feature or request label Jun 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants