diff options
author | Martin Schwenke <martin@meltin.net> | 2011-11-10 10:46:10 +1100 |
---|---|---|
committer | Stefan Metzmacher <metze@samba.org> | 2011-11-10 10:28:44 +0100 |
commit | 538748132fbf6bcc1ce0bbd474e4abf3ecdabffa (patch) | |
tree | e8c2f42d36bdbed2431a5a67d98e9e0a56f6ba62 /source3/printing/nt_printing_migrate_internal.h | |
parent | 7d84805ac06522dabeb4f8d5b69dab2918a508ba (diff) | |
download | samba-538748132fbf6bcc1ce0bbd474e4abf3ecdabffa.tar.gz samba-538748132fbf6bcc1ce0bbd474e4abf3ecdabffa.tar.bz2 samba-538748132fbf6bcc1ce0bbd474e4abf3ecdabffa.zip |
Fix -Wunused-but-set-variable compiler warnings in tevent_signal.c
The results of some read(2) and write(2) calls are assigned into a
variable that is never used. Presumably this used to avoid compiler
warnings or similar.
However, from (approximately) GCC 4.6 this produces some warnings:
[ 609/3910] Compiling lib/tevent/tevent_signal.c
../lib/tevent/tevent_signal.c: In function ‘tevent_common_signal_handler’:
../lib/tevent/tevent_signal.c:85:10: warning: variable ‘res’ set but not used [-Wunused-but-set-variable]
../lib/tevent/tevent_signal.c: In function ‘signal_pipe_handler’:
../lib/tevent/tevent_signal.c:183:10: warning: variable ‘res’ set but not used [-Wunused-but-set-variable]
The simplest thing to do is remove the variables and cast the function
return to void. There is already a comment above each call.
Signed-off-by: Martin Schwenke <martin@meltin.net>
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Diffstat (limited to 'source3/printing/nt_printing_migrate_internal.h')
0 files changed, 0 insertions, 0 deletions