diff options
| author | Roman971 <32455037+Roman971@users.noreply.github.com> | 2022-06-13 01:24:45 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-06-12 19:24:45 -0400 |
| commit | feadb0d9ee54ecf9db8d8eaea03f5565e2011271 (patch) | |
| tree | 9c5edc44deec6b288719679564c39fade27c0009 /src/code/code_800F9280.c | |
| parent | 4cb1c3345f3134670537682f4ada2c2a5bdc5a13 (diff) | |
Enable more IDO warnings and apply fixes (#1264)
* Update asm-processor and fix includes with EARLY
* Enable more IDO warnings and disable unwanted warning 516
* Fix most new and remaining warnings
* Improve skelanime comment
* Improve asmproc pragma comment
Co-authored-by: Tharo <17233964+Thar0@users.noreply.github.com>
* Add suggested comment for a wrong prototype
* Update asm-processor with the latest fix
Co-authored-by: Tharo <17233964+Thar0@users.noreply.github.com>
Diffstat (limited to 'src/code/code_800F9280.c')
| -rw-r--r-- | src/code/code_800F9280.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/code/code_800F9280.c b/src/code/code_800F9280.c index 075d609aa..b1d28d681 100644 --- a/src/code/code_800F9280.c +++ b/src/code/code_800F9280.c @@ -96,9 +96,10 @@ typedef enum { } SeqCmdType; void Audio_ProcessSeqCmd(u32 cmd) { - s32 pad[2]; + s32 pad; u16 fadeTimer; u16 channelMask; + u32 channelMaskReversed; u16 val; u8 oldSpec; u8 spec; @@ -305,9 +306,10 @@ void Audio_ProcessSeqCmd(u32 cmd) { // stop channels Audio_QueueCmdS8(0x08000000 | _SHIFTL(playerIdx, 16, 8) | 0xFF00, 1); } - if ((channelMask ^ 0xFFFF) != 0) { + channelMaskReversed = channelMask ^ 0xFFFF; + if (channelMaskReversed != 0) { // with channel mask ~channelMask... - Audio_QueueCmdU16(0x90000000 | _SHIFTL(playerIdx, 16, 8), (channelMask ^ 0xFFFF)); + Audio_QueueCmdU16(0x90000000 | _SHIFTL(playerIdx, 16, 8), channelMaskReversed); // unstop channels Audio_QueueCmdS8(0x08000000 | _SHIFTL(playerIdx, 16, 8) | 0xFF00, 0); } |