log: show ^ for boundary

Boundary commits are commits which are discarded by the revision walk,
i.e. they are no "positive part" of the specified revision set but show
nevertheless (because of `--boundary`).

So far, `log` shows them with a `-` prefix and `log --graph` with a `o`.
besides the obvious inconsistency, these choices are bad for other
reasons, too:

`-` is typically used in a diff whereas negative revs are denoted by
`^`.

`o` reminds of "origin/origo" at least as much as of "omitted";
visually, it's not clear that it "should not be present".

So, change both `log` and `log --graph` to use the same symbol `^` which
the revision parser uses for negative revs, too.

Author: mjg

Documentation/rev-list-options.adoc

@@ -349,7 +349,7 @@ Under `--pretty=reference`, this information will not be shown at all.
 
 `--boundary`::
 	Output excluded boundary commits. Boundary commits are
-	prefixed with `-`.
+	prefixed with `^`.
 
 ifdef::git-rev-list[]
 `--use-bitmap-index`::
@@ -1211,7 +1211,7 @@ endif::git-rev-list[]
 	Mark which side of a symmetric difference a commit is reachable from.
 	Commits from the left side are prefixed with `<` and those from
 	the right with `>`.  If combined with `--boundary`, those
-	commits are prefixed with `-`.
+	commits are prefixed with `^`.
 +
 For example, if you have this topology:
 +
@@ -1232,8 +1232,8 @@ you would get an output like this:
 	>bbbbbbb... 2nd on b
 	<aaaaaaa... 3rd on a
 	<aaaaaaa... 2nd on a
-	-yyyyyyy... 1st on b
-	-xxxxxxx... 1st on a
+	^yyyyyyy... 1st on b
+	^xxxxxxx... 1st on a
 -----------------------------------------------------------------------
 
 `--graph`::

graph.c

@@ -938,12 +938,12 @@ static void graph_output_pre_commit_line(struct git_graph *graph,
 static void graph_output_commit_char(struct git_graph *graph, struct graph_line *line)
 {
 	/*
-	 * For boundary commits, print 'o'
+	 * For boundary commits, print '^'
 	 * (We should only see boundary commits when revs->boundary is set.)
 	 */
 	if (graph->commit->object.flags & BOUNDARY) {
 		assert(graph->revs->boundary);
-		graph_line_addch(line, 'o');
+		graph_line_addch(line, '^');
 		return;
 	}
 

perl/Git/SVN/Log.pm

@@ -285,7 +285,7 @@ sub cmd_show_log {
 	my (@k, $c, $d, $stat);
 	my $esc_color = qr/(?:\033\[(?:(?:\d+;)*\d*)?m)*/;
 	while (<$log>) {
-		if (/^${esc_color}commit (?:- )?($::oid_short)/o) {
+		if (/^${esc_color}commit (?:\^ )?($::oid_short)/o) {
 			my $cmt = $1;
 			if ($c && cmt_showable($c) && $c->{r} != $r_last) {
 				$r_last = $c->{r};

revision.c

@@ -4531,7 +4531,7 @@ struct commit *get_revision(struct rev_info *revs)
 const char *get_revision_mark(const struct rev_info *revs, const struct commit *commit)
 {
 	if (commit->object.flags & BOUNDARY)
-		return "-";
+		return "^";
 	else if (commit->object.flags & UNINTERESTING)
 		return "U";
 	else if (commit->object.flags & PATCHSAME)

t/t3430-rebase-merges.sh

@@ -269,7 +269,7 @@ test_expect_success 'with a branch tip that was cherry-picked already' '
 	| * A1
 	* | B1
 	|/
-	o A2
+	^ A2
 	EOF
 '
 
@@ -279,7 +279,7 @@ test_expect_success '--no-rebase-merges countermands --rebase-merges' '
 	test_cmp_graph C.. <<-\EOF
 	* B
 	* D
-	o C
+	^ C
 	EOF
 '
 
@@ -297,7 +297,7 @@ test_expect_success 'do not rebase cousins unless asked for' '
 	| * D
 	| * G
 	|/
-	o H
+	^ H
 	EOF
 '
 
@@ -311,7 +311,7 @@ test_expect_success 'rebase.rebaseMerges=rebase-cousins is equivalent to --rebas
 	| * D
 	| * G
 	|/
-	o H
+	^ H
 	EOF
 '
 
@@ -322,7 +322,7 @@ test_expect_success '--no-rebase-merges overrides rebase.rebaseMerges=no-rebase-
 	test_cmp_graph C.. <<-\EOF
 	* B
 	* D
-	o C
+	^ C
 	EOF
 '
 
@@ -456,7 +456,7 @@ test_expect_success 'A root commit can be a cousin, treat it that way' '
 	*   Merge branch '\''khnum'\'' into asherah
 	|\
 	| * yama
-	o shamkat
+	^ shamkat
 	EOF
 	test_tick &&
 	git rebase --rebase-merges=rebase-cousins HEAD^ &&
@@ -465,7 +465,7 @@ test_expect_success 'A root commit can be a cousin, treat it that way' '
 	|\
 	| * yama
 	|/
-	o shamkat
+	^ shamkat
 	EOF
 '
 
@@ -515,7 +515,7 @@ test_expect_success 'octopus merges' '
 	| |/
 	* / one
 	|/
-	o before-octopus
+	^ before-octopus
 	EOF
 '
 

t/t6016-rev-list-graph-simplify-history.sh

@@ -233,13 +233,13 @@ test_expect_success '--graph --boundary ^C3' '
 	| * | | B2
 	| * | | B1
 	* | | | A3
-	o | | | A2
+	^ | | | A2
 	|/ / /
-	o / / A1
+	^ / / A1
 	 / /
-	| o C3
+	| ^ C3
 	|/
-	o C2
+	^ C2
 	EOF
 '