➜

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.

classification
Title: Use sets instead of list in posixpath._resolve_link
Type: performance Stage:
Components: Library (Lib) Versions: Python 2.7
process
Status: closed Resolution: accepted
Dependencies: Superseder:
Assigned To: Nosy List: benjamin.peterson, ggenellina, tzot
Priority: normal Keywords: patch

Created on 2009-01-26 14:18 by tzot, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
posixpath.diff tzot, 2009-01-26 14:18 Applied to python/trunk/Lib/posixpath.py
Messages (3)
msg80570 - (view) Author: Χρήστος Γεωργίου (Christos Georgiou) (tzot) * Date: 2009-01-26 14:18 
The paths_seen object is a list; a set is more appropriate, since its
main use is a lookup as in "path in paths_seen"
msg80602 - (view) Author: Gabriel Genellina (ggenellina) Date: 2009-01-27 00:32 
Simple and correct.
msg80612 - (view) Author: Benjamin Peterson (benjamin.peterson) * (Python committer) Date: 2009-01-27 03:08 
Thanks for the patch! Applied in r69003.
History
Date User Action Args
2022-04-11 14:56:44adminsetgithub: 49319
2009-01-27 03:08:28benjamin.petersonsetstatus: open -> closed
nosy: + benjamin.peterson
resolution: accepted
messages: + msg80612
2009-01-27 00:32:28ggenellinasetnosy: + ggenellina
messages: + msg80602
2009-01-26 14:18:12tzotcreate