diff options
author | Jeff King <peff@peff.net> | 2020-09-27 11:39:59 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2020-09-27 22:21:05 +0300 |
commit | 92338c450bc41e8e3da31a10d2ab73844e8b0634 (patch) | |
tree | 4efb1599c7d62617ea6a77bb413b2a9046f571c7 /shortlog.h | |
parent | 45d93eb82403a35d248d17dd0ea7dc3d690b1323 (diff) |
shortlog: add grouping option
In preparation for adding more grouping types, let's refactor the
committer/author grouping code and add a user-facing option that binds
them together. In particular:
- the main option is now "--group", to make it clear
that the various group types are mutually exclusive. The
"--committer" option is an alias for "--group=committer".
- we keep an enum rather than a binary flag, to prepare
for more values
- we prefer switch statements to ternary assignment, since
other group types will need more custom code
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'shortlog.h')
-rw-r--r-- | shortlog.h | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/shortlog.h b/shortlog.h index 2fa61c4294..876a52158d 100644 --- a/shortlog.h +++ b/shortlog.h @@ -15,7 +15,11 @@ struct shortlog { int in2; int user_format; int abbrev; - int committer; + + enum { + SHORTLOG_GROUP_AUTHOR = 0, + SHORTLOG_GROUP_COMMITTER, + } group; char *common_repo_prefix; int email; |