Re: [PATCH v2 1/2] merge/pull: --compact-summary

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



"Kristoffer Haugsbakk" <kristofferhaugsbakk@xxxxxxxxxxxx> writes:

> On Fri, Jun 13, 2025, at 00:25, Junio C Hamano wrote:
>>  static struct option builtin_merge_options[] = {
>>  	OPT_SET_INT('n', NULL, &show_diffstat,
>>  		N_("do not show a diffstat at the end of the merge"), 0),
>>  	OPT_BOOL(0, "stat", &show_diffstat,
>>  		N_("show a diffstat at the end of the merge")),
>>  	OPT_BOOL(0, "summary", &show_diffstat, N_("(synonym to --stat)")),
>> +	OPT_CALLBACK_F(0, "compact-summary", &show_diffstat, N_("compact-summary"),
>> +		       N_("show a compactstat at the end of the merge"),
>> +		       PARSE_OPT_NOARG,
>> +		       option_parse_compact_summary),
>
> `compactstat` is very dense.  s/compactstat/compact stat/ ?

Nice spotting.  I think "compact summary" (to match the diff option
"--compact-summary") may be even better.





[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux