Review: Needs Fixing I just noticed a little misinterpretation: case SDLK_KP_ENTER is actually not an empty switch case. It just falls through to case SDLK_RETURN to handle the Enter key on the keypad the same way as the Return key. A case is only redundant if it falls through to the default case. Even a case foo: break; might be useful as it prevents the default case from execution. -- https://code.launchpad.net/~widelands-dev/widelands/handle_tab/+merge/209406 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/handle_tab.
_______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
