-
Notifications
You must be signed in to change notification settings - Fork 577
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
Make a XML for ganon_mant #961
Conversation
Co-authored-by: Tharo <17233964+Thar0@users.noreply.github.com>
|
||
<DList Name="gMantTexDL" Offset="0x3318"/> | ||
|
||
<!-- ! @bug gMantUnusedTex is 32x32 not 32x64, however this dlist is unused --> |
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.
Is this the right way to document a bug in assets? Or just lack of a better option
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.
when we move toward committing the c files it will be easier to just put it right in the file. but for now this is all we got i guess
#define MANT_TEX_WIDTH 32 | ||
#define MANT_TEX_HEIGHT 64 | ||
|
||
static u64 sForceDmaAlignment = 0; |
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.
I dont think this should mention anything "dma" related, this "boundary" is inside of a file as far as the dma table knows
Maybe just sForceAlignment
idk
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.
I just moved over the already existing variable from the _gfx.c
file.
But you are right, I'll rename it
* Make a XML for ganon_mant * cleanup * Apply suggestions from code review Co-authored-by: Tharo <17233964+Thar0@users.noreply.github.com> * sForceAlignment Co-authored-by: Tharo <17233964+Thar0@users.noreply.github.com>
I just reused the names from
ovl_en_ganon_mant_gfx.c