This moves case ...: labels that were previously immediately outside of the switch block to case ...: break; statements at the top of the switch.
This avoids syntax errors (and is easier to read). It fixes the issue that camthesaxman brought up on Discord about func_80048084.
This might not be the absolute cleanest way of implementing this: it's a bit odd to remove labels from case_nodes[...]. But it doesn't break --gotos-only where add_labels_for_switch(...) is used without build_switch_statement(...).
Diffs in PPC projects -- it's relatively small for 6 projects, and it all looks reasonable to me.
This moves
case ...:labels that were previously immediately outside of the switch block tocase ...: break;statements at the top of the switch.This avoids syntax errors (and is easier to read). It fixes the issue that camthesaxman brought up on Discord about
func_80048084.This might not be the absolute cleanest way of implementing this: it's a bit odd to remove labels from
case_nodes[...]. But it doesn't break--gotos-onlywhereadd_labels_for_switch(...)is used withoutbuild_switch_statement(...).Diffs in PPC projects -- it's relatively small for 6 projects, and it all looks reasonable to me.