rust-brotli

rust-brotli copied to clipboard

BSD or MIT

5

Hello, Cargo.toml lists BSD OR MIT, however LICENSE file contains only info about BSD... Who is right?

ignatenkobrain

Performance metrics

1

Hi, thanks for the awesome implementation (I esp liked the ability to concatenate, and I wish it would become a core standard without the extra flags). Are there any performance...

nyurik

Unnecessary loop

3

https://github.com/dropbox/rust-brotli/blob/5d82b69d1aaee6e0fdac6a35654cf7cea40c1057/src/enc/backward_references/hq.rs#L920 This loop `continue30` is unnecessary because it will eventually break at line 1018. If we remove this loop and change all the `break 'continue30;` into `continue` and change all...

yyzdtccjdtc

Fix partial flush on flush_or_close()

1

`BROTLI_OPERATION_FLUSH` must be re-tried until no more data is available as input and internal buffers are empty (this can take more than one call). Otherwise, the writer is left in...

mtth-bfft

Thanks for your work porting brotli to Rust :) I've had an issue when using your crate, specifically when flushing streams: after a flush, all write operations would fail. This...

mtth-bfft

Feature: Add option to force compressed data to be byte aligned

12

This adds `byte_align` and `bare_stream` parameters to enable production of Brotli compressed blocks that can be used to construct a complete stream with trivial byte-wise concatonation. To my knowledge, this...

ryancdotorg

chmod -x *.rs

2

Signed-off-by: Igor Gnatenko

ignatenkobrain

chore: Exclude unneeded files from crates.io

2

Signed-off-by: Igor Gnatenko

ignatenkobrain

switch back to the C brotli default of 5 bytes of backwards info for hashing copies

2

Right now the q9 has been modded in 1d07ab8b8b08c59e5b47e5c01e27148aac1ff8ca to only use 4 bytes to compute the hashes to copy bytes back from This restores it to the defaults that...

danielrh

Make a reusable Python package, and publish to PyPI

2

What an incredibly useful library! Seeing how there's already the Python bindings, maybe it's possible to package and put on PyPI? In case it's already there, please disregard — I...

mvasilkov