Skip to content
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

Fixed Lz4Raw compression error (if input is tiny) #118

Merged

Conversation

dantengsky
Copy link
Contributor

Issue

By using lz4::block::compress_bound to determine the size of output buffer

Using `lz4::block::compress_bound` to determine the size of output buffer
@codecov-commenter
Copy link

Codecov Report

Merging #118 (ea7820d) into main (f8eb243) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #118   +/-   ##
=======================================
  Coverage   74.51%   74.51%           
=======================================
  Files          77       77           
  Lines        3410     3410           
=======================================
  Hits         2541     2541           
  Misses        869      869           
Impacted Files Coverage Δ
src/compression.rs 82.22% <100.00%> (ø)

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 f8eb243...ea7820d. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit 00da3d0 into jorgecarleitao:main Apr 6, 2022
@jorgecarleitao jorgecarleitao added the bug Something isn't working label Apr 6, 2022
@jorgecarleitao
Copy link
Owner

Thanks a lot!

@jorgecarleitao jorgecarleitao changed the title Fix Lz4Raw compression error (if input is tiny) Fixed Lz4Raw compression error (if input is tiny) Apr 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants