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

[PATCH 2/2] ref-filter: get rid of goto

From
Olga Telezhnaya <olyatelezhnaya@gmail.com>
Date
Feb 21, 2018, 06:59 UTC
Message-ID
<01020161b7288324-d478f224-7a37-4341-993f-3c82a9941b7f-000000@eu-west-1.amazonses.com>
In-Reply-To
<01020161b728823f-daec904e-44ff-45f9-9c89-d447b89c7802-000000@eu-west-1.amazonses.com>
Get rid of goto command in ref-filter for better readability.
Signed-off-by: Olga Telezhnaia <olyatelezhnaya@gmail.com>
Mentored-by: Christian Couder <christian.couder@gmail.com>
Mentored by: Jeff King <peff@peff.net>
---
 ref-filter.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/ref-filter.c b/ref-filter.c
index 83ffd84affe52..28df6e21fb996 100644
--- a/ref-filter.c
+++ b/ref-filter.c
@@ -1494,11 +1494,11 @@ static void populate_value(struct ref_array_item *ref)
 	for (i = 0; i < used_atom_cnt; i++) {
 		struct atom_value *v = &ref->value[i];
 		if (v->s == NULL)
-			goto need_obj;
+			break;
 	}
-	return;
+	if (used_atom_cnt <= i)
+		return;
 
- need_obj:
 	get_object(ref, &ref->objectname, 0, &obj);
 
 	/*

--
https://github.com/git/git/pull/460
Previous: Olga TelezhnayaNext: Junio C Hamano
Message 2 of 7 in “ref-filter: get rid of duplicate code”
  1. 1/2 ref-filter: get rid of duplicate codeOlga Telezhnaya, Feb 21, 2018
  2. 2/2 ref-filter: get rid of gotoOlga Telezhnaya, Feb 21, 2018
  3. Junio C HamanoFeb 21, 2018
  4. Оля ТележнаяFeb 22, 2018
  5. 1/2 ref-filter: get rid of duplicate codeOlga Telezhnaya, Feb 21, 2018
  6. 2/2 ref-filter: get rid of gotoOlga Telezhnaya, Feb 21, 2018
  7. Junio C HamanoFeb 21, 2018

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.