-
Notifications
You must be signed in to change notification settings - Fork 57
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
Gif support #4
Comments
Hi, Thanks for the information, I tried your decoder, but there is a HUGE problem of performance, I tried it with this huge Gif image and ran into a stack overflow and when launching with more stack space ( For a direct inclusion in the library, a partial rewrite to handle huge loads is needed, but don't worry, it's doable, and having a reference implementation is always good to have. |
Yes this is mostly a problem with the lzw implementation since it uses a very inefficient Map. If you look into it, you will see why (it maps List on Int. so the key is a List which makes for very bad performance). Also, my example main does load the same gif in memory 6 times (and i didn't test if it was garbage collected) |
Reimplemented |
I created a haskell gif parser and writer a while back: https://bitbucket.org/nudded/gif-parser/
I thought you might be interested. It should be able to read any GIF87a file (and most of the GIF89a files). But the lzw code is there (it is quite hard to cover all the edge cases).
I'm having exams atm, so i will be looking into this after them.
The text was updated successfully, but these errors were encountered: