-
Notifications
You must be signed in to change notification settings - Fork 0
Update streamly to the latest master #52
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
base: master
Are you sure you want to change the base?
Conversation
|
Benchmarks: |
src/Streamly/Internal/LZ4.hs
Outdated
| ++ " maxCompLenC: " ++ show maxCompLenC | ||
| ++ " uncompLenC: " ++ show uncompLenC | ||
| (MArray.Array cont dstBegin_ dstBegin dstMax) <- | ||
| newarr@(MArray.Array cont arrStart arrEnd arrBound) <- |
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.
Make a new pinned array!
|
Need to update stack.yaml for appveyor build. |
|
Hi, is this PR still running? I'd like to use lz4 in latest streamly and happy to help :) |
|
@TheKK the last update on this was long ago, so it is stalled. @adithyaov can you update this, I will review. See if @TheKK can help in some way if you are busy. |
|
Hey @TheKK this PR is stalled. If possible please send a new PR or update this PR accordingly. I can review it.
|
|
Got it. I'd like to work on this one but still have other task to work on. |
No description provided.