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

From: Taylor Blau <me@ttaylorr.com>
Date: Oct 23, 2024, 20:33 UTC
Message-ID: <ZxldiQVBxAWbXoT4@nand.local>
In-Reply-To: <Zxi5TeHM9qD3lrbx@pks.im>

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

Show 7 quoted lines

> On Wed, Oct 23, 2024 at 07:40:17AM +0000, Usman Akinyemi via GitGitGadget wrote:
> > 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

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.