Terry J. Reedy <tjre...@udel.edu> added the comment:

I verified the looping behavior of the testcase in both 2.7.1 and, with minor 
mods, 3.1.3 and 3.2b1, so this is a valid issue.

The HTMLParcer docs (2.7, 3.2) do not mention the .error method. The default is
    def error(self, message):
        raise HTMLParseError(message, self.getpos())

If this is *not* intended to be part of the api and over-ridden, the name 
should be changed to ._error and .error deprecated. If it is, it should be 
documented.

I think the self.error call should be followed either by j+=1 so parsing 
continues with the next char or by a raise statememt so it is definitely 
stopped.

----------
versions: +Python 3.1, Python 3.2

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue8885>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to