Skip to content
This repository has been archived by the owner on Feb 7, 2023. It is now read-only.

Suggestion: More context with the error message #176

Open
laurinenas opened this issue Jan 29, 2020 · 0 comments · May be fixed by #177
Open

Suggestion: More context with the error message #176

laurinenas opened this issue Jan 29, 2020 · 0 comments · May be fixed by #177

Comments

@laurinenas
Copy link

Hey,

Firstly, I would like to express gratitude for the great library!

We rely on it a lot in our code and sometimes get a Wide characters are not allowed error, but then some time is spent locating which character has induced the error.

I believe it would be nice to get some more information about the error. For example:

Wide characters are not allowed. At the position 4963 this character was found: ’

I know this is a trivial change, but I hope it would make the library more user-friendly.

Best wishes,
@laurinenas laurinenas linked a pull request Jan 29, 2020 that will close this issue
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add more context in the error message laurinenas/asmcrypto.js
1 participant
@laurinenas