diff options
| author | John Keeping <john@keeping.me.uk> | 2013-05-18 16:21:36 +0100 | 
|---|---|---|
| committer | Jason A. Donenfeld <Jason@zx2c4.com> | 2013-05-22 12:53:06 +0200 | 
| commit | 1e9f1ee64e5f8f75a361260a2165996bc137b961 (patch) | |
| tree | e8ce7148ac4ad40dc703b0c48ba7e3aa67843357 | |
| parent | html.c: die when write fails (diff) | |
| download | cgit-1e9f1ee64e5f8f75a361260a2165996bc137b961.tar.gz cgit-1e9f1ee64e5f8f75a361260a2165996bc137b961.tar.bz2 cgit-1e9f1ee64e5f8f75a361260a2165996bc137b961.zip | |
shared.c: use die_errno() where appropriate
This replaces some code that is re-implementing die_errno by just
calling the function.
Signed-off-by: John Keeping <john@keeping.me.uk>
| -rw-r--r-- | shared.c | 9 | 
1 files changed, 4 insertions, 5 deletions
| @@ -15,21 +15,21 @@ struct cgit_context ctx;  int chk_zero(int result, char *msg)  {  	if (result != 0) -		die("%s: %s", msg, strerror(errno)); +		die_errno("%s", msg);  	return result;  }  int chk_positive(int result, char *msg)  {  	if (result <= 0) -		die("%s: %s", msg, strerror(errno)); +		die_errno("%s", msg);  	return result;  }  int chk_non_negative(int result, char *msg)  {  	if (result < 0) -		die("%s: %s", msg, strerror(errno)); +		die_errno("%s", msg);  	return result;  } @@ -468,8 +468,7 @@ int cgit_open_filter(struct cgit_filter *filter)  		chk_non_negative(dup2(filter->pipe_fh[0], STDIN_FILENO),  			"Unable to use pipe as STDIN");  		execvp(filter->cmd, filter->argv); -		die("Unable to exec subprocess %s: %s (%d)", filter->cmd, -			strerror(errno), errno); +		die_errno("Unable to exec subprocess %s", filter->cmd);  	}  	close(filter->pipe_fh[0]);  	chk_non_negative(dup2(filter->pipe_fh[1], STDOUT_FILENO), | 
