-
Notifications
You must be signed in to change notification settings - Fork 79
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
Throws error on Gif87a header #10
Comments
Have you tried the change locally? I believe I didn't include 87a because I was unsure of any subtle differences in the specs |
What I did was change the header of my gif to 89a manually and it worked. A On Mon, 11 May 2015 16:50 Dean McNamee notifications@github.com wrote:
|
I haven't seen many 87a files, so just curious, which software was outputting 87a? |
PS: Published in NPM as omggif@1.0.6 |
A tool called Pixen that I found in the Mac App Store. Thanks so much for the quick fix. I will be publishing a dependent project On Mon, 11 May 2015 18:28 Dean McNamee notifications@github.com wrote:
|
Hi There,
A program I am using outputs gif87a format files and although your decoder can read them (I think they are a subset of Gif89a) you have a sanity check that makes sure the header is gif 89a. If I get a chance I can fix this and make a PR. Would you like me to do that or fix it yourself.
Ben
The text was updated successfully, but these errors were encountered: