Skip to content

Fix: Small dict leads to panic#133

Merged
PSeitz merged 1 commit intomainfrom
no_panic
Jun 7, 2023
Merged

Fix: Small dict leads to panic#133
PSeitz merged 1 commit intomainfrom
no_panic

Conversation

@PSeitz
Copy link
Copy Markdown
Owner

@PSeitz PSeitz commented Jun 7, 2023

Fixes #93 that lead to a panic when the dict length was smaller than 4. A match has the minimum length of 4 bytes, smaller dicts will be ignored

Fixes #131 that lead to a panic when the dict length was smaller than 4. A match has the minimum length of 4 bytes, smaller dicts will be ignored
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 7, 2023

Codecov Report

Merging #133 (84498b8) into main (15e4ffa) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #133      +/-   ##
==========================================
+ Coverage   89.81%   89.85%   +0.04%     
==========================================
  Files          13       13              
  Lines        2474     2485      +11     
==========================================
+ Hits         2222     2233      +11     
  Misses        252      252              
Impacted Files Coverage Δ
src/block/compress.rs 98.07% <100.00%> (+0.03%) ⬆️

@PSeitz PSeitz merged commit 2d83a3d into main Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Small ext_dict leads to panic

1 participant