From 30cdaa7ca8bfa03d90e170c2260d71a5819996b2 Mon Sep 17 00:00:00 2001 From: Ben Harris Date: Mon, 30 May 2016 22:37:32 +0100 Subject: [PATCH] unix: make select_result() return void. Nothing was using its return value anyway. --- unix/unix.h | 2 +- unix/uxsel.c | 6 ++---- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/unix/unix.h b/unix/unix.h index 7ab2ca24..68f749ac 100644 --- a/unix/unix.h +++ b/unix/unix.h @@ -183,7 +183,7 @@ void uxsel_init(void); typedef int (*uxsel_callback_fn)(int fd, int event); void uxsel_set(int fd, int rwx, uxsel_callback_fn callback); void uxsel_del(int fd); -int select_result(int fd, int event); +void select_result(int fd, int event); int first_fd(int *state, int *rwx); int next_fd(int *state, int *rwx); /* The following are expected to be provided _to_ uxsel.c by the frontend */ diff --git a/unix/uxsel.c b/unix/uxsel.c index ef25cdb5..52f346a4 100644 --- a/unix/uxsel.c +++ b/unix/uxsel.c @@ -111,7 +111,7 @@ int first_fd(int *state, int *rwx) return next_fd(state, rwx); } -int select_result(int fd, int event) +void select_result(int fd, int event) { struct fd *fdstruct = find234(fds, &fd, uxsel_fd_findcmp); /* @@ -120,7 +120,5 @@ int select_result(int fd, int event) * fd I've stopped being interested in. Sigh. */ if (fdstruct) - return fdstruct->callback(fd, event); - else - return 1; + fdstruct->callback(fd, event); }