[PATCH 1/5] syndaemon: remove enable/disable_touchpad(), use toggle_touchpad instead
Peter Hutterer
peter.hutterer at who-t.net
Sun Mar 1 18:46:36 PST 2009
Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
---
tools/syndaemon.c | 55 +++++++++++++++++++++++-----------------------------
1 files changed, 24 insertions(+), 31 deletions(-)
diff --git a/tools/syndaemon.c b/tools/syndaemon.c
index ec36f11..820b36c 100644
--- a/tools/syndaemon.c
+++ b/tools/syndaemon.c
@@ -74,22 +74,32 @@ usage(void)
exit(1);
}
-static int
-enable_touchpad(void)
+/**
+ * Toggle touchpad enabled/disabled state, decided by value.
+ */
+static void
+toggle_touchpad(int value)
{
- int ret = 0;
- if (pad_disabled) {
- synshm->touchpad_off = 0;
- pad_disabled = 0;
- ret = 1;
- }
- return ret;
+ if (pad_disabled && !value)
+ {
+ if (!background)
+ printf("Enable\n");
+ } else if (!pad_disabled && value)
+ {
+ if (!background)
+ printf("Disable\n");
+ } else
+ return;
+
+ pad_disabled = value;
+ synshm->touchpad_off = value;
}
static void
signal_handler(int signum)
{
- enable_touchpad();
+ toggle_touchpad(0);
+
if (pid_file)
unlink(pid_file);
kill(getpid(), signum);
@@ -202,20 +212,9 @@ main_loop(Display *display, double idle_time, int poll_delay)
last_activity = 0.0;
if (current_time > last_activity + idle_time) { /* Enable touchpad */
- if (enable_touchpad()) {
- if (!background)
- printf("Enable\n");
- }
+ toggle_touchpad(0);
} else { /* Disable touchpad */
- if (!pad_disabled && !synshm->touchpad_off) {
- if (!background)
- printf("Disable\n");
- pad_disabled = 1;
- if (disable_taps_only)
- synshm->touchpad_off = 2;
- else
- synshm->touchpad_off = 1;
- }
+ toggle_touchpad(disable_taps_only ? 2 : 1);
}
usleep(poll_delay);
@@ -418,17 +417,11 @@ void record_main_loop(Display* display, double idle_time) {
timeout.tv_sec = (int)idle_time;
timeout.tv_usec = (idle_time-(double)timeout.tv_sec) * 1.e6;
- if (!pad_disabled) {
- pad_disabled=1;
- if (!background) printf("disable touchpad\n");
-
- if (!synshm->touchpad_off)
- synshm->touchpad_off = disable_taps_only ? 2 : 1;
- }
+ toggle_touchpad(disable_taps_only ? 2 : 1);
}
if (ret == 0 && pad_disabled) { /* timeout => enable event */
- enable_touchpad();
+ toggle_touchpad(0);
if (!background) printf("enable touchpad\n");
}
--
1.6.0.6
More information about the xorg-devel
mailing list