Quantcast

[PATCH] rev-list --count: separate count for --cherry-mark

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[PATCH] rev-list --count: separate count for --cherry-mark

MichaelJGruber
When --count is used with --cherry-mark, omit the patch equivalent
commits from the count for left and right commits and print the count of
equivalent commits separately.

Signed-off-by: Michael J Gruber <[hidden email]>
---
I noticed this in passing. It does not break anything and can be done
easily. It might even be useful for gitprompt, branch -vv and the like.

Resent for the new cycle.
---
 Documentation/rev-list-options.txt   |    5 ++++-
 builtin/rev-list.c                   |   10 ++++++++--
 revision.h                           |    1 +
 t/t6007-rev-list-cherry-pick-file.sh |   27 +++++++++++++++++++++++++++
 4 files changed, 40 insertions(+), 3 deletions(-)

diff --git a/Documentation/rev-list-options.txt b/Documentation/rev-list-options.txt
index 5c6850f..e092a7c 100644
--- a/Documentation/rev-list-options.txt
+++ b/Documentation/rev-list-options.txt
@@ -715,7 +715,10 @@ ifdef::git-rev-list[]
  Print a number stating how many commits would have been
  listed, and suppress all other output.  When used together
  with '--left-right', instead print the counts for left and
- right commits, separated by a tab.
+ right commits, separated by a tab. When used together with
+ '--cherry-mark', omit patch equivalent commits from these
+ counts and print the count for equivalent commits separated
+ by a tab.
 endif::git-rev-list[]
 
 
diff --git a/builtin/rev-list.c b/builtin/rev-list.c
index f458cb7..8761e9a 100644
--- a/builtin/rev-list.c
+++ b/builtin/rev-list.c
@@ -51,7 +51,9 @@ static void show_commit(struct commit *commit, void *data)
  graph_show_commit(revs->graph);
 
  if (revs->count) {
- if (commit->object.flags & SYMMETRIC_LEFT)
+ if (commit->object.flags & PATCHSAME)
+ revs->count_same++;
+ else if (commit->object.flags & SYMMETRIC_LEFT)
  revs->count_left++;
  else
  revs->count_right++;
@@ -402,8 +404,12 @@ int cmd_rev_list(int argc, const char **argv, const char *prefix)
      &info);
 
  if (revs.count) {
- if (revs.left_right)
+ if (revs.left_right && revs.cherry_mark)
+ printf("%d\t%d\t%d\n", revs.count_left, revs.count_right, revs.count_same);
+ else if (revs.left_right)
  printf("%d\t%d\n", revs.count_left, revs.count_right);
+ else if (revs.cherry_mark)
+ printf("%d\t%d\n", revs.count_left + revs.count_right, revs.count_same);
  else
  printf("%d\n", revs.count_left + revs.count_right);
  }
diff --git a/revision.h b/revision.h
index ae94860..08900b3 100644
--- a/revision.h
+++ b/revision.h
@@ -141,6 +141,7 @@ struct rev_info {
  /* commit counts */
  int count_left;
  int count_right;
+ int count_same;
 };
 
 #define REV_TREE_SAME 0
diff --git a/t/t6007-rev-list-cherry-pick-file.sh b/t/t6007-rev-list-cherry-pick-file.sh
index cacf3de..28d4f6b 100755
--- a/t/t6007-rev-list-cherry-pick-file.sh
+++ b/t/t6007-rev-list-cherry-pick-file.sh
@@ -157,6 +157,33 @@ test_expect_success '--cherry' '
  test_cmp actual.named expect
 '
 
+cat >expect <<EOF
+1 1
+EOF
+
+test_expect_success '--cherry --count' '
+ git rev-list --cherry --count F...E -- bar > actual &&
+ test_cmp actual expect
+'
+
+cat >expect <<EOF
+2 2
+EOF
+
+test_expect_success '--cherry-mark --count' '
+ git rev-list --cherry-mark --count F...E -- bar > actual &&
+ test_cmp actual expect
+'
+
+cat >expect <<EOF
+1 1 2
+EOF
+
+test_expect_success '--cherry-mark --left-right --count' '
+ git rev-list --cherry-mark --left-right --count F...E -- bar > actual &&
+ test_cmp actual expect
+'
+
 test_expect_success '--cherry-pick with independent, but identical branches' '
  git symbolic-ref HEAD refs/heads/independent &&
  rm .git/index &&
--
1.7.5.270.gafca7

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [hidden email]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [PATCH] rev-list --count: separate count for --cherry-mark

Junio C Hamano
Michael J Gruber <[hidden email]> writes:

> When --count is used with --cherry-mark, omit the patch equivalent
> commits from the count for left and right commits and print the count of
> equivalent commits separately.
>
> Signed-off-by: Michael J Gruber <[hidden email]>
> ---
> I noticed this in passing. It does not break anything and can be done
> easily. It might even be useful for gitprompt, branch -vv and the like.

This is in the same spirit as your earlier change to not count filtered
commits, right?  Sensible.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [hidden email]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Loading...