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

Fix EOF on inputstream #1

Closed
wants to merge 1 commit into from
Closed

Fix EOF on inputstream #1

wants to merge 1 commit into from

Conversation

ilyail3
Copy link

@ilyail3 ilyail3 commented May 10, 2017

No description provided.

@kervel
Copy link

kervel commented Jan 17, 2019

Thumbs up for this pull request.. i spent quite some time debugging lz4_stream before i found this pull request and the fix works for me. Thanks!

Frank

@laudrup
Copy link
Owner

laudrup commented Jan 21, 2019

@kervel

I'm sorry I've completely forgotten to look into this for such a long time.

Do you think you could provide some example data where this fails?

This will make it a lot easier for me to fix.

Thanks a lot.

@laudrup
Copy link
Owner

laudrup commented Jan 22, 2019

@kervel OK. I see the problem and have found a way to reproduce it.

Will look into to it.

@laudrup
Copy link
Owner

laudrup commented Jan 28, 2019

77faf81 should hopefully be the correct fix for this. Closing this PR umerged. Please let me know if any of you are still experiencing issues after this update.

Thanks a lot.

@laudrup laudrup closed this Jan 28, 2019
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.

None yet

3 participants