| commit | cdc034a0ac64363b5d603b24ea7226cef2f429e3 | |
| author | Patrick Steinhardt <ps@pks.im> | |
| Wed, 10 May 2023 12:34:32 +0000 (10 14:34 +0200) | ||
| committer | Junio C Hamano <gitster@pobox.com> | |
| Wed, 10 May 2023 17:35:25 +0000 (10 10:35 -0700) | ||
| tree | 0483bbb82cdac2cf983a4d83ce1eb147d7073aee | treesnapshot (tar.gz zip) |
| parent | 58afbe885c678c5cc6f6f83badca159871fc2cb3 | commitdiff |
fetch: move option related variables into main function
The options of git-fetch(1) which we pass to `parse_options()` are
declared globally in `builtin/fetch.c`. This means we're forced to use
global variables for all the options, which is more likely to cause
confusion than explicitly passing state around.
Refactor the code to move the options into `cmd_fetch()`. Move variables
that were previously forced to be declared globally and which are only
used by `cmd_fetch()` into function-local scope.
Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
The options of git-fetch(1) which we pass to `parse_options()` are
declared globally in `builtin/fetch.c`. This means we're forced to use
global variables for all the options, which is more likely to cause
confusion than explicitly passing state around.
Refactor the code to move the options into `cmd_fetch()`. Move variables
that were previously forced to be declared globally and which are only
used by `cmd_fetch()` into function-local scope.
Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| builtin/fetch.c | diffblobblamehistory |