kernel
/
git
mirror of https://git.kernel.org/pub/scm/git/git.git
1
0
Fork 0
Code Releases Activity

upload-pack: rename `enum` to reflect the operation 

While 3145ea957d (upload-pack: introduce fetch server command,
2018-03-15) added support for the `fetch` command, from the server's
point of view it is an upload, and hence the `enum` should really be
called `upload_state` instead of `fetch_state`. Likewise, rename its
values.

This also helps unconfuse CodeQL which would otherwise be at sixes or
sevens about having _two_ non-local definitions of the same `enum` with
the same values.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
maint
Johannes Schindelin 2025-05-15 13:11:42 +00:00 committed by Junio C Hamano
parent 7f3ed75ff5
commit bf0468e2ba
1 changed files with 17 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
upload-pack.c
View File

@ -1780,16 +1780,16 @@ static void send_shallow_info(struct upload_pack_data *data)
packet_delim(1); packet_delim(1);
} }


enum fetch_state { enum upload_state {
FETCH_PROCESS_ARGS = 0, UPLOAD_PROCESS_ARGS = 0,
FETCH_SEND_ACKS, UPLOAD_SEND_ACKS,
FETCH_SEND_PACK, UPLOAD_SEND_PACK,
FETCH_DONE, UPLOAD_DONE,
}; };


int upload_pack_v2(struct repository *r, struct packet_reader *request) int upload_pack_v2(struct repository *r, struct packet_reader *request)
{ {
enum fetch_state state = FETCH_PROCESS_ARGS; enum upload_state state = UPLOAD_PROCESS_ARGS;
struct upload_pack_data data; struct upload_pack_data data;


clear_object_flags(the_repository, ALL_FLAGS); clear_object_flags(the_repository, ALL_FLAGS);
@ -1798,9 +1798,9 @@ int upload_pack_v2(struct repository *r, struct packet_reader *request)
data.use_sideband = LARGE_PACKET_MAX; data.use_sideband = LARGE_PACKET_MAX;
get_upload_pack_config(r, &data); get_upload_pack_config(r, &data);


while (state != FETCH_DONE) { while (state != UPLOAD_DONE) {
switch (state) { switch (state) {
case FETCH_PROCESS_ARGS: case UPLOAD_PROCESS_ARGS:
process_args(request, &data); process_args(request, &data);


if (!data.want_obj.nr && !data.wait_for_done) { if (!data.want_obj.nr && !data.wait_for_done) {
@ -1811,27 +1811,27 @@ int upload_pack_v2(struct repository *r, struct packet_reader *request)
* to just send 'have's without 'want's); guess * to just send 'have's without 'want's); guess
* they didn't want anything. * they didn't want anything.
*/ */
state = FETCH_DONE; state = UPLOAD_DONE;
} else if (data.seen_haves) { } else if (data.seen_haves) {
/* /*
* Request had 'have' lines, so lets ACK them. * Request had 'have' lines, so lets ACK them.
*/ */
state = FETCH_SEND_ACKS; state = UPLOAD_SEND_ACKS;
} else { } else {
/* /*
* Request had 'want's but no 'have's so we can * Request had 'want's but no 'have's so we can
* immediately go to construct and send a pack. * immediately go to construct and send a pack.
*/ */
state = FETCH_SEND_PACK; state = UPLOAD_SEND_PACK;
} }
break; break;
case FETCH_SEND_ACKS: case UPLOAD_SEND_ACKS:
if (process_haves_and_send_acks(&data)) if (process_haves_and_send_acks(&data))
state = FETCH_SEND_PACK; state = UPLOAD_SEND_PACK;
else else
state = FETCH_DONE; state = UPLOAD_DONE;
break; break;
case FETCH_SEND_PACK: case UPLOAD_SEND_PACK:
send_wanted_ref_info(&data); send_wanted_ref_info(&data);
send_shallow_info(&data); send_shallow_info(&data);


@ -1841,9 +1841,9 @@ int upload_pack_v2(struct repository *r, struct packet_reader *request)
packet_writer_write(&data.writer, "packfile\n"); packet_writer_write(&data.writer, "packfile\n");
create_pack_file(&data, NULL); create_pack_file(&data, NULL);
} }
state = FETCH_DONE; state = UPLOAD_DONE;
break; break;
case FETCH_DONE: case UPLOAD_DONE:
continue; continue;
} }
} }