Re: [PATCH v4 00/15] Remove submodule from files-backend.c

From: Duy Nguyen <pclouds@gmail.com>
Date: Feb 20, 2017, 12:47 UTC
Message-ID: <CACsJy8CsiB39G36FC1pqjWqmrqSnW2Vf4ooBLzhQ97Dk8tXuFA@mail.gmail.com>
In-Reply-To: <cdf01f55-f3df-d3ce-149c-0249a03d27bf@alum.mit.edu>

On Mon, Feb 20, 2017 at 7:42 PM, Michael Haggerty <mhagger@alum.mit.edu> wrote:

Show 7 quoted lines

> On 02/18/2017 02:32 PM, Nguyễn Thái Ngọc Duy wrote:
>> v4:
>
> I very much like the direction of this patch series. I reviewed the
> design pretty carefully and left some comments about it, and skimmed
> through the code changes. But I haven't yet reviewed the code in detail.
> I'll wait for your reaction to my design comments before doing so.

Unless there are some mails in transit, thanks for the review. The comments I haven't replied to usually mean "I agree" (but not writing unless I could add anything after).

-- 
Duy

$ 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.