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

unexpand: handle too large "tabs" arguments #3655

Merged
merged 1 commit into from
Jun 22, 2022

Conversation

cakebaker
Copy link
Contributor

Return ParseError::TabSizeTooLarge instead of ParseError::InvalidCharacter if the provided tabs argument is too large.

This PR makes the obs-ovflo test in https://github.com/coreutils/coreutils/blob/master/tests/misc/unexpand.pl pass.

@sylvestre
Copy link
Sponsor Contributor

sylvestre commented Jun 20, 2022

Indeed!

Warning: Congrats! The gnu test tests/misc/unexpand is no longer failing!

Congrat!

@sylvestre sylvestre force-pushed the show_tab_size_too_large_error branch from 86644b8 to cf605c2 Compare June 22, 2022 07:30
@sylvestre sylvestre merged commit 1e79035 into uutils:main Jun 22, 2022
@cakebaker cakebaker deleted the show_tab_size_too_large_error branch June 22, 2022 14:03
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.

None yet

2 participants