-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add tool to extract hardcoded data from swep1rcr.exe #27
base: master
Are you sure you want to change the base?
Conversation
96582ee
to
e5298ee
Compare
d2d8abb
to
87f01b2
Compare
extract-data.c
Outdated
printf("PodracerHandlingData podracerHandling[] = {\n"); | ||
for(unsigned int i = 0; i < podracerHandlingCount; i++) { | ||
PodracerHandlingData* d = &podracerHandling[i]; | ||
printf(" { %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f, %f },", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: float suffix missing
|
||
// Dump the list of replacement parts | ||
unsigned int replacementPartCount = 7 * 6; // 7 categories x 6 levels - FIXME: Read from file? | ||
PartData* replacementParts = readExe(f, replacementPartOffset, replacementPartCount * sizeof(PartData)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rename to upgradeParts or just upgrades
float BumpMass; // 48 | ||
float DmgImmunity; // 52 | ||
float IsectRadius; // 56 //FIXME: Not sure | ||
} PACKED PodracerHandlingData; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PodracerHandlingTableData
; this allows re-using this structure elsewhere.
See handling.h in future swe1r-re PR.
Closes OpenSWE1R/swe1r-re#13
Closes OpenSWE1R/swe1r-re#15
Closes OpenSWE1R/swe1r-re#16
It also makes the gcc
CFLAGS
more strict and turns on-Werror
. This is to protect againstprintf
issues and other oddities.After merge, a new issue has to be created that reminds us to make this tool more compatible with other versions of the game.