Hi Duy,
On Wed, 3 May 2017, Nguyễn Thái Ngọc Duy wrote:
> diff --git a/rerere.c b/rerere.c
> index 971bfedfb2..c26c29f87a 100644
> --- a/rerere.c
> +++ b/rerere.c
> @@ -484,13 +484,14 @@ static int handle_file(const char *path, unsigned char *sha1, const char *output
> io.input = fopen(path, "r");
> io.io.wrerror = 0;
> if (!io.input)
> - return error("Could not open %s", path);
> + return error_errno("Could not open %s", path);
IMO the error() -> error_errno() changes should all be part of the same
commit, as they probably share the explanation why fopen_or_warn() is not
appropriate here.
> if (output) {
> io.io.output = fopen(output, "w");
> if (!io.io.output) {
> + error_errno("Could not write %s", output);
> fclose(io.input);
> - return error("Could not write %s", output);
> + return -1;
> }
This one is logically different from the change above, as it not only
cannot be replaced by fopen_or_warn(), but also requires the reordering
due to the different nature of the change.
Ciao,
Dscho