Re: [PATCH v2 0/7] making log --first-parent imply -m

Junio C Hamano [off-list ref] writes:

Sergey Organov [off-list ref] writes:

quoted

Jeff King [off-list ref] writes:

quoted

quoted

+	} else if ((argcount = parse_long_opt("diff-merges", argv, &optarg))) {
 		revs->ignore_merges = 0;
+		if (!strcmp(optarg, "off")) {
+			revs->ignore_merges = 1;
+		} else if (!strcmp(optarg, "all")) {
+			revs->diff = 0;

Should this be revs->ignore_merges = 0?

It's 4 lines above, as it's in fact common for all the cases but the
first one.

I may be mistaken, but I thought Peff was asking about turning
revs->diff off.

No, but this one was in his follow-up that I already answered a few
minutes ago.

I somehow thought that the equivalence planned for
the short term is:

            (new)               (peff's)         (master)
	diff-merges=none == --no-diff-merges == ! -m
	diff-merges=all  == --diff-merges    == -m

The second one is somewhat problematic, so I excluded it for now (see
aforementioned answer for more discussion).

Thanks,
-- Sergey