From 3d913526b19545eb4e463f96d2f3b8db8daae6ef Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Sat, 12 Sep 2009 10:54:32 +0200 Subject: use write_str_in_full helper to avoid literal string lengths This is the same fix to use write_str_in_full() helper to write a constant string out without counting the length of it ourselves. Signed-off-by: Jim Meyering Signed-off-by: Junio C Hamano --- transport-helper.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) (limited to 'transport-helper.c') diff --git a/transport-helper.c b/transport-helper.c index b1ea7e6f4..832d81f23 100644 --- a/transport-helper.c +++ b/transport-helper.c @@ -37,9 +37,7 @@ static struct child_process *get_helper(struct transport *transport) die("Unable to run helper: git %s", helper->argv[0]); data->helper = helper; - strbuf_addstr(&buf, "capabilities\n"); - write_in_full(helper->in, buf.buf, buf.len); - strbuf_reset(&buf); + write_str_in_full(helper->in, "capabilities\n"); file = fdopen(helper->out, "r"); while (1) { @@ -58,7 +56,7 @@ static int disconnect_helper(struct transport *transport) { struct helper_data *data = transport->data; if (data->helper) { - write_in_full(data->helper->in, "\n", 1); + write_str_in_full(data->helper->in, "\n"); close(data->helper->in); finish_command(data->helper); free((char *)data->helper->argv[0]); @@ -124,9 +122,7 @@ static struct ref *get_refs_list(struct transport *transport, int for_push) helper = get_helper(transport); - strbuf_addstr(&buf, "list\n"); - write_in_full(helper->in, buf.buf, buf.len); - strbuf_reset(&buf); + write_str_in_full(helper->in, "list\n"); file = fdopen(helper->out, "r"); while (1) { -- cgit v1.2.1