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.

Sign up for GitHub

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

Jump to bottom

Fix UTF8 decoding error on some non UTF8 system #79

Merged

zhangxp1998 merged 1 commit into imsnif:master from zhangxp1998:utf8

Jan 6, 2020

Collaborator

zhangxp1998 commented Jan 6, 2020 •

edited

Loading

In response to issue #50

from_utf8_lossy will always succeed, it converts bytes to UTF8 string, escaping/discarding invalid bytes


          Fix UTF8 decoding error

60305f4

Collaborator Author

zhangxp1998 commented Jan 6, 2020

Merging this PR because it's tiny in terms of LoC and change in functionality.

zhangxp1998 merged commit 2412fc9 into imsnif:master

zhangxp1998 deleted the utf8 branch

January 9, 2020 23:50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet