The loop that this executes in only evaluates press entries to begin with,
making the check here unnecessary.

V547 Expression 'press' is always true.

Signed-off-by: Jason Gerecke <jason.gere...@wacom.com>
---
 src/wcmCommon.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/wcmCommon.c b/src/wcmCommon.c
index e56045a..06cdb8d 100644
--- a/src/wcmCommon.c
+++ b/src/wcmCommon.c
@@ -249,9 +249,8 @@ static void sendAction(InputInfoPtr pInfo,  const 
WacomDeviceState* ds,
                                }
                                break;
                        case AC_MODETOGGLE:
-                               if (press)
-                                       wcmDevSwitchModeCall(pInfo,
-                                                       (is_absolute(pInfo)) ? 
Relative : Absolute); /* not a typo! */
+                               wcmDevSwitchModeCall(pInfo,
+                                               (is_absolute(pInfo)) ? Relative 
: Absolute); /* not a typo! */
                                break;
                        case AC_PANSCROLL:
                                priv->flags |= SCROLLMODE_FLAG;
-- 
2.18.0


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to