# HG changeset patch # User Franz Glasner # Date 1457520198 -3600 # Node ID 24ba462b9b4b7e5f2524ee0e969b2134e857991e # Parent 0f64d640cba0854a7cab4ecdfc79fc2c483317a2 Return the `default' argument when the given `user' argument is `None' and when it is the first call on merge diff -r 0f64d640cba0 -r 24ba462b9b4b configmix/__init__.py --- a/configmix/__init__.py Wed Mar 09 11:35:57 2016 +0100 +++ b/configmix/__init__.py Wed Mar 09 11:43:18 2016 +0100 @@ -40,31 +40,39 @@ return result -def merge(user, default): +def merge(user, default, _first=True): """A simple (YAML-)tree merge. From http://stackoverflow.com/questions/823196/yaml-merge-in-python + .. note:: `_first` is an internal argument. + """ + if _first and (user is None): + return default if isinstance(user, dict) and isinstance(default, dict): for k, v in default.items(): if k not in user: user[k] = v else: - user[k] = merge(user[k], v) + user[k] = merge(user[k], v, False) return user -def safe_merge(user, default): +def safe_merge(user, default, _first=True): """A more safe version of `merge()` that makes shallow copies of - the container objects. + the returned container objects. + + .. note:: `_first` is an internal argument. """ + if _first and (user is None): + return copy.copy(default) user = copy.copy(user) if isinstance(user, dict) and isinstance(default, dict): for k, v in default.items(): if k not in user: user[k] = copy.copy(v) else: - user[k] = merge(user[k], v) + user[k] = merge(user[k], v, False) return user