jibsen / brieflz

Small fast Lempel-Ziv compression library
zlib License
101 stars 18 forks source link

ecm brieflz changes #2 #6

Closed ecm-pushbx closed 6 years ago

ecm-pushbx commented 6 years ago

I accidentally deleted/closed the other pull request. Here's some new changes that I discovered. The unbuffering change is the same one I made first. Other changes: --blocksize can increase or decrease the block size, giving --safe makes blzpack use blz_depack_safe, --pad uses a trick to align subsequent headers on an 8-byte boundary, and I optimised the depacker a little.

ecm-pushbx commented 6 years ago

Link for reference: https://github.com/jibsen/brieflz/pull/5

codecov-io commented 6 years ago

Codecov Report

Merging #6 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master     #6   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           3      3           
  Lines         172    170    -2     
=====================================
- Hits          172    170    -2
Impacted Files Coverage Δ
depacks.c 100% <100%> (ø) :arrow_up:
depack.c 100% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a979aa4...515c287. Read the comment docs.

ecm-pushbx commented 6 years ago

Forgot to add help page entry for --pad, added in last commit.

jibsen commented 6 years ago

Thanks, I'll have a look soon.

ecm-pushbx commented 6 years ago

Rebased all the latest commits onto your recent changes now.

jibsen commented 6 years ago

Thanks! I've merged it into staging.

jibsen commented 6 years ago

Merged into master with new release, thanks again.