Skip to content

Commit

Permalink
* Added TranslateMessage call to SDL default message pump - was causi…
Browse files Browse the repository at this point in the history
…ng problems in S2 tools mode apps that used Qt. Qt relies on the WM_CHAR message generated by TranslateMessage and keyboard input was showing up in the UI as mixed-case. (Depending on which message pump got a given message - both SDL and Qt pump messages for the entire process.) Sam will review and possibly tweak this change before propagating to public SDL, but I'm checking this version in so I can integrate into S2 and fix the issue there.
  • Loading branch information
slouken committed Jan 30, 2014
1 parent 366b172 commit 5c58bd6
Showing 1 changed file with 1 addition and 18 deletions.
19 changes: 1 addition & 18 deletions src/video/windows/SDL_windowsevents.c
Expand Up @@ -550,24 +550,6 @@ WIN_WindowProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
returnCode = 0;
break;

case WM_UNICHAR:
{
if (wParam == UNICODE_NOCHAR) {
returnCode = 1;
break;
}
}
/* no break */
case WM_CHAR:
{
char text[5];

WIN_ConvertUTF32toUTF8(wParam, text);
SDL_SendKeyboardText(text);
}
returnCode = 0;
break;

#ifdef WM_INPUTLANGCHANGE
case WM_INPUTLANGCHANGE:
{
Expand Down Expand Up @@ -866,6 +848,7 @@ WIN_PumpEvents(_THIS)
const Uint8 *keystate;
MSG msg;
while (PeekMessage(&msg, NULL, 0, 0, PM_REMOVE)) {
TranslateMessage(&msg);
DispatchMessage(&msg);
}

Expand Down

0 comments on commit 5c58bd6

Please sign in to comment.