-
Notifications
You must be signed in to change notification settings - Fork 127
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
Mopping #1090
Mopping #1090
Conversation
subrepo: subdir: "tools/splat" merged: "7869ef2e51" upstream: origin: "https://github.com/ethteck/splat.git" branch: "master" commit: "7869ef2e51" git-subrepo: version: "0.4.5" origin: "https://github.com/ingydotnet/git-subrepo" commit: "aa416e4"
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.
We have a lot of segments - not subsegments - with start: 0xDEADBEEF
... could we change any of these to start: auto
too or is there something blocking that?
yeah there is unfortunately. more splat work will be needed, but I plan on incrementally working on it |
* yaml simplifications * remove unneeded end markers after c files * git subrepo pull --force tools/splat subrepo: subdir: "tools/splat" merged: "7869ef2e51" upstream: origin: "https://github.com/ethteck/splat.git" branch: "master" commit: "7869ef2e51" git-subrepo: version: "0.4.5" origin: "https://github.com/ingydotnet/git-subrepo" commit: "aa416e4" * cleanin * effect_shims.h * PRomment
Removed many hard-coded rom addresses from the US yaml and updated splat to accommodate the change. I hope to eventually remove more and more hard-coded rom addresses, making it easier for us to make yamls for other versions or augment the US one.
I also changed the effect c files to literally refer to the shim functions they call (without the shim_ prefix). This is accomplished the same way as dead.h is - they're renamed when cpp runs over the file.