Re: [PATCH] format-patch: clarify --creation-factor=<factor>

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

 



On Thu, Jul 28, 2022 at 4:59 PM Junio C Hamano <gitster@xxxxxxxxx> wrote:
> Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> writes:
> >> diff --git a/Documentation/git-format-patch.txt b/Documentation/git-format-patch.txt
> >> @@ -27,7 +27,7 @@ SYNOPSIS
> >> -               [--range-diff=<previous> [--creation-factor=<percent>]]
> >> +               [--range-diff=<previous> [--creation-factor=<factor>]]
> >> @@ -321,7 +321,7 @@ product of `format-patch` is generated, and they are not passed to
> >> ---creation-factor=<percent>::
> >> +--creation-factor=<factor>::
> >
> > Looks good as far as it goes, looks like both of your patches need to
> > also tweak this bit though:
> >
> >       $ git -P grep 'percentage.*creation' -- '*.c'
> >       builtin/log.c:                      N_("percentage by which creation is weighted")),
> >       builtin/range-diff.c:                       N_("percentage by which creation is weighted")),
> >
> > Probably just s/percentage/factor/ in for those -h strings?
>
> Thanks for being extra careful.
>
> 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.

(Slightly chagrined that I forgot to check the usage-string in the C
files. Thanks, Ævar.)



[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