Skip to content

expr: Use chars().count() as we can have some multibytes chars #3133

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
Merged
tertsdiepraam merged 1 commit into from
Feb 13, 2022
Merged

expr: Use chars().count() as we can have some multibytes chars #3133

tertsdiepraam merged 1 commit into from
Feb 13, 2022

Conversation

@sylvestre
Copy link
Contributor

@sylvestre sylvestre commented Feb 13, 2022

Partially fixes #3132
Fixes one of the test of tests/misc/expr-multibyte

@sylvestre
expr: Use chars().count() as we can have some multibytes chars
7225fb6 
Partially fixes #3132
Fixes one of the test of tests/misc/expr-multibyte
@sylvestre
Copy link
Contributor Author

sylvestre commented Feb 13, 2022

Doesn't change the final GNU result

@tertsdiepraam tertsdiepraam merged commit 351a56d into uutils:main Feb 13, 2022
@tertsdiepraam
Copy link
Member

tertsdiepraam commented Feb 13, 2022

Still a bit closer probably :)

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

expr is failing with multibyte chars

2 participants

@sylvestre @tertsdiepraam