Re: [PATCH v2 0/6] tr2: plug memory leaks + logic errors + Win32 & Linux feature parity

From: Taylor Blau <me@ttaylorr.com>
Date: Aug 26, 2021, 22:38 UTC
Message-ID: <YSgX1ON+FvpwPjKB@nand.local>
In-Reply-To: <cover-v2-0.6-00000000000-20210826T121820Z-avarab@gmail.com>

On Thu, Aug 26, 2021 at 02:22:18PM +0200, Ævar Arnfjörð Bjarmason wrote:

> An early re-roll due to some very good & early review by Taylor Blau,
> this also fixes the grammar/typo pointed out by Eric Sunshine. Thanks
> both:

Thanks; aside from the cosmetic issues that Junio and Eric already brought up, this version looks good to me. I probably would have avoided 4/6 and jumped straight to 6/6, but your approach is good, too.

Thanks for being so receptive to my comments (public and private) in the earlier round of this series.

    Reviewed-by: Taylor Blau <me@ttaylorr.com>

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.