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

streamline class definitions #35

Open

robertnurnberg wants to merge 1 commit into tmaklin:master from robertnurnberg:none

Contributor

robertnurnberg commented Feb 3, 2025

This PR implements the changes discussed in #34.

This compiles and works for my local project, but apart from that no extensive testing was done.


          streamline class definitions

6a36f84

robertnurnberg mentioned this pull request

clean up and readme edit #34

Draft

Contributor Author

robertnurnberg commented Feb 3, 2025

Btw, in the line

ostream(std::ostream & os, Compression type = plaintext, int level = 6)

should this be type = z for consistency? Could add the fix to this PR if you want.

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

Labels

None yet