[PATCH] fix strbuf take #2
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:41:57
Subsystem:
the rest · Maintainer:
Linus Torvalds
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:41:57
Subsystem:
the rest · Maintainer:
Linus Torvalds
I just remembered why I placed that bogus "sb->len ==0 implies sb->eof" condition there. We need at least something like this to catch the normal EOF (that is, line termination immediately followed by EOF) case. "if (feof(fp))" fires when we have already read the eof, not when we are about read it. Signed-off-by: Junio C Hamano <redacted> --- git-diff-cache -r -z 0 | ./git-diff-helper -z
diff --git a/strbuf.c b/strbuf.c
--- a/strbuf.c
+++ b/strbuf.c@@ -37,6 +37,8 @@ void read_line(struct strbuf *sb, FILE * break; strbuf_add(sb, ch); } + if (ch == EOF && sb->len == 0) + sb->eof = 1; strbuf_end(sb); }