Skip to content
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

gh-67224: Make linecache imports relative to improve startup speed #117501

Merged
merged 2 commits into from
Apr 10, 2024
Merged

gh-67224: Make linecache imports relative to improve startup speed #117501

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f10548f
gh-67224: Make linecache imports relative to improve startup speed
pablogsal Apr 3, 2024
9ab361a
Update Lib/linecache.py
pablogsal Apr 10, 2024
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
14 changes: 11 additions & 3 deletions Lib/linecache.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,6 @@
that name.
"""

import sys
import os

__all__ = ["getline", "clearcache", "checkcache", "lazycache"]


Expand Down Expand Up @@ -66,6 +63,11 @@ def checkcache(filename=None):
size, mtime, lines, fullname = entry
if mtime is None:
continue # no-op for files loaded via a __loader__
try:
# This import can fail if the interpreter is shutting down
import os
except ImportError:
return
try:
stat = os.stat(fullname)
except OSError:
Expand All @@ -76,6 +78,12 @@ def checkcache(filename=None):


def updatecache(filename, module_globals=None):
# These imports are not at top level because linecache is in the critical
# path of the interpreter startup and importing os and sys take a lot of time
# and slow down the startup sequence.
import os
import sys

"""Update a cache entry and return its list of lines.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We lost the docstring because you put the imports before it. Filed #117948 to fix this.

If something's wrong, print a message, discard the cache entry,
and return an empty list."""
Expand Down
Loading