-
-
Notifications
You must be signed in to change notification settings - Fork 2
ZDICT_analyzeEntropy cleanup
#235
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
Conversation
3b86c18 to
b4be737
Compare
Otherwise these links prevent the docs from compiling due to E0658
b4be737 to
bd7b357
Compare
cee46fb to
c7e1c66
Compare
folkertdev
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does look good, although codecov is not too happy here. Is there any way we can improve coverage before merging these changes?
|
|
f4009f6 to
d23ceae
Compare
|
Most of the remaining uncovered lines are either error handling or debug prints |
Of course we'd really like
analyze_entropy_internalto be safe eventually, but it uses a lot of functions from the compression side of zstd that we haven't started to clean up yet