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

smx-files.md: Signed ints for sprite centre #1222

Merged
merged 1 commit into from Mar 21, 2020
Merged

Conversation

@puzzlepaint
Copy link
Contributor

puzzlepaint commented Mar 20, 2020

I think that the sprite centre values have to be interpreted as signed (instead of unsigned) integers. For example, u_vil_female_lumberjack_decayA_x1.smx has a number of frames for which the sprite centre has negative coordinates.

I think that the sprite centre values have to be interpreted as signed integers. For example, `u_vil_female_lumberjack_decayA_x1.smx` has a number of frames for which the sprite centre is outside of the graphic.
Copy link
Contributor

heinezen left a comment

Thank you :)

@heinezen heinezen merged commit f4f95a7 into SFTtech:master Mar 21, 2020
13 checks passed
13 checks passed
macosx-build
Details
kevin-ci 2 jobs succeeded
kevin-ci: arch-clang completed
Details
kevin-ci: arch-clang-00 sanity_check completed in 20.28 seconds
Details
kevin-ci: arch-clang-01 configure completed in 11.68 seconds
Details
kevin-ci: arch-clang-02 build completed in 6.89 seconds
Details
kevin-ci: arch-clang-03 test completed in 1.76 seconds
Details
kevin-ci: arch-clang-04 install completed in 1.46 seconds
Details
kevin-ci: arch-gcc completed
Details
kevin-ci: arch-gcc-00 configure completed in 11.34 seconds
Details
kevin-ci: arch-gcc-01 build completed in 10.03 seconds
Details
kevin-ci: arch-gcc-02 test completed in 1.64 seconds
Details
kevin-ci: arch-gcc-03 install completed in 1.52 seconds
Details
heinezen added a commit to heinezen/openage that referenced this pull request Mar 24, 2020
heinezen added a commit to heinezen/openage that referenced this pull request Mar 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.