[PATCH v5] help: mark boolean flags with [no-] to explain that they can be negated (RFC)

Kevin Bullock kbullock+mercurial at ringworld.org
Wed Sep 28 18:24:59 EDT 2016


> On Sep 28, 2016, at 12:07, Augie Fackler <raf at durin42.com> wrote:
> 
> 
>> On Sep 28, 2016, at 10:36, Kevin Bullock <kbullock+mercurial at ringworld.org> wrote:
>> 
>>> On Sep 28, 2016, at 08:32, Augie Fackler <raf at durin42.com> wrote:
>>> 
>>>> On Sep 28, 2016, at 08:25, Pierre-Yves David <pierre-yves.david at ens-lyon.org> wrote:
>>>> 
>>>>> which avoid the visual noise of the current approach in this change is
>>>>> unfortunate, it matches the format I'm used to seeing in man pages and
>>>>> similar documentation venues.
>>>> 
>>>> I'm confused about this sentence. It seems like some part is missing. What is unfortunate? What version is matching the format we are use to seeing in man pages?
>>> 
>>> 
>>> The current version -> the version in v5
>>> 
>>> has unfortunate visual clutter, but also matches what I'm used to seeing in man pages.
>>> 
>>> I'll grant you it's an unusual construction though. Perhaps
>>> 
>>> 
>>>> which avoid the unfortunate visual noise in this patch. In this version's favor, it's consistent with what I'm used to seeing in man pages and similar documentation venues.
>>> 
>>> is clearer?
>> 
>> Thank you, yes, this is clear now. I also had trouble parsing the original.
> 
> Should I mail a v6? How do people feel about this particular patch, commit message aside?

I'd say it's the cleanest of the options presented thus far. I'm in favor.

pacem in terris / мир / शान्ति / ‎‫سَلاَم‬ / 平和
Kevin R. Bullock



More information about the Mercurial-devel mailing list