Re: [PATCH v3 1/5] doc: add documentation for OPT_STRING_LIST

From: Junio C Hamano <gitster@pobox.com>
Date: Jan 19, 2017, 17:58 UTC
Message-ID: <xmqqh94v2apv.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <CA+P7+xrv35w0RYNEZ88K_EWC57A4utBQTibtw6+TJBCtzA9Ybw@mail.gmail.com>

Jacob Keller <jacob.keller@gmail.com> writes:

Show 10 quoted lines

> On Wed, Jan 18, 2017 at 11:45 AM, Junio C Hamano <gitster@pobox.com> wrote:
>>
>> I do not know if it is clear enough that 'option' in the last
>> sentence is a placeholder.  I then wondered if spelling it as
>> `--no-<long>` would make it a bit clearer, but that is ugly.
>
> To be fair, this is exactly how the rest of the doc spells these
> things, so I would rather be consistent with the doc as is, and a
> future patch could clean this up. See OPT_SET_INT, for an example of
> `--no-option`.

Ah, OK, then I have no issues with it.

> Maybe we can rephrase this "The list is reset via `--no-option`"?

I think I saw that in your latest interdiff and I think it is good.

Thanks.