diff mbox series

[1/1] log: ensure diffs are omitted with --oneline

Message ID 20240313090841.36717-2-james@jamesliu.io (mailing list archive)
State New
Headers show
Series log: make the --oneline option work with -L | expand

Commit Message

James Liu March 13, 2024, 9:08 a.m. UTC
Currently when `git log --oneline` is invoked with the -L flag, the
full diffs are displayed along with the single line commit message.
It appears that within the log_tree_commit() function, the special
case that executes when opt->line_level_traverse is enabled (as in
the case when -L is provided) doesn't respect the --oneline option.

Ensure the --oneline option is respected by explicitly setting
DIFF_FORMAT_NO_OUTPUT if required.

Signed-off-by: James Liu <james@jamesliu.io>
---
 builtin/log.c | 4 ++++
 1 file changed, 4 insertions(+)
diff mbox series

Patch

diff --git a/builtin/log.c b/builtin/log.c
index e5da0d1043..dd77ede00c 100644
--- a/builtin/log.c
+++ b/builtin/log.c
@@ -279,6 +279,10 @@  static void cmd_log_init_finish(int argc, const char **argv, const char *prefix,
 		rev->diffopt.output_format |= DIFF_FORMAT_NO_OUTPUT;
 	argc = setup_revisions(argc, argv, rev, opt);
 
+	/* Ensure that we don't output diffs if the --oneline flag is provided */
+	if (rev->commit_format == CMIT_FMT_ONELINE)
+		rev->diffopt.output_format |= DIFF_FORMAT_NO_OUTPUT;
+
 	/* Any arguments at this point are not recognized */
 	if (argc > 1)
 		die(_("unrecognized argument: %s"), argv[1]);