Merge branch 'ds/fetch-config-parse-microfix'

gitster · gitster · commit 6c142bc8461b · 2024-04-15T14:11:43.000-07:00
A config parser callback function fell through instead of returning
after recognising and processing a variable, wasting cycles, which
has been corrected.

* ds/fetch-config-parse-microfix:
  fetch: return when parsing submodule.recurse
diff --git a/builtin/fetch.c b/builtin/fetch.c
@@ -138,6 +138,7 @@ static int git_fetch_config(const char *k, const char *v,
 		int r = git_config_bool(k, v) ?
 			RECURSE_SUBMODULES_ON : RECURSE_SUBMODULES_OFF;
 		fetch_config->recurse_submodules = r;
+		return 0;
 	}
 
 	if (!strcmp(k, "submodule.fetchjobs")) {

Original file line number	Diff line number	Diff line change
`@@ -138,6 +138,7 @@ static int git_fetch_config(const char k, const char v,`
`138`	`138`	`int r = git_config_bool(k, v) ?`
`139`	`139`	`RECURSE_SUBMODULES_ON : RECURSE_SUBMODULES_OFF;`
`140`	`140`	`fetch_config->recurse_submodules = r;`
	`141`	`+ return 0;`
`141`	`142`	`}`
`142`	`143`
`143`	`144`	`if (!strcmp(k, "submodule.fetchjobs")) {`