facebook/sapling
1
1
Fork 0
mirror of https://github.com/facebook/sapling.git synced 2024-10-09 00:14:35 +03:00
Code Issues Projects Releases Wiki Activity

config: use the new '_unset' value for 'configbool'

Browse Source 
This should let 'configbool' delegate all special processing of the default
config value to the main 'config' method.

The default value for bool (False) is still enforced in this method if no other
default were passed.
This commit is contained in:
Pierre-Yves David 2017-06-17 12:52:16 +02:00
parent b2090538ae
commit ae76544acb
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
mercurial/ui.py
View File

@ -505,7 +505,7 @@ class ui(object):
v = os.path.join(base, os.path.expanduser(v))
return v
def configbool(self, section, name, default=False, untrusted=False):
def configbool(self, section, name, default=_unset, untrusted=False):
"""parse a configuration element as a boolean
>>> u = ui(); s = 'foo'
@ -526,8 +526,10 @@ class ui(object):
ConfigError: foo.invalid is not a boolean ('somevalue')
"""
v = self.config(section, name, None, untrusted)
v = self.config(section, name, default, untrusted=untrusted)
if v is None:
if default is _unset:
return False
return default
if isinstance(v, bool):
return v