[PATCH libinput 1/3] Add a timer subsystem

2014-06-06 Thread Hans de Goede
Currently we are using DIY timers in the touchpad softbutton and tap handling code, and at least the softbutton code gets its wrong. It uses one timer-fd per touchpad to set a timeout per touch, which means that if a timeout is set for 100ms from now for touch 1, and then 50 ms later touch 2 sets a

Re: [PATCH libinput 1/3] Add a timer subsystem

2014-06-06 Thread Hans de Goede
Hi, On 06/05/2014 05:40 AM, Peter Hutterer wrote: > On Wed, Jun 04, 2014 at 05:26:34PM +0200, Hans de Goede wrote: >> Currently we are using DIY timers in the touchpad softbutton and tap handling >> code, and at least the softbutton code gets its wrong. It uses one timer-fd >> per touchpad to set

Re: [PATCH libinput 1/3] Add a timer subsystem

2014-06-06 Thread Hans de Goede
Hi, On 06/05/2014 12:18 AM, Jonas Ådahl wrote: > On Wed, Jun 04, 2014 at 05:26:34PM +0200, Hans de Goede wrote: >> Currently we are using DIY timers in the touchpad softbutton and tap handling >> code, and at least the softbutton code gets its wrong. It uses one timer-fd >> per touchpad to set a t

Re: [PATCH libinput 1/3] Add a timer subsystem

2014-06-04 Thread Peter Hutterer
On Wed, Jun 04, 2014 at 05:26:34PM +0200, Hans de Goede wrote: > Currently we are using DIY timers in the touchpad softbutton and tap handling > code, and at least the softbutton code gets its wrong. It uses one timer-fd > per touchpad to set a timeout per touch, which means that if a timeout is >

Re: [PATCH libinput 1/3] Add a timer subsystem

2014-06-04 Thread Jonas Ådahl
On Wed, Jun 04, 2014 at 05:26:34PM +0200, Hans de Goede wrote: > Currently we are using DIY timers in the touchpad softbutton and tap handling > code, and at least the softbutton code gets its wrong. It uses one timer-fd > per touchpad to set a timeout per touch, which means that if a timeout is >

[PATCH libinput 1/3] Add a timer subsystem

2014-06-04 Thread Hans de Goede
Currently we are using DIY timers in the touchpad softbutton and tap handling code, and at least the softbutton code gets its wrong. It uses one timer-fd per touchpad to set a timeout per touch, which means that if a timeout is set for 100ms from now for touch 1, and then a timeout gets set 50 ms l