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

[PATCH 1/4] string-list: allow passing NULL for `get_entry_index`

From
shejialuo <shejialuo@gmail.com>
Date
Sep 7, 2025, 16:40 UTC
Message-ID
<aL21eowK8MwmDX11@ArchLinux>

Callers of `get_entry_index()` are required to pass a non-NULL `exact_match` parameter to receive information about whether an exact match is found. However, in some cases, callers only need the index position.

Let's allow callers to pass NULL for the `exact_match` parameter when they don't need this information, reducing unnecessary variable declarations in calling code.

Signed-off-by: shejialuo <shejialuo@gmail.com>
---
 string-list.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/string-list.c b/string-list.c
index 343cf1ca90..bf358d1a5c 100644
--- a/string-list.c
+++ b/string-list.c
@@ -29,12 +29,14 @@ static size_t get_entry_index(const struct string_list *list, const char *string
 		else if (compare > 0)
 			left = middle + 1;
 		else {
-			*exact_match = 1;
+			if (exact_match)
+				*exact_match = 1;
 			return middle;
 		}
 	}
 
-	*exact_match = 0;
+	if (exact_match)
+		*exact_match = 0;
 	return right;
 }
 
-- 
2.51.0
Next: Junio C Hamano
Message 1 of 6 in “string-list: allow passing NULL for `get_entry_index`”
  1. 1/4 string-list: allow passing NULL for `get_entry_index`shejialuo, Sep 7, 2025
  2. Junio C HamanoSep 8, 2025
  3. Ben KnobleSep 10, 2025
  4. Junio C HamanoSep 10, 2025
  5. Ben KnobleSep 10, 2025
  6. shejialuoSep 15, 2025

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.