diff options
author | Austin Clements <amdragon@MIT.EDU> | 2012-01-19 17:29:19 -0500 |
---|---|---|
committer | David Bremner <bremner@debian.org> | 2012-01-21 08:49:50 -0400 |
commit | a9a9e374e2567caad8601d1781a3b0af8a3dde13 (patch) | |
tree | 55254879111f1f5d05a4217a825a4add463a31a9 /notmuch-new.c | |
parent | 18947b95cd1668d1b98f7ea4196e97b050599f7d (diff) |
Silence buildbot warnings about unused results
This ignores the results of the two writes in sigint handlers even
harder than before.
While my libc lacks the declarations that trigger these warnings, this
can be tested by adding the following to notmuch.h:
__attribute__((warn_unused_result))
ssize_t write(int fd, const void *buf, size_t count);
Diffstat (limited to 'notmuch-new.c')
-rw-r--r-- | notmuch-new.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/notmuch-new.c b/notmuch-new.c index 3512de72..a569a544 100644 --- a/notmuch-new.c +++ b/notmuch-new.c @@ -67,7 +67,11 @@ handle_sigint (unused (int sig)) { static char msg[] = "Stopping... \n"; - (void) write(2, msg, sizeof(msg)-1); + /* This write is "opportunistic", so it's okay to ignore the + * result. It is not required for correctness, and if it does + * fail or produce a short write, we want to get out of the signal + * handler as quickly as possible, not retry it. */ + IGNORE_RESULT (write (2, msg, sizeof(msg)-1)); interrupted = 1; } |