U++ forum: Welcome to the forum

Re: Compiz fix [message #21004 is a reply to message #20999]

Sat, 25 April 2009 10:47

mirek
Messages: 13975
Registered: November 2005

Ultimate Member

phirox wrote on Fri, 24 April 2009 14:07

luzr wrote on Fri, 24 April 2009 10:42

R807 there was accepted related submitted patch...

The file is CtrlCore/X11Proc.cpp.

(Use svn history in theide).

Mirek

Thanks for pointing me in the right direction. My problem was fixed by removing the newly added code. Basically the following code has been put around the ev1 part:

	if(ev1->type == KeyPress)
		*ev2 = *ev1;
	else {
		...
	}

My guess is that it is an attempt at speeding up things but somehow the IsWaitingEvent() call is too important to skip.

Well, in fact, now I remember, it was not accepted patch, but my fix.

I believe the problem to fix is that some X11 systems, in case of autorepeat, put series of "KeyPress/KeyRelease" pairs, while others (and especially those with compiz enabled) only series of KeyPress.

I would like to find a reason for doubling characters before reversing the change. Maybe you could activate LLOGs (at the beginning of file) to see what is going on?

Mirek

Report message to a moderator