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

Reply via email to