Skip to content

Conversation

@queueRAM
Copy link
Contributor

  • Decompile levelGetWOBJ (func_8030BD28)
  • Decompile levelGetTPTS (func_8030BD48)
  • Decompile levelGetAPTS (func_8030BD68)
  • Decompile levelGetLPAD (func_8030BD88)
  • Decompile levelGetBNUS (func_8030BDA8)
  • Decompile func_8030B868 (fakematch)
  • Decompile func_8030B964 (fakematch)
  • Decompile func_8030BA60
  • Decompile func_8030B6C0
  • Decompile func_8030BA98
  • Decompile func_8030BDC8
  • Create uv_level.h and name some level object struct data

- Decompile levelGetWOBJ (func_8030BD28)
- Decompile levelGetTPTS (func_8030BD48)
- Decompile levelGetAPTS (func_8030BD68)
- Decompile levelGetLPAD (func_8030BD88)
- Decompile levelGetBNUS (func_8030BDA8)
- Decompile func_8030B868 (fakematch)
- Decompile func_8030B964 (fakematch)
- Decompile func_8030BA60
- Decompile func_8030B6C0
- Decompile func_8030BA98
- Decompile func_8030BDC8
- Create uv_level.h and name some level object struct data
src/app/level.c Outdated
return D_8034F408->countBNUS;
}

Unk8036A878* func_8030BDC8(u8 arg0) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Once again with great assistance from @Bl00D4NGEL for matching func_8030BDC8

@gcsmith
Copy link
Owner

gcsmith commented Jan 17, 2026

Take a look at AllowShortCaseLabelsOnASingleLine:
https://clang.llvm.org/docs/ClangFormatStyleOptions.html

If you think that yields a more readable default formatting for those scenarios, I'm fine with the change.

@queueRAM
Copy link
Contributor Author

Take a look at AllowShortCaseLabelsOnASingleLine: https://clang.llvm.org/docs/ClangFormatStyleOptions.html

If you think that yields a more readable default formatting for those scenarios, I'm fine with the change.

Thanks for the pointer to AllowShortCaseLabelsOnASingleLine. I tried setting this to true, but when running the format, it changes all of the existing switch statements which likely isn't good. I mostly just wanted to change this one since it is essentially a data lookup table in the form of a switch block. Since whitespace can effect code gen, I think I'll defer the decision on this

@gcsmith
Copy link
Owner

gcsmith commented Jan 18, 2026

Existing whitespace sensitive switch/case could should already be wrapped in clang-format off/on, but we can revisit formatting later.

@gcsmith gcsmith merged commit 75e952f into gcsmith:main Jan 18, 2026
1 check passed
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