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

Its not really needed anymore. The whole *Default variable concept is
at least partially related to historically the PAD device was Init'ed
before the TOUCH device on Bamboo's but that is no longer true.

And since the TOUCH device is now Init'ed before PAD, when PAD
got Init'ed, wcmTapTimeDefault would override Touch's TapTime value;
probably not what user wanted.

For times *Default was really useful, I think wcmPostInitParseOption()
is probably a better replacement concept.  I'll slowly remove these
*Defaults as I have time to test each behavior.

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

diff --git a/src/wcmCommon.c b/src/wcmCommon.c
index f1ef677..e85c6d8 100644
--- a/src/wcmCommon.c
+++ b/src/wcmCommon.c
@@ -1397,7 +1397,6 @@ WacomCommonPtr wcmNewCommon(void)
        common->wcmTPCButton = 0;          /* set Tablet PC button on/off */
        common->wcmGestureParameters.wcmScrollDirection = 0;
        common->wcmGestureParameters.wcmTapTime = 250;
-       common->wcmGestureParameters.wcmTapTimeDefault = 250;
        common->wcmRotate = ROTATE_NONE;   /* default tablet rotation to off */
        common->wcmMaxX = 0;               /* max digitizer logical X value */
        common->wcmMaxY = 0;               /* max digitizer logical Y value */
diff --git a/src/wcmValidateDevice.c b/src/wcmValidateDevice.c
index d2cf636..538db0a 100644
--- a/src/wcmValidateDevice.c
+++ b/src/wcmValidateDevice.c
@@ -900,7 +900,7 @@ Bool wcmPreInitParseOptions(InputInfoPtr pInfo, Bool 
is_primary,
 
                common->wcmGestureParameters.wcmTapTime =
                        xf86SetIntOption(pInfo->options, "TapTime",
-                       common->wcmGestureParameters.wcmTapTimeDefault);
+                       common->wcmGestureParameters.wcmTapTime);
        }
 
        /* Swap stylus buttons 2 and 3 for Tablet PCs */
diff --git a/src/xf86WacomDefs.h b/src/xf86WacomDefs.h
index 417fe1d..1cdfa1a 100644
--- a/src/xf86WacomDefs.h
+++ b/src/xf86WacomDefs.h
@@ -393,7 +393,6 @@ typedef struct {
        int wcmMaxScrollFingerSpread; /* maximum distance between fingers for 
scroll gesture */
        int wcmGestureUsed;            /* retain used gesture count within one 
in-prox event */
        int wcmTapTime;                /* minimum time between taps for a right 
click */
-       int wcmTapTimeDefault;         /* default minimum time between taps for 
a right click */
 } WacomGesturesParameters;
 
 enum WacomProtocol {
-- 
1.7.7.3


------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure 
contains a definitive record of customers, application performance, 
security threats, fraudulent activity, and more. Splunk takes this 
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to