Visitar URL original
GH-101362: Optimise pathlib by deferring path normalisation by barneygale · Pull Request #101560 · python/cpython · GitHub
Skip to content

GH-101362: Optimise pathlib by deferring path normalisation #101560

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
barneygale wants to merge 16 commits into from
Closed

GH-101362: Optimise pathlib by deferring path normalisation #101560

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
b428069
GH-101362 - Optimise pathlib by deferring path normalisation
barneygale Jan 27, 2023
e49e719
Restore str force-casting behaviour; reduce diff a little.
barneygale Feb 4, 2023
a931986
Fix pathlib usage error in importlib
barneygale Feb 4, 2023
38f70bf
Improve initialiser performance.
barneygale Feb 4, 2023
10844a6
Add NEWS blurb
barneygale Feb 4, 2023
d5231b6
Store '_fspath' as non-empty string.
barneygale Feb 4, 2023
76b2db7
Merge branch 'main' into optimize-pathlib-part-2b
barneygale Mar 6, 2023
cc2c711
Undo addition of `__init__()` and change to `_from_parsed_parts()`
barneygale Mar 6, 2023
cbf0fcd
Fix pickling of paths created via walking.
barneygale Mar 6, 2023
eb7087f
Simplify patch slightly
barneygale Mar 6, 2023
3b53e27
Remove unused import
barneygale Mar 11, 2023
dec98f2
Merge branch 'main' into optimize-pathlib-part-2b
barneygale Mar 11, 2023
d9a6080
Fix dodgy merge, add comment about _from_parsed_parts()
barneygale Mar 11, 2023
9650dca
Stop returning unnormalised path from `__fspath__()`
barneygale Mar 17, 2023
af27476
Merge branch 'main' into optimize-pathlib-part-2b
barneygale Mar 17, 2023
97995bd
Undo change to `joinpath()`, fix news blurb.
barneygale Mar 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix dodgy merge, add comment about _from_parsed_parts()
  • Loading branch information
@barneygale
barneygale committed Mar 11, 2023
commit d9a6080f78c2ffa1e4bc0f6c2d3171c36719c94e
3 changes: 2 additions & 1 deletion Lib/pathlib.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -342,6 +342,7 @@ def __fspath__(self):
try:
return self._fspath or '.'
except AttributeError:
# The _from_parsed_parts() constructor does not set _fspath.
self._fspath = str(self)
return self._fspath

Expand Down Expand Up @@ -1202,7 +1203,7 @@ def expanduser(self):
homedir = self._flavour.expanduser(self._parts[0])
if homedir[:1] == "~":
raise RuntimeError("Could not determine home directory.")
drv, root, parts = self._parse_parts((homedir,))
drv, root, parts = self._parse_path(homedir)
return self._from_parsed_parts(drv, root, parts + self._parts[1:])

return self
Expand Down