]> git.ipfire.org Git - thirdparty/git.git/commitdiff
upload-pack: use upload_pack_data writer in receive_needs()
authorChristian Couder <christian.couder@gmail.com>
Fri, 15 May 2020 10:04:48 +0000 (12:04 +0200)
committerJunio C Hamano <gitster@pobox.com>
Mon, 18 May 2020 19:58:01 +0000 (12:58 -0700)
As we cleanup 'upload-pack.c' by using 'struct upload_pack_data'
more thoroughly, let's use the 'struct packet_writer writer'
field from 'struct upload_pack_data' in receive_needs(),
instead of a local 'struct packet_writer writer' variable.

Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
upload-pack.c

index 94bf9cd0881b9059a97def5f8ed3a2aa5cecab5f..399ec60ade25046315517230bd51fe0419ff6b61 100644 (file)
@@ -917,10 +917,8 @@ static void receive_needs(struct upload_pack_data *data,
        timestamp_t deepen_since = 0;
        int deepen_rev_list = 0;
        int deepen_relative = 0;
-       struct packet_writer writer;
 
        shallow_nr = 0;
-       packet_writer_init(&writer, 1);
        for (;;) {
                struct object *o;
                const char *features;
@@ -978,7 +976,7 @@ static void receive_needs(struct upload_pack_data *data,
 
                o = parse_object(the_repository, &oid_buf);
                if (!o) {
-                       packet_writer_error(&writer,
+                       packet_writer_error(&data->writer,
                                            "upload-pack: not our ref %s",
                                            oid_to_hex(&oid_buf));
                        die("git upload-pack: not our ref %s",
@@ -1001,7 +999,7 @@ static void receive_needs(struct upload_pack_data *data,
         * by another process that handled the initial request.
         */
        if (has_non_tip)
-               check_non_tip(&data->want_obj, &writer);
+               check_non_tip(&data->want_obj, &data->writer);
 
        if (!use_sideband && daemon_mode)
                no_progress = 1;
@@ -1009,7 +1007,7 @@ static void receive_needs(struct upload_pack_data *data,
        if (depth == 0 && !deepen_rev_list && shallows.nr == 0)
                return;
 
-       if (send_shallow_list(&writer, depth, deepen_rev_list, deepen_since,
+       if (send_shallow_list(&data->writer, depth, deepen_rev_list, deepen_since,
                              &deepen_not, deepen_relative, &shallows,
                              &data->want_obj))
                packet_flush(1);