Senthil Kumaran <orsent...@gmail.com> added the comment:

I reviewed the patch and we may not go with it.
- There is recent improvements in parsing and the patch does not go well with 
it, especially the clear_cache removal.
Also there is a mistake in the patch:

-            scheme, url = url[:i].lower(), url[i+1:]
+            url = url[i+1:]

This can create problems for certain parsing logic as scheme is being used 
later in the code.

Closing the bug is invalid.

----------
resolution:  -> invalid
stage: unit test needed -> committed/rejected
status: open -> closed

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

Reply via email to