Do not test for equality when checking if ctrl is pressed.

dyharlan · dyharlan · commit 7ba7bff3d51b · 2024-07-11T10:38:57.000+08:00
The initial change I committed worked fine before.
diff --git a/core/hostevents_sdl.cpp b/core/hostevents_sdl.cpp
@@ -56,7 +56,7 @@ void EventManager::poll_events()
         case SDL_KEYUP: {
                 // Internal shortcuts to trigger mouse grab, intentionally not
                 // sent to the host.
-                if (event.key.keysym.sym == SDLK_g && ((SDL_GetModState() & KMOD_LCTRL) == KMOD_CTRL)) {
+                if (event.key.keysym.sym == SDLK_g && SDL_GetModState() & KMOD_LCTRL) {
                     if (event.type == SDL_KEYUP) {
                         toggle_mouse_grab(event.key);
                     }

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ void EventManager::poll_events()`
`56`	`56`	`case SDL_KEYUP: {`
`57`	`57`	`// Internal shortcuts to trigger mouse grab, intentionally not`
`58`	`58`	`// sent to the host.`
`59`		`- if (event.key.keysym.sym == SDLK_g && ((SDL_GetModState() & KMOD_LCTRL) == KMOD_CTRL)) {`
	`59`	`+ if (event.key.keysym.sym == SDLK_g && SDL_GetModState() & KMOD_LCTRL) {`
`60`	`60`	`if (event.type == SDL_KEYUP) {`
`61`	`61`	`toggle_mouse_grab(event.key);`
`62`	`62`	`}`