SDLBackend: Simplify shortcut consuming

Author: nullprop

src/Backends/SDLBackend.cpp

@@ -745,7 +745,7 @@ namespace gamescope
 					if ( event.type == SDL_KEYUP && ( event.key.keysym.mod & KMOD_LGUI ) )
 					{
 						auto shortcut = g_shortcutHandler.GetShortcut( key );
-						bool handled = g_shortcutHandler.HandleShortcut( shortcut );
+						g_shortcutHandler.HandleShortcut( shortcut );
 
 						switch ( shortcut )
 						{
@@ -763,7 +763,7 @@ namespace gamescope
 							default:
 								break;
 						}
-						if ( handled )
+						if ( shortcut != GAMESCOPE_SHORTCUT_NONE )
 						{
 							break;
 						}

src/shortcut.cpp

@@ -51,7 +51,7 @@ namespace gamescope
 		return GAMESCOPE_SHORTCUT_NONE;
 	}
 
-	bool ShortcutHandler::HandleShortcut( GamescopeShortcut shortcut )
+	void ShortcutHandler::HandleShortcut( GamescopeShortcut shortcut )
 	{
 		switch ( shortcut )
 		{
@@ -82,11 +82,8 @@ namespace gamescope
 				g_bGrabbed = !g_bGrabbed;
 				break;
 			default:
-				return false;
 				break;
 		}
-
-		return true;
 	}
 
 	static const char* ShortcutNames[GAMESCOPE_SHORTCUT_COUNT] = {

src/shortcut.h

@@ -34,7 +34,7 @@ namespace gamescope
 
 		GamescopeShortcut GetShortcut( uint32_t key );
 
-		bool HandleShortcut( GamescopeShortcut shortcut );
+		void HandleShortcut( GamescopeShortcut shortcut );
 
 	private:
 		std::map<uint32_t, GamescopeShortcut> m_binds;