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.

Sign up for GitHub

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

Jump to bottom

Fix Lazy decoding #8

Merged
merged 1 commit into from
Jun 25, 2018
Merged

Fix Lazy decoding #8

merged 1 commit into from
Jun 25, 2018

Conversation

tvh
Copy link
Contributor

@tvh tvh commented Jun 25, 2018

This fixes #5 and #7.

@hvr
Copy link
Member

hvr commented Jun 25, 2018

This looks almost like the version I came up with (except I went for two mutually recursive functions to avoid the Maybe)... you could have said you were working on this so I could have worked on something else instead... ;-)

@hvr hvr merged commit f6f3d7c into haskell:master Jun 25, 2018
@hvr
Copy link
Member

hvr commented Jun 25, 2018

Thanks btw!

@23Skidoo 23Skidoo mentioned this pull request Feb 5, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Lazy decoding is not lazy when decoding fails
2 participants
@tvh @hvr