wcmUSB: change MOD_BUTTONS macro into a function

Values in, values out. No magic state structs, keep it simple.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Ping Cheng <pinglinux@gmail.com>

1 files changed, 16 insertions, 0 deletions

diff --git a/test/wacom-tests.c b/test/wacom-tests.c
index 1eec90e..f7b0fcc 100644
--- a/test/wacom-tests.c
+++ b/test/wacom-tests.c
@@ -428,6 +428,21 @@ test_tilt_to_rotation(void)
 }
 
 
+static void
+test_mod_buttons(void)
+{
+	int i;
+	for (i = 0; i < sizeof(int); i++)
+	{
+		int buttons = mod_buttons(0, i, 1);
+		assert(buttons == (1 << i));
+		buttons = mod_buttons(0, i, 0);
+		assert(buttons == 0);
+	}
+
+	assert(mod_buttons(0, sizeof(int), 1) == 0);
+}
+
 int main(int argc, char** argv)
 {
 	test_common_ref();
@@ -436,6 +451,7 @@ int main(int argc, char** argv)
 	test_suppress();
 	test_initial_size();
 	test_tilt_to_rotation();
+	test_mod_buttons();
 	return 0;
 }