resolution_fixed type_patch (an actual patch, not a request for one) | by Forgon

The function 'processDebugMappings()' in './src/keymap.cpp' activates the debug mode if Player 0 is
not allocated at its invocation. This lead to two errors in the bizarre code responsible for loading games, both of which are now patched:

• skirmish savegames loaded from within skirmish games start in DEBUG mode (see './src/game.cpp')
• games launched with '--game' start in DEBUG mode. In all modes except TUTORIAL3, the player cannot move units with a left click after selecting them as in the cheat 'autogame on' (see './src/clparse.cpp')

Other changes:

• 'keyClearMappings' in './src/keymap.cpp' was a useless function and called both before and during the start of 'keyInitMappings' in several instances (see './src/init.cpp', './src/keyedit.cpp', './src/keymap.cpp')

• chat messages in campaigns and tutorials now display 'Commander' as player name (see './src/multiplay.cpp')

• no more empty line at the start of logfiles (see './lib/framework/debug.cpp')

• no more two consecutive slashes in pathnames (hopefully) (see './lib/framework/frame.cpp' and './src/levels.cpp')

• removed unused and empty functions 'StartTextInput()' and 'StopTextInput' (see './lib/qtgame/main_qt.cpp')

• fixed typos: allign -> align, allignment -> alignment, blu box -> blue box, bluboxes -> blue boxes, initalised -> initialised, initalise -> initialise, seperated -> separated, seperate -> separate, seperator -> separator

Issue migrated from trac:4612 at 2022-04-16 12:45:16 -0700