# HG changeset patch
# User Sam Lantinga <slouken@libsdl.org>
# Date 1076695036 0
# Node ID 428f688f2ad2687aa18f3f5e70532106460fbfef
# Parent  6176f9a0d61ad8a87be8c0f564888a2021bf65a6
Date: Fri, 13 Feb 2004 17:03:16 +0100
From: Max Horn
Subject: Modifier key fix

The internal modifier state can get out of sync with reality. To
trigger this, do for example this:
  1) Launch an SDL app
  2) Alt-click on the desktop (this will hide the SDL app).
  3) Bring the SDL app back to the front
  4) SDL will still think alt is pressed (and as such will treat left
clicks like middle clicks). If you press and release alt, it'll be fine
again.

The attached patch cures this by rechecking the modifier state whenever
we process an event.

diff -r 6176f9a0d61a -r 428f688f2ad2 src/video/quartz/SDL_QuartzEvents.m
--- a/src/video/quartz/SDL_QuartzEvents.m	Fri Feb 13 06:42:17 2004 +0000
+++ b/src/video/quartz/SDL_QuartzEvents.m	Fri Feb 13 17:57:16 2004 +0000
@@ -266,6 +266,9 @@
     int i;
     int bit;
     SDL_keysym key;
+    
+    if (current_mods == newMods)
+    	return;
 
     key.scancode    = 0;
     key.sym         = SDLK_UNKNOWN;
@@ -462,6 +465,8 @@
             type = [ event type ];
             isInGameWin = QZ_IsMouseInWindow (this);
 
+            QZ_DoModifiers(this, [ event modifierFlags ] );
+
             switch (type) {
                 case NSLeftMouseDown:
                     if ( getenv("SDL_HAS3BUTTONMOUSE") ) {
@@ -613,7 +618,6 @@
                     QZ_DoKey (this, SDL_PRESSED, event);
                     break;
                 case NSFlagsChanged:
-                    QZ_DoModifiers(this, [ event modifierFlags ] );
                     break;
                 case NSAppKitDefined:
                     switch ( [ event subtype ] ) {
diff -r 6176f9a0d61a -r 428f688f2ad2 src/video/quartz/SDL_QuartzVideo.m
--- a/src/video/quartz/SDL_QuartzVideo.m	Fri Feb 13 06:42:17 2004 +0000
+++ b/src/video/quartz/SDL_QuartzVideo.m	Fri Feb 13 17:57:16 2004 +0000
@@ -203,6 +203,7 @@
     current_grab_mode = SDL_GRAB_OFF;
     cursor_should_be_visible    = YES;
     cursor_visible              = YES;
+    current_mods = -1;
     
     /* register for sleep notifications so wake from sleep generates SDL_VIDEOEXPOSE */
     QZ_RegisterForSleepNotifications (this);