Re: [PATCH 0/2] give range-diff at the end of single patch output

From: Dragan Simic <dsimic@manjaro.org>
Date: May 23, 2024, 23:22 UTC
Message-ID: <83436db5996b129032c2869f331980d0@manjaro.org>
In-Reply-To: <20240523225007.2871766-1-gitster@pobox.com>

On 2024-05-24 00:50, Junio C Hamano wrote:

Show 15 quoted lines

> When running "format-patch" on a multiple patch series, the output
> coming from "--interdiff" and "--range-diff" options is inserted
> after the "shortlog" list of commits and the overall diffstat.
> 
> The idea is that shortlog/diffstat are shorter and with denser
> information content, which gives a better overview before the
> readers dive into more details of range/inter diff.
> 
> When working on a single patch, however, we stuff the inter/range
> diff output before the actual patch, next to the diffstat.  This
> pushes down the patch text way down with inter/range diff output,
> distracting readers.
> 
> Move the inter/range diff output to the very end of the output,
> after all the patch text is shown.

Hmm... I think this should be made configurable, with the current behavior being the default. Without that, we could easily disrupt many people's workflows, because the power of "muscle memory" is often really strong.

If it were just about moving a few lines up or down, making it configurable wouldn't make much sense, but with moving this large chunks of text... It's a different story.