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

_______________________________________________
Python-Dev mailing list
Python-Dev@python.org
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: 
http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com

Reply via email to