Sonar cleanup 13 - #8810

Next round of sonar clean up:

• TCPSocket: init m_socket before m_events
• ServerConfig: use ranged loops for screens section
• IKeyState: directly send args for string creation to emplace
• IKeyState: Remove use of malloc (+ use auto now for the type)
• App: make m_bye private and access via existing method bye
• DisplayInvalidException remove unneeded constructors on non mac os
• MSWindowsDesks::getCursorPos, initialize pos to 0,0
• KeyState::updateModifierState, use std::ranges::set_difference
• KeyTypes::KeyModifierNameMapEntry, remove commented code
• AppUtilUnix remove name from unused IEventQueue pointer
• Use default == operator for KeyMap, Action, KeySequence, Fingerprint
• server/Config remove redundant != operator