[Bf-blender-cvs] [883663a] master: Fix T44278: Tab, Z-Key and Spacebar not working

[Julian Eisel]

Commit: 883663a0ad7fd1cbb7bdc9cf4429217ef1f3b9de
Author: Julian Eisel
Date:   Mon Apr 6 13:13:35 2015 +0200
Branches: master
https://developer.blender.org/rB883663a0ad7fd1cbb7bdc9cf4429217ef1f3b9de

Fix T44278: Tab, Z-Key and Spacebar not working

Seems like a fix that is needed for some X11 systems causes this bug on
others :| Not sure if the systems that needed this fix are now still
fine (since I did a slight change to the click type check procedure),
but I need to check that on my system in the institute in a bit.

===================================================================

M	source/blender/windowmanager/intern/wm_event_system.c

===================================================================

diff --git a/source/blender/windowmanager/intern/wm_event_system.c b/source/blender/windowmanager/intern/wm_event_system.c
index 0a8da1d..225e313 100644
--- a/source/blender/windowmanager/intern/wm_event_system.c
+++ b/source/blender/windowmanager/intern/wm_event_system.c
@@ -3025,12 +3025,9 @@ static void wm_event_clicktype_init(wmWindow *win, wmEvent *event, wmEvent *even
 		}
 	}
 
+	/* click */
 	if ((PIL_check_seconds_timer() - event->click_time) * 1000 <= U.click_timeout) {
-		/* for any reason some X11 systems send two release events triggering two KM_CLICK events
-		 * - making the rules more strict by checking for prevval resolves this (not needed for mouse) */
-		if (event->val == KM_RELEASE &&
-		    (ISMOUSE(event->type) || event->prevval != KM_RELEASE))
-		{
+		if (event->val == KM_RELEASE) {
 			click_type = KM_CLICK;
 			if (G.debug & (G_DEBUG_HANDLERS | G_DEBUG_EVENTS)) {
 				printf("%s Send click event\n", __func__);