Re: [PATCH 2/3] config: require at least one digit when parsing numbers

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

Subject: Re: [PATCH 2/3] config: require at least one digit when parsing numbers
From: Junio C Hamano <gitster@xxxxxxxxx>
Date: Sat, 22 Oct 2022 14:00:47 -0700
Cc: Phillip Wood via GitGitGadget <gitgitgadget@xxxxxxxxx>, git@xxxxxxxxxxxxxxx, René Scharfe <l.s.r@xxxxxx>, Phillip Wood <phillip.wood@xxxxxxxxxxxxx>
In-reply-to: <Y1RRrK4/2vr/YPfz@coredump.intra.peff.net> (Jeff King's message of "Sat, 22 Oct 2022 16:25:16 -0400")
References: <pull.1389.git.1666359915.gitgitgadget@gmail.com> <cd753602e48a2faa0d59edca2f6fab0fe753f0f6.1666359915.git.gitgitgadget@gmail.com> <Y1L+Qv+cs1bjqjK9@coredump.intra.peff.net> <xmqqilkbhhlw.fsf@gitster.g> <Y1RRrK4/2vr/YPfz@coredump.intra.peff.net>
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.1 (gnu/linux)

Jeff King <peff@xxxxxxxx> writes:

> I admit I don't care much either way, though.

Me neither.  If the patch were to make --default=m mean "one mega unit"
instead of "zero mega unit", then I may care a bit more, though.

References:
- [PATCH 0/3] a few config integer parsing fixes
  - From: Phillip Wood via GitGitGadget
- [PATCH 2/3] config: require at least one digit when parsing numbers
  - From: Phillip Wood via GitGitGadget
- Re: [PATCH 2/3] config: require at least one digit when parsing numbers
  - From: Jeff King
- Re: [PATCH 2/3] config: require at least one digit when parsing numbers
  - From: Junio C Hamano
- Re: [PATCH 2/3] config: require at least one digit when parsing numbers
  - From: Jeff King

Prev by Date: Re: [PATCH v2 2/2] setup: allow Git.pm to do unsafe repo checking
Next by Date: Re: [PATCH v2 2/2] setup: allow Git.pm to do unsafe repo checking
Previous by thread: Re: [PATCH 2/3] config: require at least one digit when parsing numbers
Next by thread: Re: [PATCH 2/3] config: require at least one digit when parsing numbers
Index(es):
- Date
- Thread

[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]