From: Chris Bagwell <ch...@cnpbagwell.com>

Probably the original logic meant to be doing something like:

"if (ds[0].proximity && dsLast[0].proximity && ds[1].proximity)"

but it left out the ds[0].proximity part.  This had two issues:

1) We got 1 extra packet worth of movement because of delay in
going into LAG mode upon 2 finger touch.
2) It could go into LAG mode upon releasing 1st finger before
second finger.

Signed-off-by: Chris Bagwell <ch...@cnpbagwell.com>
---
 src/wcmTouchFilter.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/wcmTouchFilter.c b/src/wcmTouchFilter.c
index a4a64f6..0186799 100644
--- a/src/wcmTouchFilter.c
+++ b/src/wcmTouchFilter.c
@@ -219,7 +219,7 @@ void wcmGestureFilter(WacomDevicePtr priv, int channel)
         * prevents cursor movement.  Force to LAG mode if ever in NONE
         * mode to stop cursor movement.
         */
-       if (dsLast[0].proximity && ds[1].proximity)
+       if (ds[0].proximity && ds[1].proximity)
        {
                if (!common->wcmGestureMode)
                        common->wcmGestureMode = GESTURE_LAG_MODE;
-- 
1.7.6


------------------------------------------------------------------------------
Special Offer -- Download ArcSight Logger for FREE!
Finally, a world-class log management solution at an even better 
price-free! And you'll get a free "Love Thy Logs" t-shirt when you
download Logger. Secure your free ArcSight Logger TODAY!
http://p.sf.net/sfu/arcsisghtdev2dev
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to