From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 12 May 2016 21:21:35 GMT
Subject: Re: [PATCH v2 22/94] builtin/apply: move 'threeway' global into 'struct apply_state'
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. */