Stefan Beller <[email protected]> wrote:
> @@ -513,6 +531,17 @@ int send_pack(struct send_pack_args *args,
> strbuf_release(&req_buf);
> strbuf_release(&cap_buf);
>
> + if (use_push_options) {
> + struct string_list_item *item;
> + struct strbuf sb = STRBUF_INIT;
> +
> + for_each_string_list_item(item, args->push_options)
> + packet_buf_write(&sb, "%s", item->string);
> + write_or_die(out, sb.buf, sb.len);
write_or_die looks mis-indented.
> + packet_flush(out);
> + strbuf_release(&sb);
> + }
> +
> if (use_sideband && cmds_sent) {
> memset(&demux, 0, sizeof(demux));
> demux.proc = sideband_demux;
> @@ -640,6 +641,7 @@ struct transport *transport_get(struct remote *remote,
> const char *url)
> struct transport *ret = xcalloc(1, sizeof(*ret));
>
> ret->progress = isatty(2);
> + ret->push_options = NULL;
Seems unnecessary to set NULL, here, xcalloc is right above.
> if (!remote)
> die("No remote provided to transport_get()");
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html