summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLibravatar Junio C Hamano <gitster@pobox.com>2020-06-29 14:17:26 -0700
committerLibravatar Junio C Hamano <gitster@pobox.com>2020-06-29 14:17:26 -0700
commit610486749aca4ce9e5923baa70b072eff69a9867 (patch)
tree1f3e76a1110c6f5aa1ead924e09dc6cd63dc1dd7
parentMerge branch 'dl/diff-usage-comment-update' (diff)
parentstrbuf: remove unreferenced strbuf_write_fd method. (diff)
downloadtgif-610486749aca4ce9e5923baa70b072eff69a9867.tar.xz
Merge branch 'rs/retire-strbuf-write-fd'
A misdesigned strbuf_write_fd() function has been retired. * rs/retire-strbuf-write-fd: strbuf: remove unreferenced strbuf_write_fd method. bugreport.c: replace strbuf_write_fd with write_in_full
-rw-r--r--bugreport.c4
-rw-r--r--strbuf.c5
-rw-r--r--strbuf.h1
3 files changed, 3 insertions, 7 deletions
diff --git a/bugreport.c b/bugreport.c
index 28f4568b01..09579e268d 100644
--- a/bugreport.c
+++ b/bugreport.c
@@ -180,7 +180,9 @@ int cmd_main(int argc, const char **argv)
die(_("couldn't create a new file at '%s'"), report_path.buf);
}
- strbuf_write_fd(&buffer, report);
+ if (write_in_full(report, buffer.buf, buffer.len) < 0)
+ die_errno(_("unable to write to %s"), report_path.buf);
+
close(report);
/*
diff --git a/strbuf.c b/strbuf.c
index 2f1a7d3209..e3397cc4c7 100644
--- a/strbuf.c
+++ b/strbuf.c
@@ -556,11 +556,6 @@ ssize_t strbuf_write(struct strbuf *sb, FILE *f)
return sb->len ? fwrite(sb->buf, 1, sb->len, f) : 0;
}
-ssize_t strbuf_write_fd(struct strbuf *sb, int fd)
-{
- return sb->len ? write(fd, sb->buf, sb->len) : 0;
-}
-
#define STRBUF_MAXLINK (2*PATH_MAX)
int strbuf_readlink(struct strbuf *sb, const char *path, size_t hint)
diff --git a/strbuf.h b/strbuf.h
index 7062eb6410..223ee2094a 100644
--- a/strbuf.h
+++ b/strbuf.h
@@ -473,7 +473,6 @@ int strbuf_readlink(struct strbuf *sb, const char *path, size_t hint);
* NUL bytes.
*/
ssize_t strbuf_write(struct strbuf *sb, FILE *stream);
-ssize_t strbuf_write_fd(struct strbuf *sb, int fd);
/**
* Read a line from a FILE *, overwriting the existing contents of