git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 6/6] do_for_each_extra_ref(): simplify signature

From
mhagger@alum.mit.edu <mhagger@alum.mit.edu>
Date
Dec 13, 2011, 20:06 UTC
Message-ID
<1323806811-5798-7-git-send-email-mhagger@alum.mit.edu>
In-Reply-To
<1323806811-5798-1-git-send-email-mhagger@alum.mit.edu>
From: Michael Haggerty <mhagger@alum.mit.edu>

Now that do_for_each_extra_ref() is only called from one place, it can be less general.

Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
---
 refs.c |   14 +++++---------
 1 files changed, 5 insertions(+), 9 deletions(-)
diff --git a/refs.c b/refs.c
index f52d8b5..4b2ba3f 100644
--- a/refs.c
+++ b/refs.c
@@ -706,19 +706,15 @@ fallback:
 	return -1;
 }
 
-static int do_for_each_extra_ref(const char *base, each_ref_fn fn,
-				 int trim, void *cb_data)
+static int do_for_each_extra_ref(each_ref_fn fn, void *cb_data)
 {
 	struct extra_ref *extra;
 
 	current_ref = NULL;
 	for (extra = extra_refs; extra; extra = extra->next) {
-		if (!prefixcmp(extra->name, base)) {
-			int retval = fn(extra->name + trim, extra->sha1,
-					0, cb_data);
-			if (retval)
-				return retval;
-		}
+		int retval = fn(extra->name, extra->sha1, 0, cb_data);
+		if (retval)
+			return retval;
 	}
 	return 0;
 }
@@ -794,7 +790,7 @@ int head_ref_submodule(const char *submodule, each_ref_fn fn, void *cb_data)
 
 int for_each_ref(each_ref_fn fn, void *cb_data)
 {
-	int retval = do_for_each_extra_ref("", fn, 0, cb_data);
+	int retval = do_for_each_extra_ref(fn, cb_data);
 	if (retval)
 		return retval;
 	return do_for_each_ref(NULL, "", fn, 0, 0, cb_data);
-- 
1.7.8
Previous: mhagger@alum.mit.edu
Message 7 of 7 in “Handle extra_refs separately from ref_caches”
  1. 0/6 Handle extra_refs separately from ref_cachesmhagger@alum.mit.edu, Dec 13, 2011
  2. 1/6 t5519: push two branches to alternate repomhagger@alum.mit.edu, Dec 13, 2011
  3. 2/6 add_extra_ref(): remove flag argumentmhagger@alum.mit.edu, Dec 13, 2011
  4. 3/6 Extract a function do_for_each_extra_ref()mhagger@alum.mit.edu, Dec 13, 2011
  5. 4/6 Store extra_refs in a separate data structuremhagger@alum.mit.edu, Dec 13, 2011
  6. 5/6 Omit extra_refs except when iterating using for_each_ref()mhagger@alum.mit.edu, Dec 13, 2011
  7. 6/6 do_for_each_extra_ref(): simplify signaturemhagger@alum.mit.edu, Dec 13, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.