From 84335f5dd343628539ceb13f0bcec45adf2acaa5 Mon Sep 17 00:00:00 2001 From: Sam Lantinga Date: Tue, 14 Dec 2004 18:23:27 +0000 Subject: [PATCH] Date: Tue, 14 Dec 2004 12:08:30 +0100 From: Marcin Konicki Subject: Re: [SDL] SDL 1.2.8 Prerelease I'm sending small fix for BeOS, which prevents filling up SDL's message queue too fast. Without it, SDL receives "key down" messages from BeOS code, for each key repeat (BeOS handles key repeats itself, and application can check if received "key down" message from BeOS is first time key down, or if it's repeat, and which repeat it is). Since there is no way for "sdl driver" to turn off "default" SDL's key-repeat mechanism, they were working both at the same time (and queue could be filled up very fast). So this patch removes handling "key down" message from BeOS if it's key_repeat "type". --- src/video/bwindow/SDL_BWin.h | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/src/video/bwindow/SDL_BWin.h b/src/video/bwindow/SDL_BWin.h index b232534d1..567b8df66 100644 --- a/src/video/bwindow/SDL_BWin.h +++ b/src/video/bwindow/SDL_BWin.h @@ -425,11 +425,13 @@ class SDL_BWin : public BDirectWindow { /* mouse up doesn't give which button was released, only state of buttons (after release, so it's always = 0), - which is is not what we need ;] + which is not what we need ;] So we need to store button in mouse down, and restore in mouse up :( - mouse up is (similarly to mouse down) send only when - no more buttons are down */ + mouse up is (similarly to mouse down) send only for + first button down (ie. it's no send if we click another button + without releasing previous one first) - but that's probably + because of how drivers are written?, not BeOS itself. */ int32 buttons; int sdl_buttons = 0; if (msg->FindInt32("buttons", &buttons) == B_OK) { @@ -471,6 +473,11 @@ class SDL_BWin : public BDirectWindow { int32 key; int32 modifiers; + int32 key_repeat; + /* Workaround for SDL message queue being filled too fast because of BeOS own key-repeat mechanism */ + if (msg->FindInt32("be:key_repeat", &key_repeat) == B_OK && key_repeat > 0) + break; + if (msg->FindInt32("key", &key) == B_OK && msg->FindInt32("modifiers", &modifiers) == B_OK) { SDL_keysym keysym; keysym.scancode = key;