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

add specification for lrzip #664

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

armijnhemel
Copy link
Collaborator

This PR adds a specification for the lrzip format. It mostly works, except that if there is a trailing MD5 sum it isn't easy to automatically recognize it because it is necessary to first find out where the data ends and propagate that (and I don't know how to do that cleanly with instances).

Also, this one will not work with very big files, as I was too lazy to implement something like b40le, b48le and b56le, so there is still room for improvement.

@KOLANICH
Copy link
Contributor

I guess

switch-on: byte_width
cases:
  1: u1
  2: u2
  3: b24le
  4: u4
  8: u8

can be an own type.

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.

2 participants