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

fix(hocon_token): fix scan error message format #257

Merged
merged 1 commit into from
Jun 10, 2023

Conversation

zmstone
Copy link
Member

@zmstone zmstone commented Jun 4, 2023

prior to this change, the scan error is formated as a utf8 binary string, this caused some unreadable error messages if the error is not formated to console using ~ts.

this commit changes the error report to a map, in which, at least the line number and file paht/name are universally printable --- this is usually enough to find the error.

@zmstone zmstone force-pushed the 0604-fix-scan-error-unicode-display branch from c1ce786 to 1e1119f Compare June 5, 2023 18:46
prior to this change, the scan error is formated as a
utf8 binary string, this caused some unreadable error messages
if the error is not formated to console using ~ts.

this commit changes the error report to a map in which
the line number and file paht/name is readable.
this is usually enough to find the error.
@zmstone zmstone force-pushed the 0604-fix-scan-error-unicode-display branch from 1e1119f to 4d7036e Compare June 9, 2023 06:45
@zmstone zmstone merged commit bd46e34 into emqx:master Jun 10, 2023
@zmstone zmstone deleted the 0604-fix-scan-error-unicode-display branch June 10, 2023 10:18
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.

2 participants