Skip to content
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

fixed: sprite user data positioned right after "Palette Chunk (0x2019)" leads to error while parsing aseprite file #26

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AnAverageGitUser
Copy link

@AnAverageGitUser AnAverageGitUser commented Sep 17, 2024

See point 3. under https://github.com/aseprite/aseprite/blob/v1.3.8/docs/ase-file-specs.md#user-data-chunk-0x2020 for reference.

Until now only the two older palette chunks ("Old palette chunk (0x0004)", "Old palette chunk (0x0011)") showed the correct behaviour.

I don't know in what exact circumstances aseprite decides to use what palette chunk type, but I apparently have a mixture of these in my files, which is how I noticed.

So to clarify: The following chunk orders parsed just fine:

Chunk type: ColorProfile
Chunk type: OldPalette04
Chunk type: UserData
Chunk type: Layer
Chunk type: ColorProfile
Chunk type: OldPalette11
Chunk type: UserData
Chunk type: Layer

The following chunk order failed parsing but parses now fine with this pull request:

Chunk type: ColorProfile
Chunk type: Palette
Chunk type: UserData
Chunk type: Layer

…)" leads to error while parsing aseprite file

See point 3. under https://github.com/aseprite/aseprite/blob/v1.3.8/docs/ase-file-specs.md#user-data-chunk-0x2020 for reference. Until now only the two older palette chunks ("Old palette chunk (0x0004)", "Old palette chunk (0x0011)") showed the correct behaviour. I don't know in what exact circumstances aseprite decides to use what palette chunk type, but I apparently have a mixture of these in my files, which is how I noticed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant