From 505abd8434f41f5ed4d01ddceeaf78f37e90c368 Mon Sep 17 00:00:00 2001 From: Karel Zak Date: Mon, 23 Apr 2012 11:57:50 +0200 Subject: [PATCH] lib/pager: fix compiler warnings MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit pager.c: In function ‘start_command’: pager.c:82:7: warning: unused variable ‘err’ [-Wunused-variable] pager.c:49:25: warning: unused variable ‘fderr’ [-Wunused-variable] pager.c:49:15: warning: unused variable ‘fdout’ [-Wunused-variable] pager.c:48:25: warning: unused variable ‘need_err’ [-Wunused-variable] pager.c:48:15: warning: unused variable ‘need_out’ [-Wunused-variable] Signed-off-by: Karel Zak --- lib/pager.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/lib/pager.c b/lib/pager.c index 1bc4134cf..f02d55d4f 100644 --- a/lib/pager.c +++ b/lib/pager.c @@ -45,8 +45,8 @@ static inline void dup_devnull(int to) static int start_command(struct child_process *cmd) { - int need_in, need_out, need_err; - int fdin[2], fdout[2], fderr[2]; + int need_in; + int fdin[2]; /* * In case of errors we must keep the promise to close FDs @@ -79,12 +79,10 @@ static int start_command(struct child_process *cmd) } if (cmd->pid < 0) { - int err = errno; if (need_in) close_pair(fdin); else if (cmd->in) close(cmd->in); - return -1; }