Skip to content

Add zstd #213

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
Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add zstd #213

wants to merge 5 commits into from

Conversation

@mikkoi
Copy link

@mikkoi mikkoi commented Oct 25, 2025

Implement #205

mikkoi added 2 commits October 25, 2025 18:20
@mikkoi
Add support for Zstandard compression
01c9869 
* Use IO::Compress::Zstd instead of Compress::Zstd because the latter
  has no GitHub activity from the part of the developer since 2019.
* Issue: libwww-perl#205
@mikkoi
Add notice about Base64 encoding availability
075558c
@mikkoi
Copy link
Author

mikkoi commented Oct 25, 2025

This PR is only to implement Zstandard compression but I would like to fix the following issues in the code, if you see these worth doing:

  • Repo has tabs and spaces. Move to 4 space indentation everywhere.
  • Use use Carp instead of require Carp. Then remove the Carp:: part from Carp::croak.
  • Use Module::Load::Conditional with the other optional dependencies.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mikkoi