Remco, Shouldn't it be htmllib.HTMLParser.__init__(self, formatter.NullFormatter()) instead of htmllib.HTMLParser.__init__(self, formatter.NullFormatter) (and "self.linklist" instead of "linklist" in: def change_html(self, html): # Use the linklist and the modify_link function to return new html for link in linklist: html = html.replace(link, modify_link(link)) return html )? Walter