At least one ! too many. Just remove the check, overwriting a value with the
already existing  value in a non-time-critical path is not the end of the
world.

Signed-off-by: Peter Hutterer <peter.hutte...@who-t.net>
---
 src/wcmXCommand.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/wcmXCommand.c b/src/wcmXCommand.c
index 7c9d7de..014fd17 100644
--- a/src/wcmXCommand.c
+++ b/src/wcmXCommand.c
@@ -770,7 +770,7 @@ int wcmSetProperty(DeviceIntPtr dev, Atom property, 
XIPropertyValuePtr prop,
                if (!IsStylus(priv))
                        return BadMatch;
 
-               if (!checkonly && common->wcmTPCButton != !values[0])
+               if (!checkonly)
                        common->wcmTPCButton = values[0];
 #ifdef DEBUG
        } else if (property == prop_debuglevels)
-- 
1.7.3.4


------------------------------------------------------------------------------
Gaining the trust of online customers is vital for the success of any company
that requires sensitive data to be transmitted over the Web.   Learn how to 
best implement a security strategy that keeps consumers' information secure 
and instills the confidence they need to proceed with transactions.
http://p.sf.net/sfu/oracle-sfdevnl 
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to