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

Augie Fackler raf at durin42.com
Wed Sep 28 17:07:02 UTC 2016


> 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?

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



More information about the Mercurial-devel mailing list