Message 158010 - Python tracker

➜

This issue tracker has been migrated to GitHub , and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

In-reply-to
Author	v+python
Recipients	Giovanni.Funchal, facundobatista, fdrake, orsenthil, python-dev, v+python
Date	2012年04月11日.06:35:23
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1334126123.72.0.387958713986.issue10484@psf.upfronthosting.co.za>

Content
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.

Content

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.

History
Date	User	Action	Args
2012年04月11日 06:35:23	v+python	set	recipients: + v+python, fdrake, facundobatista, orsenthil, python-dev, Giovanni.Funchal
2012年04月11日 06:35:23	v+python	set	messageid: <1334126123.72.0.387958713986.issue10484@psf.upfronthosting.co.za>
2012年04月11日 06:35:23	v+python	link	issue10484 messages
2012年04月11日 06:35:23	v+python	create

homepage