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

Move GameFrameworkNx and DebugFontMgrNvn for correct file layout #131

Merged
merged 2 commits into from
Aug 31, 2023

Conversation

MonsterDruide1
Copy link
Contributor

@MonsterDruide1 MonsterDruide1 commented Aug 22, 2023

Moving around some of the files according to some messages in the Discord-Server:
https://discord.com/channels/688807550715560050/794951602396987432/1115848275581485107

(not directly quoted, but adjusted/corrected according to later messages)

DebugFontMgrJis1Nvn has it's own file instead of being tied to seadFontMgr
so it would be gfx/nvn/seadDebugFontMgrNvn.cpp

also seadGameFrameworkNx should be in framework/nx instead of just framework


This change is Reviewable

@MonsterDruide1
Copy link
Contributor Author

Might close #128 ?

Copy link

@break-core break-core left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Also, for skeptical maintainers, these file paths were sourced from MK8DX:
D:/home/Project/TurboS/App/Lib/sead/library/modules/src/framework/nx/seadGameFrameworkNx.cpp
D:/home/Project/TurboS/App/Lib/sead/library/modules/src/gfx/nvn/seadDebugFontMgrNvn.cpp

Reviewable status: 0 of 6 files reviewed, all discussions resolved

@MonsterDruide1
Copy link
Contributor Author

@break-core Does this fully cover the second point you described in the issue? I was confused by your sentence about merging stuff...

@break-core
Copy link

@MonsterDruide1 Yes, this should fully cover that. Everything looks to be correct, sorry about any confusion.

@leoetlino leoetlino merged commit 42cbaae into open-ead:master Aug 31, 2023
1 check passed
@MonsterDruide1 MonsterDruide1 deleted the troy-reorder branch August 31, 2023 21:16
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.

3 participants