[issue14679] Define an __all__ for html.parser

Ezio Melotti report at bugs.python.org
Sat Mar 30 00:53:43 CET 2013


Ezio Melotti added the comment:

The module only defines 2 public objects: HTMLParser and HTMLParseError.  
The attached patch adds an __all__ with only HTMLParser.  Should HTMLParseError be included too, even though it's deprecated and will be removed in 3.5?

@Michele:
__all__ won't fix that, but at least it will make clear that the module-level regex (like tagfind) are not public.  Renaming the methods is not a good idea IMHO, we could move the "# internal" comments in the docstrings though.

----------
keywords: +patch
stage: needs patch -> patch review
Added file: http://bugs.python.org/file29607/issue14679.diff

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue14679>
_______________________________________


More information about the Python-bugs-list mailing list