Fix various sparse warnings in the git source code

torvalds · gitster · commit 2af202be3d2f · 2009-06-20T21:52:55.000-07:00
There are a few remaining ones, but this fixes the trivial ones. It boils
down to two main issues that sparse complains about:

 - warning: Using plain integer as NULL pointer

   Sparse doesn't like you using '0' instead of 'NULL'. For various good
   reasons, not the least of which is just the visual confusion. A NULL
   pointer is not an integer, and that whole "0 works as NULL" is a
   historical accident and not very pretty.

   A few of these remain: zlib is a total mess, and Z_NULL is just a 0.
   I didn't touch those.

 - warning: symbol 'xyz' was not declared. Should it be static?

   Sparse wants to see declarations for any functions you export. A lack
   of a declaration tends to mean that you should either add one, or you
   should mark the function 'static' to show that it's in file scope.

   A few of these remain: I only did the ones that should obviously just
   be made static.

That 'wt_status_submodule_summary' one is debatable. It has a few related
flags (like 'wt_status_use_color') which _are_ declared, and are used by
builtin-commit.c. So maybe we'd like to export it at some point, but it's
not declared now, and not used outside of that file, so 'static' it is in
this patch.

Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/bisect.c b/bisect.c
@@ -454,7 +454,7 @@ static int read_bisect_refs(void)
 	return for_each_ref_in("refs/bisect/", register_ref, NULL);
 }
 
-void read_bisect_paths(struct argv_array *array)
+static void read_bisect_paths(struct argv_array *array)
 {
 	struct strbuf str = STRBUF_INIT;
 	const char *filename = git_path("BISECT_NAMES");
@@ -780,7 +780,7 @@ static void handle_bad_merge_base(void)
 	exit(1);
 }
 
-void handle_skipped_merge_base(const unsigned char *mb)
+static void handle_skipped_merge_base(const unsigned char *mb)
 {
 	char *mb_hex = sha1_to_hex(mb);
 	char *bad_hex = sha1_to_hex(current_bad_sha1);
diff --git a/builtin-add.c b/builtin-add.c
@@ -189,7 +189,7 @@ int interactive_add(int argc, const char **argv, const char *prefix)
 	return status;
 }
 
-int edit_patch(int argc, const char **argv, const char *prefix)
+static int edit_patch(int argc, const char **argv, const char *prefix)
 {
 	char *file = xstrdup(git_path("ADD_EDIT.patch"));
 	const char *apply_argv[] = { "apply", "--recount", "--cached",
diff --git a/builtin-apply.c b/builtin-apply.c
@@ -2614,7 +2614,7 @@ static int get_current_sha1(const char *path, unsigned char *sha1)
 static void build_fake_ancestor(struct patch *list, const char *filename)
 {
 	struct patch *patch;
-	struct index_state result = { 0 };
+	struct index_state result = { NULL };
 	int fd;
 
 	/* Once we start supporting the reverse patch, it may be
diff --git a/builtin-clone.c b/builtin-clone.c
@@ -141,7 +141,7 @@ static char *guess_dir_name(const char *repo, int is_bundle, int is_bare)
 	if (is_bare) {
 		struct strbuf result = STRBUF_INIT;
 		strbuf_addf(&result, "%.*s.git", (int)(end - start), start);
-		dir = strbuf_detach(&result, 0);
+		dir = strbuf_detach(&result, NULL);
 	} else
 		dir = xstrndup(start, end - start);
 	/*
diff --git a/builtin-fsck.c b/builtin-fsck.c
@@ -104,7 +104,7 @@ static int mark_object(struct object *obj, int type, void *data)
 
 static void mark_object_reachable(struct object *obj)
 {
-	mark_object(obj, OBJ_ANY, 0);
+	mark_object(obj, OBJ_ANY, NULL);
 }
 
 static int traverse_one_object(struct object *obj, struct object *parent)
@@ -292,7 +292,7 @@ static int fsck_sha1(const unsigned char *sha1)
 		fprintf(stderr, "Checking %s %s\n",
 			typename(obj->type), sha1_to_hex(obj->sha1));
 
-	if (fsck_walk(obj, mark_used, 0))
+	if (fsck_walk(obj, mark_used, NULL))
 		objerror(obj, "broken links");
 	if (fsck_object(obj, check_strict, fsck_error_func))
 		return -1;
diff --git a/builtin-help.c b/builtin-help.c
@@ -394,7 +394,7 @@ static void get_html_page_path(struct strbuf *page_path, const char *page)
  * HTML.
  */
 #ifndef open_html
-void open_html(const char *path)
+static void open_html(const char *path)
 {
 	execl_git_cmd("web--browse", "-c", "help.browser", path, NULL);
 }
diff --git a/builtin-log.c b/builtin-log.c
@@ -94,7 +94,7 @@ static void show_early_header(struct rev_info *rev, const char *stage, int nr)
 	printf("Final output: %d %s\n", nr, stage);
 }
 
-struct itimerval early_output_timer;
+static struct itimerval early_output_timer;
 
 static void log_show_early(struct rev_info *revs, struct commit_list *list)
 {
@@ -977,7 +977,7 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix)
 		strbuf_addch(&buf, '\n');
 	}
 
-	rev.extra_headers = strbuf_detach(&buf, 0);
+	rev.extra_headers = strbuf_detach(&buf, NULL);
 
 	if (start_number < 0)
 		start_number = 1;
diff --git a/builtin-merge.c b/builtin-merge.c
@@ -370,7 +370,7 @@ static void merge_name(const char *remote, struct strbuf *msg)
 
 	strbuf_addstr(&buf, "refs/heads/");
 	strbuf_addstr(&buf, remote);
-	resolve_ref(buf.buf, branch_head, 0, 0);
+	resolve_ref(buf.buf, branch_head, 0, NULL);
 
 	if (!hashcmp(remote_head->sha1, branch_head)) {
 		strbuf_addf(msg, "%s\t\tbranch '%s' of .\n",
@@ -409,7 +409,7 @@ static void merge_name(const char *remote, struct strbuf *msg)
 		strbuf_addstr(&truname, "refs/heads/");
 		strbuf_addstr(&truname, remote);
 		strbuf_setlen(&truname, truname.len - len);
-		if (resolve_ref(truname.buf, buf_sha, 0, 0)) {
+		if (resolve_ref(truname.buf, buf_sha, 0, NULL)) {
 			strbuf_addf(msg,
 				    "%s\t\tbranch '%s'%s of .\n",
 				    sha1_to_hex(remote_head->sha1),
diff --git a/builtin-remote.c b/builtin-remote.c
@@ -740,7 +740,7 @@ static int rm(int argc, const char **argv)
 	return result;
 }
 
-void clear_push_info(void *util, const char *string)
+static void clear_push_info(void *util, const char *string)
 {
 	struct push_info *info = util;
 	free(info->dest);
@@ -815,7 +815,7 @@ struct show_info {
 	int any_rebase;
 };
 
-int add_remote_to_show_info(struct string_list_item *item, void *cb_data)
+static int add_remote_to_show_info(struct string_list_item *item, void *cb_data)
 {
 	struct show_info *info = cb_data;
 	int n = strlen(item->string);
@@ -825,7 +825,7 @@ int add_remote_to_show_info(struct string_list_item *item, void *cb_data)
 	return 0;
 }
 
-int show_remote_info_item(struct string_list_item *item, void *cb_data)
+static int show_remote_info_item(struct string_list_item *item, void *cb_data)
 {
 	struct show_info *info = cb_data;
 	struct ref_states *states = info->states;
@@ -852,7 +852,7 @@ int show_remote_info_item(struct string_list_item *item, void *cb_data)
 	return 0;
 }
 
-int add_local_to_show_info(struct string_list_item *branch_item, void *cb_data)
+static int add_local_to_show_info(struct string_list_item *branch_item, void *cb_data)
 {
 	struct show_info *show_info = cb_data;
 	struct ref_states *states = show_info->states;
@@ -874,7 +874,7 @@ int add_local_to_show_info(struct string_list_item *branch_item, void *cb_data)
 	return 0;
 }
 
-int show_local_info_item(struct string_list_item *item, void *cb_data)
+static int show_local_info_item(struct string_list_item *item, void *cb_data)
 {
 	struct show_info *show_info = cb_data;
 	struct branch_info *branch_info = item->util;
@@ -906,7 +906,7 @@ int show_local_info_item(struct string_list_item *item, void *cb_data)
 	return 0;
 }
 
-int add_push_to_show_info(struct string_list_item *push_item, void *cb_data)
+static int add_push_to_show_info(struct string_list_item *push_item, void *cb_data)
 {
 	struct show_info *show_info = cb_data;
 	struct push_info *push_info = push_item->util;
@@ -935,7 +935,7 @@ static int cmp_string_with_push(const void *va, const void *vb)
 	return cmp ? cmp : strcmp(a_push->dest, b_push->dest);
 }
 
-int show_push_info_item(struct string_list_item *item, void *cb_data)
+static int show_push_info_item(struct string_list_item *item, void *cb_data)
 {
 	struct show_info *show_info = cb_data;
 	struct push_info *push_info = item->util;
@@ -1197,7 +1197,7 @@ static int get_one_remote_for_update(struct remote *remote, void *priv)
 	return 0;
 }
 
-struct remote_group {
+static struct remote_group {
 	const char *name;
 	struct string_list *list;
 } remote_group;
diff --git a/builtin-unpack-objects.c b/builtin-unpack-objects.c
@@ -158,7 +158,7 @@ struct obj_info {
 #define FLAG_WRITTEN (1u<<21)
 
 static struct obj_info *obj_list;
-unsigned nr_objects;
+static unsigned nr_objects;
 
 /*
  * Called only from check_object() after it verified this object
@@ -200,7 +200,7 @@ static int check_object(struct object *obj, int type, void *data)
 
 	if (fsck_object(obj, 1, fsck_error_function))
 		die("Error in object");
-	if (!fsck_walk(obj, check_object, 0))
+	if (!fsck_walk(obj, check_object, NULL))
 		die("Error on reachable objects of %s", sha1_to_hex(obj->sha1));
 	write_cached_object(obj);
 	return 1;
@@ -210,7 +210,7 @@ static void write_rest(void)
 {
 	unsigned i;
 	for (i = 0; i < nr_objects; i++)
-		check_object(obj_list[i].obj, OBJ_ANY, 0);
+		check_object(obj_list[i].obj, OBJ_ANY, NULL);
 }
 
 static void added_object(unsigned nr, enum object_type type,
diff --git a/connect.c b/connect.c
@@ -464,7 +464,7 @@ static void git_proxy_connect(int fd[2], char *host)
 
 #define MAX_CMD_LEN 1024
 
-char *get_port(char *host)
+static char *get_port(char *host)
 {
 	char *end;
 	char *p = strchr(host, ':');
diff --git a/daemon.c b/daemon.c
@@ -453,7 +453,7 @@ static void parse_host_arg(char *extra_args, int buflen)
 		memset(&hints, 0, sizeof(hints));
 		hints.ai_flags = AI_CANONNAME;
 
-		gai = getaddrinfo(hostname, 0, &hints, &ai);
+		gai = getaddrinfo(hostname, NULL, &hints, &ai);
 		if (!gai) {
 			struct sockaddr_in *sin_addr = (void *)ai->ai_addr;
 
diff --git a/http-push.c b/http-push.c
@@ -1750,7 +1750,7 @@ static int delete_remote_branch(char *pattern, int force)
 	return 0;
 }
 
-void run_request_queue(void)
+static void run_request_queue(void)
 {
 #ifdef USE_CURL_MULTI
 	is_running_queue = 1;
diff --git a/imap-send.c b/imap-send.c
@@ -238,7 +238,7 @@ static const char *Flags[] = {
 #ifndef NO_OPENSSL
 static void ssl_socket_perror(const char *func)
 {
-	fprintf(stderr, "%s: %s\n", func, ERR_error_string(ERR_get_error(), 0));
+	fprintf(stderr, "%s: %s\n", func, ERR_error_string(ERR_get_error(), NULL));
 }
 #endif
 
diff --git a/index-pack.c b/index-pack.c
@@ -469,7 +469,7 @@ static void sha1_object(const void *data, unsigned long size,
 				die("invalid %s", typename(type));
 			if (fsck_object(obj, 1, fsck_error_function))
 				die("Error in object");
-			if (fsck_walk(obj, mark_link, 0))
+			if (fsck_walk(obj, mark_link, NULL))
 				die("Not all child objects of %s are reachable", sha1_to_hex(obj->sha1));
 
 			if (obj->type == OBJ_TREE) {
diff --git a/mailmap.c b/mailmap.c
@@ -103,7 +103,7 @@ static char *parse_name_and_email(char *buffer, char **name,
 		char **email, int allow_empty_email)
 {
 	char *left, *right, *nstart, *nend;
-	*name = *email = 0;
+	*name = *email = NULL;
 
 	if ((left = strchr(buffer, '<')) == NULL)
 		return NULL;
@@ -136,7 +136,7 @@ static int read_single_mailmap(struct string_list *map, const char *filename, ch
 	if (f == NULL)
 		return 1;
 	while (fgets(buffer, sizeof(buffer), f) != NULL) {
-		char *name1 = 0, *email1 = 0, *name2 = 0, *email2 = 0;
+		char *name1 = NULL, *email1 = NULL, *name2 = NULL, *email2 = NULL;
 		if (buffer[0] == '#') {
 			static const char abbrev[] = "# repo-abbrev:";
 			int abblen = sizeof(abbrev) - 1;
@@ -200,7 +200,7 @@ int map_user(struct string_list *map,
 	if (!p) {
 		/* email passed in might not be wrapped in <>, but end with a \0 */
 		p = memchr(email, '\0', maxlen_email);
-		if (p == 0)
+		if (!p)
 			return 0;
 	}
 	if (p - email + 1 < sizeof(buf))
diff --git a/merge-recursive.c b/merge-recursive.c
@@ -38,7 +38,7 @@ static struct tree *shift_tree_object(struct tree *one, struct tree *two)
  * A virtual commit has (const char *)commit->util set to the name.
  */
 
-struct commit *make_virtual_commit(struct tree *tree, const char *comment)
+static struct commit *make_virtual_commit(struct tree *tree, const char *comment)
 {
 	struct commit *commit = xcalloc(1, sizeof(struct commit));
 	commit->tree = tree;
diff --git a/parse-options.c b/parse-options.c
@@ -478,7 +478,7 @@ static int usage_argh(const struct option *opts)
 #define USAGE_OPTS_WIDTH 24
 #define USAGE_GAP         2
 
-int usage_with_options_internal(const char * const *usagestr,
+static int usage_with_options_internal(const char * const *usagestr,
 				const struct option *opts, int full)
 {
 	if (!usagestr)
diff --git a/quote.c b/quote.c
@@ -272,8 +272,8 @@ void write_name_quoted(const char *name, FILE *fp, int terminator)
 	fputc(terminator, fp);
 }
 
-extern void write_name_quotedpfx(const char *pfx, size_t pfxlen,
-                                 const char *name, FILE *fp, int terminator)
+void write_name_quotedpfx(const char *pfx, size_t pfxlen,
+			  const char *name, FILE *fp, int terminator)
 {
 	int needquote = 0;
 
diff --git a/remote.c b/remote.c
@@ -307,7 +307,7 @@ static void read_branches_file(struct remote *remote)
 		strbuf_addstr(&branch, "HEAD:");
 	}
 	add_url_alias(remote, p);
-	add_fetch_refspec(remote, strbuf_detach(&branch, 0));
+	add_fetch_refspec(remote, strbuf_detach(&branch, NULL));
 	/*
 	 * Cogito compatible push: push current HEAD to remote #branch
 	 * (master if missing)
@@ -318,7 +318,7 @@ static void read_branches_file(struct remote *remote)
 		strbuf_addf(&branch, ":refs/heads/%s", frag);
 	else
 		strbuf_addstr(&branch, ":refs/heads/master");
-	add_push_refspec(remote, strbuf_detach(&branch, 0));
+	add_push_refspec(remote, strbuf_detach(&branch, NULL));
 	remote->fetch_tags = 1; /* always auto-follow */
 }
 
@@ -1119,7 +1119,7 @@ int match_refs(struct ref *src, struct ref **dst,
 	int send_all = flags & MATCH_REFS_ALL;
 	int send_mirror = flags & MATCH_REFS_MIRROR;
 	int errs;
-	static const char *default_refspec[] = { ":", 0 };
+	static const char *default_refspec[] = { ":", NULL };
 	struct ref **dst_tail = tail_ref(dst);
 
 	if (!nr_refspec) {
diff --git a/test-parse-options.c b/test-parse-options.c
@@ -9,7 +9,7 @@ static int verbose = 0, dry_run = 0, quiet = 0;
 static char *string = NULL;
 static char *file = NULL;
 
-int length_callback(const struct option *opt, const char *arg, int unset)
+static int length_callback(const struct option *opt, const char *arg, int unset)
 {
 	printf("Callback: \"%s\", %d\n",
 		(arg ? arg : "not set"), unset);
@@ -20,7 +20,7 @@ int length_callback(const struct option *opt, const char *arg, int unset)
 	return 0;
 }
 
-int number_callback(const struct option *opt, const char *arg, int unset)
+static int number_callback(const struct option *opt, const char *arg, int unset)
 {
 	*(int *)opt->value = strtol(arg, NULL, 10);
 	return 0;
diff --git a/unpack-trees.c b/unpack-trees.c
@@ -140,7 +140,7 @@ static int unpack_index_entry(struct cache_entry *ce, struct unpack_trees_option
 	return call_unpack_fn(src, o);
 }
 
-int traverse_trees_recursive(int n, unsigned long dirmask, unsigned long df_conflicts, struct name_entry *names, struct traverse_info *info)
+static int traverse_trees_recursive(int n, unsigned long dirmask, unsigned long df_conflicts, struct name_entry *names, struct traverse_info *info)
 {
 	int i;
 	struct tree_desc t[MAX_UNPACK_TREES];
diff --git a/wt-status.c b/wt-status.c
@@ -13,7 +13,7 @@
 
 int wt_status_relative_paths = 1;
 int wt_status_use_color = -1;
-int wt_status_submodule_summary;
+static int wt_status_submodule_summary;
 static char wt_status_colors[][COLOR_MAXLEN] = {
 	GIT_COLOR_NORMAL, /* WT_STATUS_HEADER */
 	GIT_COLOR_GREEN,  /* WT_STATUS_UPDATED */

Original file line number	Diff line number	Diff line change
`@@ -454,7 +454,7 @@ static int read_bisect_refs(void)`
`454`	`454`	`return for_each_ref_in("refs/bisect/", register_ref, NULL);`
`455`	`455`	`}`
`456`	`456`
`457`		`-void read_bisect_paths(struct argv_array *array)`
	`457`	`+static void read_bisect_paths(struct argv_array *array)`
`458`	`458`	`{`
`459`	`459`	`struct strbuf str = STRBUF_INIT;`
`460`	`460`	`const char *filename = git_path("BISECT_NAMES");`
`@@ -780,7 +780,7 @@ static void handle_bad_merge_base(void)`
`780`	`780`	`exit(1);`
`781`	`781`	`}`
`782`	`782`
`783`		`-void handle_skipped_merge_base(const unsigned char *mb)`
	`783`	`+static void handle_skipped_merge_base(const unsigned char *mb)`
`784`	`784`	`{`
`785`	`785`	`char *mb_hex = sha1_to_hex(mb);`
`786`	`786`	`char *bad_hex = sha1_to_hex(current_bad_sha1);`
Original file line number	Diff line number	Diff line change
`@@ -189,7 +189,7 @@ int interactive_add(int argc, const char *argv, const char prefix)`
`189`	`189`	`return status;`
`190`	`190`	`}`
`191`	`191`
`192`		`-int edit_patch(int argc, const char *argv, const char prefix)`
	`192`	`+static int edit_patch(int argc, const char *argv, const char prefix)`
`193`	`193`	`{`
`194`	`194`	`char *file = xstrdup(git_path("ADD_EDIT.patch"));`
`195`	`195`	`const char *apply_argv[] = { "apply", "--recount", "--cached",`
Original file line number	Diff line number	Diff line change
`@@ -2614,7 +2614,7 @@ static int get_current_sha1(const char path, unsigned char sha1)`
`2614`	`2614`	`static void build_fake_ancestor(struct patch list, const char filename)`
`2615`	`2615`	`{`
`2616`	`2616`	`struct patch *patch;`
`2617`		`- struct index_state result = { 0 };`
	`2617`	`+ struct index_state result = { NULL };`
`2618`	`2618`	`int fd;`
`2619`	`2619`
`2620`	`2620`	`/* Once we start supporting the reverse patch, it may be`
Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ static int mark_object(struct object obj, int type, void data)`
`104`	`104`
`105`	`105`	`static void mark_object_reachable(struct object *obj)`
`106`	`106`	`{`
`107`		`- mark_object(obj, OBJ_ANY, 0);`
	`107`	`+ mark_object(obj, OBJ_ANY, NULL);`
`108`	`108`	`}`
`109`	`109`
`110`	`110`	`static int traverse_one_object(struct object obj, struct object parent)`
`@@ -292,7 +292,7 @@ static int fsck_sha1(const unsigned char *sha1)`
`292`	`292`	`fprintf(stderr, "Checking %s %s\n",`
`293`	`293`	`typename(obj->type), sha1_to_hex(obj->sha1));`
`294`	`294`
`295`		`- if (fsck_walk(obj, mark_used, 0))`
	`295`	`+ if (fsck_walk(obj, mark_used, NULL))`
`296`	`296`	`objerror(obj, "broken links");`
`297`	`297`	`if (fsck_object(obj, check_strict, fsck_error_func))`
`298`	`298`	`return -1;`
Original file line number	Diff line number	Diff line change
`@@ -394,7 +394,7 @@ static void get_html_page_path(struct strbuf page_path, const char page)`
`394`	`394`	`* HTML.`
`395`	`395`	`*/`
`396`	`396`	`#ifndef open_html`
`397`		`-void open_html(const char *path)`
	`397`	`+static void open_html(const char *path)`
`398`	`398`	`{`
`399`	`399`	`execl_git_cmd("web--browse", "-c", "help.browser", path, NULL);`
`400`	`400`	`}`
Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,7 @@ static void show_early_header(struct rev_info rev, const char stage, int nr)`
`94`	`94`	`printf("Final output: %d %s\n", nr, stage);`
`95`	`95`	`}`
`96`	`96`
`97`		`-struct itimerval early_output_timer;`
	`97`	`+static struct itimerval early_output_timer;`
`98`	`98`
`99`	`99`	`static void log_show_early(struct rev_info revs, struct commit_list list)`
`100`	`100`	`{`
`@@ -977,7 +977,7 @@ int cmd_format_patch(int argc, const char *argv, const char prefix)`
`977`	`977`	`strbuf_addch(&buf, '\n');`
`978`	`978`	`}`
`979`	`979`
`980`		`- rev.extra_headers = strbuf_detach(&buf, 0);`
	`980`	`+ rev.extra_headers = strbuf_detach(&buf, NULL);`
`981`	`981`
`982`	`982`	`if (start_number < 0)`
`983`	`983`	`start_number = 1;`
Original file line number	Diff line number	Diff line change
`@@ -740,7 +740,7 @@ static int rm(int argc, const char **argv)`
`740`	`740`	`return result;`
`741`	`741`	`}`
`742`	`742`
`743`		`-void clear_push_info(void util, const char string)`
	`743`	`+static void clear_push_info(void util, const char string)`
`744`	`744`	`{`
`745`	`745`	`struct push_info *info = util;`
`746`	`746`	`free(info->dest);`
`@@ -815,7 +815,7 @@ struct show_info {`
`815`	`815`	`int any_rebase;`
`816`	`816`	`};`
`817`	`817`
`818`		`-int add_remote_to_show_info(struct string_list_item item, void cb_data)`
	`818`	`+static int add_remote_to_show_info(struct string_list_item item, void cb_data)`
`819`	`819`	`{`
`820`	`820`	`struct show_info *info = cb_data;`
`821`	`821`	`int n = strlen(item->string);`
`@@ -825,7 +825,7 @@ int add_remote_to_show_info(struct string_list_item item, void cb_data)`
`825`	`825`	`return 0;`
`826`	`826`	`}`
`827`	`827`
`828`		`-int show_remote_info_item(struct string_list_item item, void cb_data)`
	`828`	`+static int show_remote_info_item(struct string_list_item item, void cb_data)`
`829`	`829`	`{`
`830`	`830`	`struct show_info *info = cb_data;`
`831`	`831`	`struct ref_states *states = info->states;`
`@@ -852,7 +852,7 @@ int show_remote_info_item(struct string_list_item item, void cb_data)`
`852`	`852`	`return 0;`
`853`	`853`	`}`
`854`	`854`
`855`		`-int add_local_to_show_info(struct string_list_item branch_item, void cb_data)`
	`855`	`+static int add_local_to_show_info(struct string_list_item branch_item, void cb_data)`
`856`	`856`	`{`
`857`	`857`	`struct show_info *show_info = cb_data;`
`858`	`858`	`struct ref_states *states = show_info->states;`
`@@ -874,7 +874,7 @@ int add_local_to_show_info(struct string_list_item branch_item, void cb_data)`
`874`	`874`	`return 0;`
`875`	`875`	`}`
`876`	`876`
`877`		`-int show_local_info_item(struct string_list_item item, void cb_data)`
	`877`	`+static int show_local_info_item(struct string_list_item item, void cb_data)`
`878`	`878`	`{`
`879`	`879`	`struct show_info *show_info = cb_data;`
`880`	`880`	`struct branch_info *branch_info = item->util;`
`@@ -906,7 +906,7 @@ int show_local_info_item(struct string_list_item item, void cb_data)`
`906`	`906`	`return 0;`
`907`	`907`	`}`
`908`	`908`
`909`		`-int add_push_to_show_info(struct string_list_item push_item, void cb_data)`
	`909`	`+static int add_push_to_show_info(struct string_list_item push_item, void cb_data)`
`910`	`910`	`{`
`911`	`911`	`struct show_info *show_info = cb_data;`
`912`	`912`	`struct push_info *push_info = push_item->util;`
`@@ -935,7 +935,7 @@ static int cmp_string_with_push(const void va, const void vb)`
`935`	`935`	`return cmp ? cmp : strcmp(a_push->dest, b_push->dest);`
`936`	`936`	`}`
`937`	`937`
`938`		`-int show_push_info_item(struct string_list_item item, void cb_data)`
	`938`	`+static int show_push_info_item(struct string_list_item item, void cb_data)`
`939`	`939`	`{`
`940`	`940`	`struct show_info *show_info = cb_data;`
`941`	`941`	`struct push_info *push_info = item->util;`
`@@ -1197,7 +1197,7 @@ static int get_one_remote_for_update(struct remote remote, void priv)`
`1197`	`1197`	`return 0;`
`1198`	`1198`	`}`
`1199`	`1199`
`1200`		`-struct remote_group {`
	`1200`	`+static struct remote_group {`
`1201`	`1201`	`const char *name;`
`1202`	`1202`	`struct string_list *list;`
`1203`	`1203`	`} remote_group;`