Skip to content
This repository was archived by the owner on Jul 3, 2019. It is now read-only.

Fix PNG color type read#2443

Open
demurgos wants to merge 1 commit into
mozilla:masterfrom
open-flash:read-png-color-type
Open

Fix PNG color type read#2443
demurgos wants to merge 1 commit into
mozilla:masterfrom
open-flash:read-png-color-type

Conversation

@demurgos

@demurgos demurgos commented Feb 12, 2018

Copy link
Copy Markdown

The color type is the byte at offset 9 in the IHDR PNG chunk
(offset 13 from the chunk size and 25 since the start of the file).
Shumway was incorrectly reading the byte 14 corresponding to the
compression method.

The color type is the byte at offset 9 in the IHDR PNG chunk
(offset 13 from the chunk size and 25 since the start of the file).
Shumway was incorrectly reading the byte 10 corresponding to the
compression method.
@ivanpopelyshev

ivanpopelyshev commented Mar 27, 2018

Copy link
Copy Markdown

You can make that against https://github.com/ivanpopelyshev/shumway , it supports latest typescript version.

UPD. I've added this fix manually :)

ivanpopelyshev added a commit to ivanpopelyshev/shumway that referenced this pull request Mar 27, 2018
@demurgos

Copy link
Copy Markdown
Author

Thanks for the reply. I noticed your project previously but it has a lot of changes and I haven't time to review it.

I am myself working on a player based off Shumway (Open Flash) but I have larger architectural changes (to support Rust/Wasm but also reuse only some parts for other tools). That's how I found the issue. Still, Shumway does not seem to use this info in the end because it resorts to the browser's parser and does not use the image size before the image is loaded.

@ivanpopelyshev

Copy link
Copy Markdown

I have two branches there - one "master" thats just updated to latest typescript, and one WIP refactoring that i didnt finish yet. Its supposed to be new pixi plugin :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants