Pierre Quentel <pierre.quen...@gmail.com> added the comment: Thanks for the explanation
I still think that the patch can be simplified, not using path lengths and the "found" flag collapsed_path = _url_collapse_path(self.path) for head in self.cgi_directories: if head==collapsed_path: self.cgi_info = (head,'') return True elif collapsed_path.startswith(head) \ and collapsed_path[len(head)]=='/': self.cgi_info = head, collapsed_path[len(head)+1:] return True return False BTW the last "return False" is rather useless since is_cgi() is only used in tests like "if is_cgi()" ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue14565> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com