On Wed, 2010-04-14 at 16:11 -0400, Seth Vidal wrote: > def __init__(self, elem): > - self.type = elem.attrib.get('type') > + self.type = None > + if elem: > + self.type = elem.attrib.get('type') Why not make the param default to None, if we make that valid?