git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] abspath.h is created.

From
Taylor Blau <me@ttaylorr.com>
Date
Sep 28, 2022, 15:39 UTC
Message-ID
<YzRqt2rxdhfG9CdX@nand.local>
In-Reply-To
<CA+J78MXbXG4M1w50EAM8t0VkYP_saCt9O90jFKad3+cB+RCZ-Q@mail.gmail.com>
Hi Skrab,
On Wed, Sep 28, 2022 at 01:02:22PM +0530, Skrab Sah wrote:
> What is wrong here and what should i do to make it correct.

I suspect that Junio's concern is that it appears the tool you're using to auto-generate these .h files moves the comments away from the declaration and into the header.

I imagine that such a change would be improved if:
  - You kept the comment attached to the function declaration (and
    ensured that it was present in the header file).
  - And we dropped the "this file is auto-generated by ..." comment,
    since I don't imagine that folks will be interested in adding a new
    tool to the development toolchain.
Thanks.
Taylor
Previous: Skrab SahNext: Junio C Hamano
Message 4 of 5 in “abspath.h is created.”
  1. abspath.h is created.skrab-sah via GitGitGadget, Sep 27, 2022
  2. Junio C HamanoSep 28, 2022
  3. Skrab SahSep 28, 2022
  4. Taylor BlauSep 28, 2022
  5. Junio C HamanoSep 28, 2022

Read the whole thread, see it on lore, or plain text.

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