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

The wc cli has been fixed up #152

Open
wants to merge 1 commit into
base: main-dev
Choose a base branch
from

Conversation

MarkReedZ
Copy link

In the wc cli I fixed up a couple bugs, added the files from arg, justified the output to match wc, and sped up the longest line search.

ref #97

$ time python ../StringZilla/cli/wc.py -L  --files0-from delme
        67 sz.js
     51648 enwik9.txt
        67 sz.js
       102 tst.js
     51884 total

real	0m1.184s
user	0m1.080s
sys	0m0.104s

$ time wc -L --files0-from delme
        67 sz.js
     51648 enwik9.txt
        67 sz.js
       102 tst.js
wc: ''$'\n': No such file or directory
     51648 total

real	0m4.327s
user	0m4.184s
sys	0m0.142s

@ashvardanian
Copy link
Owner

Does this relate to #139 in any way? cc @lborcard

@MarkReedZ
Copy link
Author

No, haven't looked at split yet. I'm guessing for 139 he ran out of space. We can update the split code to print a better error message.

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