Thanks to visit codestin.com
Credit goes to github.com

Skip to content

LZW Encoding/Decoding #55

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 1 commit into from
May 11, 2015
Merged

LZW Encoding/Decoding #55

merged 1 commit into from
May 11, 2015

Conversation

filipefalcaos
Copy link
Contributor

Lempel–Ziv–Welch (LZW) is a universal lossless data compression
algorithm. It is an improved implementation of the LZ78 algorithm.

@filipefalcaos
LZW Encoding/Decoding
25dc6aa 
Lempel–Ziv–Welch (LZW) is a universal lossless data compression
algorithm. It is an improved implementation of the LZ78 algorithm.
@mgechev
Copy link
Owner

mgechev commented May 11, 2015

Great! This was in my todo list! Thanks!

mgechev added a commit that referenced this pull request May 11, 2015
@mgechev
Merge pull request #55 from FilipeFalcaoBatista/lzw-compression
94a316a 
LZW Encoding/Decoding
@mgechev mgechev merged commit 94a316a into mgechev:master May 11, 2015
@filipefalcaos filipefalcaos deleted the lzw-compression branch May 11, 2015 11:03
@Yaffle
Copy link

Yaffle commented Dec 19, 2015

I am sorry, but the code in example does not work

+ * var lzw = new lzwModule.LZW();
 + *
 + * var compressed = lzw.compress("ABCABCABCABCABCABC");
 + * console.log(compressed);
 + *
 + * var decompressed = lzw.decompress(compressed);
 + * console.log(decompressed);

because the "dictionarySize" field of lzw was modified... Why not to have a static property instead?

@mgechev
Copy link
Owner

mgechev commented Dec 20, 2015

@Yaffle sorry for the inconvenience. Would you initiate a pull request with the change you suggested?

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.
3 participants
@filipefalcaos @mgechev @Yaffle