diff mbox series

[1/3] log: drop "--cc implies -m" logic

Message ID 20200728163637.GA2650252@coredump.intra.peff.net
State New, archived
Headers show
Series making --first-parent imply -m | expand

Commit Message

Jeff King July 28, 2020, 4:36 p.m. UTC
This was added by 82dee4160c (log: show merge commit when --cc is given,
2015-08-20), which explains why we need it. But that commit failed to
notice that setup_revisions() already does the same thing, since
cd2bdc5309 (Common option parsing for "git log --diff" and friends,

Signed-off-by: Jeff King <peff@peff.net>
Just a cleanup that would be worth taking regardless of the rest of the

 builtin/log.c | 4 ----
 1 file changed, 4 deletions(-)
diff mbox series


diff --git a/builtin/log.c b/builtin/log.c
index d104d5c688..281d2ae8eb 100644
--- a/builtin/log.c
+++ b/builtin/log.c
@@ -731,10 +731,6 @@  static void log_setup_revisions_tweak(struct rev_info *rev,
 	/* Turn --cc/-c into -p --cc/-c when -p was not given */
 	if (!rev->diffopt.output_format && rev->combine_merges)
 		rev->diffopt.output_format = DIFF_FORMAT_PATCH;
-	/* Turn -m on when --cc/-c was given */
-	if (rev->combine_merges)
-		rev->ignore_merges = 0;
 int cmd_log(int argc, const char **argv, const char *prefix)