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

[PATCH 0/3] R atoi · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-12
[PATCH 1/3] t3404: avoid losing exit status with focus on `git show` and `git cat-file` · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-12
Re: [PATCH 1/3] t3404: avoid losing exit status with focus on `git show` and `git cat-file` · Taylor Blau <hidden> · 2024-10-14
[PATCH 2/3] t3404: replace test with test_line_count() · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-12
Re: [PATCH 2/3] t3404: replace test with test_line_count() · Taylor Blau <hidden> · 2024-10-14
[PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-12
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-13
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Phillip Wood <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Patrick Steinhardt <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Phillip Wood <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Patrick Steinhardt <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Phillip Wood <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-15
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Taylor Blau <hidden> · 2024-10-15
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-16
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · <hidden> · 2024-10-15
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Phillip Wood <hidden> · 2024-10-16
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-16
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-17
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Patrick Steinhardt <hidden> · 2024-10-17
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-17
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Phillip Wood <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Kristoffer Haugsbakk <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Phillip Wood <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · <hidden> · 2024-10-14
Re: [PATCH 3/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-17
[PATCH v2 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-18
[PATCH v2 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-18
Re: [PATCH v2 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Patrick Steinhardt <hidden> · 2024-10-21
Re: [PATCH v2 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-21
Re: [PATCH v2 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Taylor Blau <hidden> · 2024-10-21
Re: [PATCH v2 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-21
[PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-18
Re: [PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · Patrick Steinhardt <hidden> · 2024-10-21
Re: [PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · Usman Akinyemi <hidden> · 2024-10-21
Re: [PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · Taylor Blau <hidden> · 2024-10-21
Re: [PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · Usman Akinyemi <hidden> · 2024-10-21
Re: [PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · Usman Akinyemi <hidden> · 2024-10-21
Re: [PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · Taylor Blau <hidden> · 2024-10-21
Re: [PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · Phillip Wood <hidden> · 2024-10-30
Re: [PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · Usman Akinyemi <hidden> · 2024-10-30
Re: [PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · Phillip Wood <hidden> · 2024-10-31
Re: [PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · Usman Akinyemi <hidden> · 2024-10-31
Re: [PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · Usman Akinyemi <hidden> · 2024-11-06
Re: [PATCH v2 2/3] merge: replace atoi() with strtol_i() for marker size validation · <hidden> · 2024-11-06
[PATCH v2 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-18
Re: [PATCH v2 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Patrick Steinhardt <hidden> · 2024-10-21
Re: [PATCH v2 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Usman Akinyemi <hidden> · 2024-10-21
Re: [PATCH v2 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Patrick Steinhardt <hidden> · 2024-10-21
Re: [PATCH v2 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Usman Akinyemi <hidden> · 2024-10-21
Re: [PATCH v2 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Taylor Blau <hidden> · 2024-10-21
Re: [PATCH v2 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Usman Akinyemi <hidden> · 2024-10-21
Re: [PATCH v2 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Usman Akinyemi <hidden> · 2024-10-22
Re: [PATCH v2 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Taylor Blau <hidden> · 2024-10-18
Re: [PATCH v2 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-18
Re: [PATCH v2 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Taylor Blau <hidden> · 2024-10-18
Re: [PATCH v2 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-18
[PATCH v3 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-22
[PATCH v3 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-22
Re: [PATCH v3 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Taylor Blau <hidden> · 2024-10-22
Re: [PATCH v3 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-22
[PATCH v3 2/3] merge: replace atoi() with strtol_i() for marker size validation · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-22
[PATCH v3 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-22
[PATCH v4 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-22
[PATCH v4 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-22
[PATCH v4 2/3] merge: replace atoi() with strtol_i() for marker size validation · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-22
[PATCH v4 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-22
Re: [PATCH v4 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Patrick Steinhardt <hidden> · 2024-10-23
Re: [PATCH v4 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Usman Akinyemi <hidden> · 2024-10-23
[PATCH v5 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-23
[PATCH v5 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-23
Re: [PATCH v5 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Taylor Blau <hidden> · 2024-10-23
Re: [PATCH v5 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-24
[PATCH v5 2/3] merge: replace atoi() with strtol_i() for marker size validation · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-23
Re: [PATCH v5 2/3] merge: replace atoi() with strtol_i() for marker size validation · Taylor Blau <hidden> · 2024-10-23
Re: [PATCH v5 2/3] merge: replace atoi() with strtol_i() for marker size validation · Usman Akinyemi <hidden> · 2024-10-24
[PATCH v5 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-23
Re: [PATCH v5 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Patrick Steinhardt <hidden> · 2024-10-23
Re: [PATCH v5 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Taylor Blau <hidden> · 2024-10-23
Re: [PATCH v5 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-24
[PATCH v6 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-24
[PATCH v6 1/3] daemon: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-24
[PATCH v6 2/3] merge: replace atoi() with strtol_i() for marker size validation · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-24
[PATCH v6 3/3] imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT parsing · Usman Akinyemi via GitGitGadget <hidden> · 2024-10-24
Re: [PATCH v6 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Taylor Blau <hidden> · 2024-10-24
Re: [PATCH v6 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Patrick Steinhardt <hidden> · 2024-10-25
Re: [PATCH v6 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Usman Akinyemi <hidden> · 2024-10-25
Re: [PATCH v6 0/3] parse: replace atoi() with strtoul_ui() and strtol_i() · Taylor Blau <hidden> · 2024-10-25

From: Patrick Steinhardt <hidden>
Date: 2024-10-25 05:07:07

On Thu, Oct 24, 2024 at 02:03:12PM -0400, Taylor Blau wrote:

On Thu, Oct 24, 2024 at 12:24:55AM +0000, Usman Akinyemi via GitGitGadget wrote:

quoted

Usman Akinyemi (3):
  daemon: replace atoi() with strtoul_ui() and strtol_i()
  merge: replace atoi() with strtol_i() for marker size validation
  imap: replace atoi() with strtol_i() for UIDVALIDITY and UIDNEXT
    parsing

Thanks, this new round looks quite good to me. Do others have thoughts
on this, or are we ready to start merging it down?

I'm happy with this version.

Patrick

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help