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

[PATCH 06/28] make cmd_log_walk void

From
DRDavid Rientjes <rientjes@google.com>
Date
Aug 14, 2006, 20:23 UTC
Message-ID
<Pine.LNX.4.63.0608141322440.19383@chino.corp.google.com>
Makes cmd_log_walk void.
		David
Signed-off-by: David Rientjes <rientjes@google.com>
---
 builtin-log.c |   12 +++++++-----
 1 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/builtin-log.c b/builtin-log.c
index 691cf3a..cf103ea 100644
--- a/builtin-log.c
+++ b/builtin-log.c
@@ -29,7 +29,7 @@ static void cmd_log_init(int argc, const
 		die("unrecognized argument: %s", argv[1]);
 }
 
-static int cmd_log_walk(struct rev_info *rev)
+static void cmd_log_walk(struct rev_info *rev)
 {
 	struct commit *commit;
 
@@ -41,7 +41,6 @@ static int cmd_log_walk(struct rev_info 
 		free_commit_list(commit->parents);
 		commit->parents = NULL;
 	}
-	return 0;
 }
 
 int cmd_whatchanged(int argc, const char **argv, const char *prefix)
@@ -56,7 +55,8 @@ int cmd_whatchanged(int argc, const char
 	cmd_log_init(argc, argv, prefix, &rev);
 	if (!rev.diffopt.output_format)
 		rev.diffopt.output_format = DIFF_FORMAT_RAW;
-	return cmd_log_walk(&rev);
+	cmd_log_walk(&rev);
+	return 0;
 }
 
 int cmd_show(int argc, const char **argv, const char *prefix)
@@ -73,7 +73,8 @@ int cmd_show(int argc, const char **argv
 	rev.ignore_merges = 0;
 	rev.no_walk = 1;
 	cmd_log_init(argc, argv, prefix, &rev);
-	return cmd_log_walk(&rev);
+	cmd_log_walk(&rev);
+	return 0;
 }
 
 int cmd_log(int argc, const char **argv, const char *prefix)
@@ -84,7 +85,8 @@ int cmd_log(int argc, const char **argv,
 	init_revisions(&rev, prefix);
 	rev.always_show_header = 1;
 	cmd_log_init(argc, argv, prefix, &rev);
-	return cmd_log_walk(&rev);
+	cmd_log_walk(&rev);
+	return 0;
 }
 
 static int istitlechar(char c)
-- 
1.4.2.g89bb-dirty
Message 1 of 1 in “make cmd_log_walk void”
  1. 06/28 make cmd_log_walk voidDavid 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.