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

[PATCH 05/28] builtin-grep.c cleanup

From
DRDavid Rientjes <rientjes@google.com>
Date
Aug 14, 2006, 20:22 UTC
Message-ID
<Pine.LNX.4.63.0608141321250.19383@chino.corp.google.com>
Removes conditional return.
		David
Signed-off-by: David Rientjes <rientjes@google.com>
---
 builtin-grep.c |    4 +---
 1 files changed, 1 insertions(+), 3 deletions(-)
diff --git a/builtin-grep.c b/builtin-grep.c
index a561612..3ec99b7 100644
--- a/builtin-grep.c
+++ b/builtin-grep.c
@@ -390,9 +390,7 @@ static int buffer_is_binary(const char *
 {
 	if (FIRST_FEW_BYTES < size)
 		size = FIRST_FEW_BYTES;
-	if (memchr(ptr, 0, size))
-		return 1;
-	return 0;
+	return !!memchr(ptr, 0, size);
 }
 
 static int fixmatch(const char *pattern, char *line, regmatch_t *match)
-- 
1.4.2.g89bb-dirty
Message 1 of 1 in “builtin-grep.c cleanup”
  1. 05/28 builtin-grep.c cleanupDavid Rientjes, Aug 14, 2006

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.