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