diff options
author | Kyle Kienapfel <Docteh@users.noreply.github.com> | 2022-11-11 04:12:37 -0800 |
---|---|---|
committer | Kyle Kienapfel <Docteh@users.noreply.github.com> | 2022-11-13 16:30:55 -0800 |
commit | 6fa3faec658893c9fae19116232d24dac08babc7 (patch) | |
tree | 4e04753b659357d6595fa22766ae305586fd90d4 /src/video_core/macro | |
parent | 5eb30c7827959075cd240f39c6ab57d2c2a8197e (diff) |
Add break for default cases
Visual Studio has an option to search all files in a solution, so I
did a search in there for "default:" looking for any missing break
statements.
I've left out default statements that return something, and that throw
something, even if via ThrowInvalidType. UNREACHABLE leads towards throw
R_THROW macro leads towards a return
Diffstat (limited to 'src/video_core/macro')
-rw-r--r-- | src/video_core/macro/macro_interpreter.cpp | 2 | ||||
-rw-r--r-- | src/video_core/macro/macro_jit_x64.cpp | 1 |
2 files changed, 3 insertions, 0 deletions
diff --git a/src/video_core/macro/macro_interpreter.cpp b/src/video_core/macro/macro_interpreter.cpp index c0d32c112..0d63495a9 100644 --- a/src/video_core/macro/macro_interpreter.cpp +++ b/src/video_core/macro/macro_interpreter.cpp @@ -201,6 +201,7 @@ bool MacroInterpreterImpl::Step(bool is_delay_slot) { } default: UNIMPLEMENTED_MSG("Unimplemented macro operation {}", opcode.operation.Value()); + break; } // An instruction with the Exit flag will not actually @@ -297,6 +298,7 @@ void MacroInterpreterImpl::ProcessResult(Macro::ResultOperation operation, u32 r break; default: UNIMPLEMENTED_MSG("Unimplemented result operation {}", operation); + break; } } diff --git a/src/video_core/macro/macro_jit_x64.cpp b/src/video_core/macro/macro_jit_x64.cpp index 25c1ce798..7347cbd88 100644 --- a/src/video_core/macro/macro_jit_x64.cpp +++ b/src/video_core/macro/macro_jit_x64.cpp @@ -652,6 +652,7 @@ void MacroJITx64Impl::Compile_ProcessResult(Macro::ResultOperation operation, u3 break; default: UNIMPLEMENTED_MSG("Unimplemented macro operation {}", operation); + break; } } |