From: Jeff King Date: Mon, 25 Sep 2017 20:27:57 +0000 (-0400) Subject: notes-merge: drop dead zero-write code X-Git-Tag: v2.15.0-rc0~19^2~5 X-Git-Url: https://git.lorimer.id.au/gitweb.git/diff_plain/a1f3515da74504db0a046759d9ac1615a1d5f4b8?ds=inline;hp=--cc notes-merge: drop dead zero-write code We call write_in_full() with a size that we know is greater than zero. The return value can never be zero, then, since write_in_full() converts such a failed write() into ENOSPC and returns -1. We can just drop this branch of the error handling entirely. Suggested-by: Jonathan Nieder Signed-off-by: Jeff King Reviewed-by: Jonathan Nieder Signed-off-by: Junio C Hamano --- a1f3515da74504db0a046759d9ac1615a1d5f4b8 diff --git a/notes-merge.c b/notes-merge.c index 597d43f65c..4352c34a6e 100644 --- a/notes-merge.c +++ b/notes-merge.c @@ -308,8 +308,6 @@ static void write_buf_to_worktree(const struct object_id *obj, if (errno == EPIPE) break; die_errno("notes-merge"); - } else if (!ret) { - die("notes-merge: disk full?"); } size -= ret; buf += ret;