diff options
author | Peter Hutterer <peter.hutterer@who-t.net> | 2010-05-06 15:05:35 +1000 |
---|---|---|
committer | Peter Hutterer <peter.hutterer@who-t.net> | 2010-05-10 11:20:14 +1000 |
commit | 39aed3b773c077a55f54a7c143102ab09dae8237 (patch) | |
tree | 8466b8d32517445aa2793c75901ce030e369c98b | |
parent | 3782060b41ede96d87dcbc508b2235c6723f0a78 (diff) |
When scrolling, hardcode the button/press release cycle.
Not sure why the values for !hw->up and hw->up were used here instead, but this
could potentially lead to unreleased buttons. Also, it's harder to read and
generally confusing, since the same wasn't used for left/right scrolling.
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
-rw-r--r-- | src/synaptics.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/synaptics.c b/src/synaptics.c index c66b6aa..a269e70 100644 --- a/src/synaptics.c +++ b/src/synaptics.c @@ -2262,12 +2262,12 @@ HandleState(LocalDevicePtr local, struct SynapticsHwState *hw) */ if (inside_active_area) { while (scroll.up-- > 0) { - xf86PostButtonEvent(local->dev, FALSE, 4, !hw->up, 0, 0); - xf86PostButtonEvent(local->dev, FALSE, 4, hw->up, 0, 0); + xf86PostButtonEvent(local->dev, FALSE, 4, TRUE, 0, 0); + xf86PostButtonEvent(local->dev, FALSE, 4, FALSE, 0, 0); } while (scroll.down-- > 0) { - xf86PostButtonEvent(local->dev, FALSE, 5, !hw->down, 0, 0); - xf86PostButtonEvent(local->dev, FALSE, 5, hw->down, 0, 0); + xf86PostButtonEvent(local->dev, FALSE, 5, TRUE, 0, 0); + xf86PostButtonEvent(local->dev, FALSE, 5, FALSE, 0, 0); } while (scroll.left-- > 0) { xf86PostButtonEvent(local->dev, FALSE, 6, TRUE, 0, 0); @@ -2282,8 +2282,8 @@ HandleState(LocalDevicePtr local, struct SynapticsHwState *hw) if (double_click) { int i; for (i = 0; i < 2; i++) { - xf86PostButtonEvent(local->dev, FALSE, 1, !hw->left, 0, 0); - xf86PostButtonEvent(local->dev, FALSE, 1, hw->left, 0, 0); + xf86PostButtonEvent(local->dev, FALSE, 1, TRUE, 0, 0); + xf86PostButtonEvent(local->dev, FALSE, 1, FALSE, 0, 0); } } |