From d9f61d919cf809f7847635ec3f90122826169f11 Mon Sep 17 00:00:00 2001 From: "Devin J. Pohly" Date: Mon, 31 Dec 2012 19:31:38 -0500 Subject: [PATCH] answer some commented questions --- src/arch/InputHandler/InputHandler_X11.cpp | 2 +- src/arch/LowLevelWindow/LowLevelWindow_X11.cpp | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/arch/InputHandler/InputHandler_X11.cpp b/src/arch/InputHandler/InputHandler_X11.cpp index cec3478f2a..56f0febb74 100644 --- a/src/arch/InputHandler/InputHandler_X11.cpp +++ b/src/arch/InputHandler/InputHandler_X11.cpp @@ -201,7 +201,7 @@ void InputHandler_X11::Update() lastEvent.type = 0; } - // Why only the zero index? + // Get the first defined keysym for this event's key lastDB = XSymToDeviceButton( XLookupKeysym(&event.xkey, 0) ); if( lastDB == DeviceButton_Invalid ) diff --git a/src/arch/LowLevelWindow/LowLevelWindow_X11.cpp b/src/arch/LowLevelWindow/LowLevelWindow_X11.cpp index 6d75d3f384..c4a05df104 100644 --- a/src/arch/LowLevelWindow/LowLevelWindow_X11.cpp +++ b/src/arch/LowLevelWindow/LowLevelWindow_X11.cpp @@ -161,7 +161,8 @@ RString LowLevelWindow_X11::TryVideoMode( const VideoModeParams &p, bool &bNewDe XSelectInput( Dpy, Win, winAttrib.your_event_mask | StructureNotifyMask ); XMapWindow( Dpy, Win ); - // XXX: Why do we need to wait for the MapNotify event? + // Wait until we actually have a mapped window before trying to + // use it! while( true ) { XEvent event;