Skip to content

Conversation

DitherWither
Copy link

Should close #5707

This adds the --length (short form -l) option, along with unit tests for it

It allows you to specify the output digest size for the blake2b algorithm

Copy link
Contributor

@cakebaker cakebaker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

There are two special cases of length values that are currently not covered and lead to a panic: 0 and lengths > 512 that are divisible by 8.

@DitherWither
Copy link
Author

@cakebaker

Ok, I handled those values, and also added tests for the same

@cakebaker cakebaker merged commit 8fe1b26 into uutils:main Dec 30, 2023
@cakebaker
Copy link
Contributor

Great job, thanks :)

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.

Implement cksum -l
2 participants