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 two algorithm bugs #1

Merged
merged 3 commits into from Feb 9, 2015

Conversation

Projects
None yet
2 participants
@kappa
Contributor

kappa commented Feb 8, 2015

These commits fix two deep little bugs that actually prevented proper decompression.

Now all the tests pass, yay.

merrilymeredith added a commit that referenced this pull request Feb 9, 2015

Merge pull request #1 from kappa/last_byte
Fix two algorithm bugs (thanks @kappa)

@merrilymeredith merrilymeredith merged commit 7e9fa16 into merrilymeredith:master Feb 9, 2015

@merrilymeredith

This comment has been minimized.

Owner

merrilymeredith commented Feb 9, 2015

Oh cool, thank you! And thanks for being part of the PR challenge; I'm working on it too!

@kappa

This comment has been minimized.

Contributor

kappa commented Feb 9, 2015

You are welcome :)
This was a very interesting challenge, because the released version on CPAN is clean all around, no low-hanging fruit.

So I had to actually remember LZW :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment