Glenn Linderman <v+pyt...@g.nevcal.com> added the comment:

A minor detail of efficiency:

_url_collapse_path_split is used only one place, from is_cgi.
_url_collapse_path_split splits the path, and is_cgi now puts it back together. 
 This seems inefficient.  Would it not be better for _url_collapse_path_split 
to be renamed to _url_collapse_path and simply return a single value, the path 
canonicalized (.. removed)?

Then is_cgi would not need to put it back together before taking it apart 
another way.

----------

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

Reply via email to