[PATCH RFC] forest extension

Josef "Jeff" Sipek jeffpc at josefsipek.net
Sat Dec 16 16:24:06 CST 2006


On Sun, Dec 10, 2006 at 03:02:09PM +0100, Robin Farine wrote:
> On Sunday December 10 2006 12:37, Rafael Villar Burke wrote:
> > Robin Farine escribió:
> 
> > >
> > > walkhg = (0|1)
> > >
> > >   Whether repositories under a .hg directory should be skipped
> > > (0) or not (1). The default value is 0. Some commands accept
> > > the --walkhg command-line option to override the behavior
> > > selected by this item.
> >
> > I'm being a bit picky here, but wouldn't be using False|True
> > instead of 0|1 more user friendly?. It's more readable by humans
> > than the C convention.
> 
> Yes, I agree that it would be more user friendly. I chose (0|1) 
> rather than (False|True) for the configuration item to be 
> consistent with the associated command-line option. Perhaps using a 
> string option instead so that one writes --walkhg=False rather 
> than --walkhg=0 on the command-line is preferable...

I think all of these should be accepted: 0|1 or No|Yes or False|True

Just my 2 cents,

Jeff.

-- 
Defenestration n. (formal or joc.):
  The act of removing Windows from your computer in disgust, usually
  followed by the installation of Linux or some other Unix-like operating
  system.


More information about the Mercurial-devel mailing list