And change a parameter name to make the function more self-explanatory.

Signed-off-by: Peter Hutterer <[email protected]>
---
 tools/xsetwacom.c |    9 +++++++--
 1 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/tools/xsetwacom.c b/tools/xsetwacom.c
index 7cc6847..2b6cac9 100644
--- a/tools/xsetwacom.c
+++ b/tools/xsetwacom.c
@@ -950,13 +950,18 @@ static char *convert_specialkey(const char *modifier)
        return m->converted ? m->converted : (char*)modifier;
 }
 
-static int is_modifier(const char* modifier)
+/**
+ * @param keysym An X Keysym
+ * @return nonzero if the given keysym is a modifier (as per the modifiers
+ * list) or zero otherwise.
+ */
+static int is_modifier(const char* keysym)
 {
        struct modifier *m = modifiers;
 
        while(m->name)
        {
-               if (strcmp(modifier, m->converted) == 0)
+               if (strcmp(keysym, m->converted) == 0)
                        break;
                m++;
        }
-- 
1.7.3.5


------------------------------------------------------------------------------
The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE:
Pinpoint memory and threading errors before they happen.
Find and fix more than 250 security defects in the development cycle.
Locate bottlenecks in serial and parallel code that limit performance.
http://p.sf.net/sfu/intel-dev2devfeb
_______________________________________________
Linuxwacom-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to