Re: [PATCH] format-patch: clarify --creation-factor=<factor>
- From
Eric Sunshine <sunshine@sunshineco.com>
- Date
- Jul 30, 2022, 00:25 UTC
- Message-ID
- <CAPig+cQ94oC-6wftkP_=RSEsOVPKeuBWuMshDkwV+Xu_FjrKeg@mail.gmail.com>
- In-Reply-To
- <CAPig+cTxM2vr11XRU-zrNAJRwveP0W9d0x8Euvr6HKmQicKgPQ@mail.gmail.com>
On Thu, Jul 28, 2022 at 5:09 PM Eric Sunshine <sunshine@sunshineco.com> wrote:
Show 11 quoted lines
> On Thu, Jul 28, 2022 at 4:59 PM Junio C Hamano <gitster@pobox.com> wrote: > > >> - [--range-diff=<previous> [--creation-factor=<percent>]] > > >> + [--range-diff=<previous> [--creation-factor=<factor>]] > > > > Eric, I am not sure if the other patch for the range-diff command as > > a standalone patch is worth it. Perhaps you can help me by > > submitting a single combined patch to cover the above as well as > > what we wrote in the two patches, with you marked as the primary > > author and with Helped-by: that credits Ævar? > > Will do.
Given Dscho's response in [1] which states that "percent" is more accurate than "factor", perhaps these two patches are unwanted after all, and instead the documentation of "creation factor" in Documentation/git-range-diff.txt ought to be expanded with a better explanation, as hinted in [2].
[1]: https://lore.kernel.org/git/85snn12q-po05-osqs-n1o0-n6040392q01q@tzk.qr/ [2]: https://lore.kernel.org/git/xmqq5yjf4l60.fsf@gitster.g/