On Wed, May 13, 2020 at 06:48:26PM +0200, Andrea Bolognani wrote:
> On Tue, 2020-05-12 at 16:42 +0200, Erik Skultety wrote:
> > + def _validate_section(self, config, section, mandatory_keys):
> > + # remove keys we don't recognize
> > + self._remove_unknown_keys(config[section], self.values[section].keys())
> > +
> > + # check that the mandatory keys are present and non-empty
> > + for key in mandatory_keys:
> > + if config.get(section).get(key, None) is None:
>
> Isn't
>
> foo.get("bar")
>
> defined as returning None if the "bar" key is not defined for the
> dictionary foo? Basically, I think you could write this as
>
> if config.get(section).get(key) is None:
Correct, I was just being explicit, it might not be clear at first glance that
None is the default return value, but I agree that what you suggest is the
Pythonic way, so I'll happily change it.
--
Erik Skultety
