Author Gumnos
Recipients Gumnos, eric.araujo, meatballhat
Date 2010-05-10.00:51:10
SpamBayes Score 1.56992e-05
Marked as misclassified No
Message-id <1273452686.58.0.445680195725.issue8666@psf.upfronthosting.co.za>
In-reply-to
Content
Yes, the use-case is the same as a dict.get(key, default) which I frequently use -- so it can be used in things like

  result = some_function(
     cp.get('MySection', 'MyValue', default='hello'),
     cp.getint('MySection', 'MyInt', default=42)
     )

To write something similar with the current ConfigParser requires a lot more code or an inelegant wrapper (either passing the config-parser object each call, or capturing the parser object in a closure which makes the "def" something repeated):

  try:
    mv = cp.get('MySection', 'MyValue')
  except (NoSectionError, NoOptionError), e:
    mv = 'hello'
  try:
    mi = cp.getint('MySection', 'MyInt')
  except (NoSectionError, NoOptionError), e:
    mi = 42
  result = some_function(mv, mi)
  del mv, mi  # leaving the namespace as it was before

The above can be done in a scattering of wrapper functions, but the addition in stock ConfigParser prevents a lot of duplicate code.  This is the fourth project in which I've used the ConfigParser and have reached for a default value in every one of them, to be frustrated by the lack.

The old-style "raise ValueError" you mention was the original code (just indented), but I've changed it to the preferred format.

For the dangling close-paren or "):" on its own line, I tend to do it for the same reason a trailing comma is allowed/encouraged in lists/tuples/dicts/param lists, so it's easy to add further comma-separated entries without giving cluttered diffs.  The source against which I'm patching has several dangling close-parens already (search for "^\s*)" to find the calls to re.compile).

I pulled down the branches/py3k and patched against it. (attached)
History
Date User Action Args
2010-05-10 00:51:26Gumnossetrecipients: + Gumnos, eric.araujo, meatballhat
2010-05-10 00:51:26Gumnossetmessageid: <1273452686.58.0.445680195725.issue8666@psf.upfronthosting.co.za>
2010-05-10 00:51:24Gumnoslinkissue8666 messages
2010-05-10 00:51:17Gumnoscreate