-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Bring GzipSink/GzipSource to the rest of the targets #1550
Comments
Are you specifically interested in JS and Wasm targets? We’ve already got gzip on JVM and Kotlin/Native. |
Could you please share your use case for gzip on these other platforms? They’re usually sandboxed and the enclosing container can often handle this kind of thing for us. |
I've got GZiped resources that I need to be decompressed: a file format that involves portions of it being gziped. (an uncompressed binary header section, then the rest is compressed) |
Do you need this to work on Wasm? In Node? In the browser? |
for my use case it would be WASM and JS. But without supporting them all we wouldn't be able to use them in common code I don't think right? |
Unfortunately there’s no built-in zlib on browser or Wasm platforms. That means whatever we would do in Okio would either embed zlib or depend on a third party port. I’m quite reluctant to do either! I would like to see a standalone open source project do this! But we don’t need it and I think it’s a bit out of scope for us to integrate external libraries. |
I take your point. I think browsers have support for it as of just a few years ago: but that wouldn't solve WASI, so we still wouldn't get it in common code. I actually have about half of Deflate implemented in KMP, the Huffman Codes part. I was going to move on to the LZ77 part, but the performance wasn't great so far, so I put the project down for now. Well something to keep in mind maybe if things change down the line. |
Being able to use Gzip in common code would be a killer feature.
I surveyed the other options out there:
Would love to see a pure KMP implementation of Gzip!
The text was updated successfully, but these errors were encountered: