Skip to content

fix both error of issue 739 #756

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
Merged
merged 2 commits into from
Oct 12, 2020
Merged

fix both error of issue 739 #756

merged 2 commits into from
Oct 12, 2020

Conversation

Liikt
Copy link

@Liikt Liikt commented Oct 1, 2020

This should fix #739

@Liikt Liikt added Hacktoberfest The label for all Hacktoberfest related things! Implementation Edit This provides an edit to an algorithm implementation. (Code and maybe md files are edited.) labels Oct 1, 2020
@thecnoNSMB
Copy link
Contributor

I just ran this past my property test and it passes both cases without finding any further failure modes. It also fixes #659.

leios
leios approved these changes Oct 12, 2020
View reviewed changes
@leios leios merged commit 9dcc06f into algorithm-archivists:master Oct 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leios leios leios approved these changes

Assignees
No one assigned
Labels
Hacktoberfest The label for all Hacktoberfest related things! Implementation Edit This provides an edit to an algorithm implementation. (Code and maybe md files are edited.)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

The Python implementation of Huffman encoding fails on 1-character-long strings and empty strings
3 participants
@Liikt @thecnoNSMB @leios