better PNG files, fix WebP#101
Open
landfillbaby wants to merge 6 commits intomathiasbynens:masterfrom
landfillbaby:master
Open
better PNG files, fix WebP#101landfillbaby wants to merge 6 commits intomathiasbynens:masterfrom landfillbaby:master
landfillbaby wants to merge 6 commits intomathiasbynens:masterfrom
landfillbaby:master
Conversation
changed IDAT zlib header to max compression level, min decompression memory truncated a further 4 bytes (the zlib adler32 sum) add version of above with IDAT length header 6 instead of 10 same size but different error (both are ignored by most decoders anyway)
new one is 30 bytes instead of 26, but it decodes at least hopefully a .gitattributes will fix it should probably check other formats too...
Closed
webp-odd.webp: manually removed RIFF even-byte-count padding webp-vp8l.webp: the VP8L chunk without the RIFF, decodes with dwebp webp-vp8l-raw.webp: the data from said chunk, dwebp accepts this too
why not use `-std=C90`?
-std=c90
since we're apparently going by whether the compiler fails rather than standards, which, no matter what version you use, say main is a function (it's just a plain int here) and newline at EOF is required
Author
|
update: there are 28-byte WebPs. |
|
Also, there's an updated |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
changed IDAT zlib header to max compression level, min decompression memory
truncated a further 4 bytes (the zlib adler32 sum)
added 2nd truncated version with IDAT length header 6 instead of 10 to account for truncation being inside the chunk now, same size but different error, one example i found that needs this is GIMP (both are ignored by most decoders anyway), not sure what the rules are here
(i meant to do this pr like a year ago and just found the files again)