Re: [PATCH v2 22/94] builtin/apply: move 'threeway' global into 'struct apply_state'

From: Junio C Hamano <gitster@pobox.com>
Date: May 12, 2016, 21:21 UTC
Message-ID: <xmqqeg97c6qo.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <CAP8UFD1VYW6+T_oTHJbemUhi8MPrMseaObWgoy0Xou=VCoKt-A@mail.gmail.com>

Christian Couder <christian.couder@gmail.com> writes:

> Ok, I will try to group knobs like that, but the comments tend to
> break the groups.

By keeping the comment on a single field short, and reserve comment occupying its own line(s) to comment on group, you can do

	/* These control what gets looked at and modified */
        int	apply;	/* this is not a dry-run */
        int	check_index; /* preimage must match the indexed version */
        int	cached; /* apply to the index only */

	/* These control cosmetic aspect of the output */
        int	diffstat; /* show diffstat */
        int	numstat; /* numerical stat */
        int	summary; /* report creation, deletion, etc. */

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