Re: [PATCH v5 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: Taylor Blau <hidden>
Date: 2024-10-23 20:33:15

On Wed, Oct 23, 2024 at 10:52:36AM +0200, Patrick Steinhardt wrote:

On Wed, Oct 23, 2024 at 07:40:17AM +0000, Usman Akinyemi via GitGitGadget wrote:

quoted

Changes from Version 4:

 * Fix incorrect indentation and remove unnecessary braces to avoid
   confusion.

Thanks, this version looks good to me!

Thanks for reviewing. This one is looking pretty close, and I've moved
it to 'jch' in my tree, but I think there are still a few lingering
comments that I'd like to see addressed in a subsequent round before we
start merging this one down.

Thanks,
Taylor

`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