[BUG] xinput bug in Xorg 1.12-rc2

2012-01-30 Thread mathieu . taillefumier
Hello, i encounter a bug in the last rc of the xserver (present before but I was using it so far) when I use it in combination with kdm (and only kdm, I did not try xdm). The server simply crash when kdm is running and I touch any of the keyboard keys but works perfectly fine when I run it from a

Re: [BUG] xinput bug in Xorg 1.12-rc2

2012-01-31 Thread Mathieu Taillefumier
Hello, thank you for the answer. On 01/31/2012 12:28 AM, Jeremy Huddleston wrote: I really don't see how 2abe83df686ed64c4f4df711ac3c1fd12131c2e4 could be the culprit. It's trivial. I made a mistake somewhere so to double check I did the bisection again starting from 1.11.99.1 (working) to 1

Re: [BUG] xinput bug in Xorg 1.12-rc2

2012-02-02 Thread Mathieu Taillefumier
rGrabbedEvents but since I do not know the code I can not give more informations except the gdb session that you already have. Mathieu Ps : I will modify the bug report with the new information. On Jan 31, 2012, at 6:05 AM, Mathieu Taillefumier wrote: Hello, thank you for the answer

Re: [BUG] xinput bug in Xorg 1.12-rc2

2012-02-03 Thread Mathieu Taillefumier
On 02/02/2012 06:43 PM, Peter Hutterer wrote: On Thu, Feb 02, 2012 at 05:00:58PM +0100, Mathieu Taillefumier wrote: On 01/31/2012 07:20 PM, Jeremy Huddleston wrote: That's pretty much an entire merge you are blaming now. Are you sure you can't narrow it down further than that?

Re: [BUG] xinput bug in Xorg 1.12-rc2

2012-02-03 Thread mathieu . taillefumier
he compiler did not seem to complain) - DeliverDeviceEvents() -> if(win) DeliverDeviceEvents I also check the function DeliverDeviceEvents and it seems that the "if(win)" is not necessary since it is a loop over the win parameter. regards Mathieu ----- Mail original ----- D

Re: [BUG] xinput bug in Xorg 1.12-rc2

2012-02-04 Thread Mathieu Taillefumier
thanks, you're right, this is a bug. I've just sent out the patch reverting the commit - since it was supposed to only clean up code but not include any functional changes, this is the simpler way than trying to fix a patch that's just cosmetic anyway. Yes but the initial idea is good and the