Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()

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

Subject: Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
From: Usman Akinyemi <usmanakinyemi202@xxxxxxxxx>
Date: Wed, 16 Oct 2024 18:00:00 +0000
Cc: Patrick Steinhardt <ps@xxxxxx>, Usman Akinyemi via GitGitGadget <gitgitgadget@xxxxxxxxx>, git@xxxxxxxxxxxxxxx
In-reply-to: <87090897-7ff9-4396-b81c-0fc423593602@gmail.com>
References: <pull.1810.git.git.1728774574.gitgitgadget@gmail.com> <c93bc2d81ffb33a2a61dda2878fa3b9987545e0b.1728774574.git.gitgitgadget@gmail.com> <CAPSxiM-V1qOB9QXUY3aDh+_nGdDHBWXJZ54U9p_XxKfHoODu7A@mail.gmail.com> <Zwz4B4osJnYJw6pd@pks.im> <2a937b6f-a3fb-4f2a-997b-5508f0e20e65@gmail.com> <Zw0kGLZ-mcYjb6Je@pks.im> <2160f8ea-5f00-49d9-8e02-d71d4d827d39@gmail.com> <CAPSxiM9ncwaZ3HF72wsRwmen7joWk3mjipsu78WxKEzLX607sw@mail.gmail.com> <CAPSxiM-aptyjesMX1H-P5QJjA-6CUonA01Bo84cq2_t==TqFgw@mail.gmail.com> <84dbe9f1-976d-45f8-a49a-d0f942906686@gmail.com> <CAPSxiM8-C6DAE3nYqMUCs+UgHN1R41grwVE+S-cSi6gZGvCpYw@mail.gmail.com> <9952b3a7-0ebc-4555-b8b3-f50f6f383704@gmail.com> <87090897-7ff9-4396-b81c-0fc423593602@gmail.com>

On Wed, Oct 16, 2024 at 9:20 AM Phillip Wood <phillip.wood123@xxxxxxxxx> wrote:
>
> On 15/10/2024 19:28, phillip.wood123@xxxxxxxxx wrote:
> > On 15/10/2024 16:17, Usman Akinyemi wrote:
> >>
> >> I grepped the error message and I noticed that the message is gotten
> >> from parse-options.c and it ensures that the arg is negative. How to
> >> proceed in such a case ?
> >
> > The code you're changing parses the conflict-marker-size attribute so
> > you need to set up a .gitattributes file with an invalid marker size and
> > then run "git merge" or "git cherry-pick"
Thanks.
>
> t/t6406-merge-attr.sh would be a good place to add this test
Thank you Philip.
>
> Best Wishes
>
> Phillip
>

References:
- [PATCH 0/3] R atoi
  - From: Usman Akinyemi via GitGitGadget
- [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
  - From: Usman Akinyemi via GitGitGadget
- Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
  - From: Usman Akinyemi
- Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
  - From: Patrick Steinhardt
- Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
  - From: Phillip Wood
- Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
  - From: Patrick Steinhardt
- Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
  - From: Phillip Wood
- Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
  - From: Usman Akinyemi
- Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
  - From: Usman Akinyemi
- Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
  - From: phillip . wood123
- Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
  - From: Usman Akinyemi
- Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
  - From: phillip . wood123
- Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
  - From: Phillip Wood

Prev by Date: Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
Next by Date: [PATCH v3 0/2] builtin/shortlog: explicitly set hash algo when there is no repo
Previous by thread: Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
Next by thread: Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i()
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]