[Python-Dev] Question regarding: Lib/_markupbase.py

Developer Developer just_another_developer at yahoo.de
Mon Feb 11 13:16:48 CET 2013

Hello all,

I was having a look at the file: Lib/_markupbase.py (@ 82151), function: "_parse_doctype_element" and have seen something that has caught my attention:

if '>' in rawdata[j:]:
    return rawdata.find(">", j) + 1

Wouldn't it be better to do the following?
pos = rawdata.find(">", j)
if pos != -1:
    return pos + 1

Otherwise I think we are scanning rawdata[j:] twice.

Best regards

More information about the Python-Dev mailing list