Re: [PATCH] format-patch: use --compact-summary instead of --summary

2018-03-17 Thread Duy Nguyen
On Fri, Mar 16, 2018 at 10:41 PM, Junio C Hamano wrote: > Nguyễn Thái Ngọc Duy writes: > >> For reviewers, a couple fewer lines from diffstat (either cover letter >> or patches) is a good thing because there's less to read. > > I can certainly accept that

Re: [PATCH] format-patch: use --compact-summary instead of --summary

2018-03-16 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy writes: > For reviewers, a couple fewer lines from diffstat (either cover letter > or patches) is a good thing because there's less to read. I can certainly accept that there may be some reviewers who share that view, but me personally, I would rather

[PATCH] format-patch: use --compact-summary instead of --summary

2018-03-16 Thread Nguyễn Thái Ngọc Duy
For reviewers, a couple fewer lines from diffstat (either cover letter or patches) is a good thing because there's less to read. Signed-off-by: Nguyễn Thái Ngọc Duy --- I realize that rename percentage is lost with this change. I just hope that nobody cares about that, or