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

Mopping #1090

Merged
merged 7 commits into from
Jul 19, 2023
Merged

Mopping #1090

merged 7 commits into from
Jul 19, 2023

Conversation

ethteck
Copy link
Member

@ethteck ethteck commented Jul 19, 2023

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.

@ethteck ethteck requested a review from bates64 July 19, 2023 07:59
Copy link
Member

@bates64 bates64 left a 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?

tools/build/configure.py Outdated Show resolved Hide resolved
@ethteck
Copy link
Member Author

ethteck commented Jul 19, 2023

or is there something blocking that?

yeah there is unfortunately. more splat work will be needed, but I plan on incrementally working on it

@ethteck ethteck merged commit 9fdb109 into main Jul 19, 2023
1 check passed
@ethteck ethteck deleted the misc-decomp branch July 19, 2023 09:01
pull bot pushed a commit to skyhoshi/games-papermario that referenced this pull request Jul 19, 2023
* 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
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.

2 participants