[proxy] web.archive.org — ← back | site home | direct (HTTPS) ↗ | proxy home | ◑ dark◐ light
/ cpython Public
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.

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

Jump to bottom

bpo-32173: linecache.py add lazycache to __all__ and use dict.clear to clear the cache #10130

Closed
wants to merge 2 commits into from
Closed

bpo-32173: linecache.py add lazycache to __all__ and use dict.clear to clear the cache #10130

wants to merge 2 commits into from

Conversation

Copy link
Contributor

srinivasreddy commented Oct 26, 2018 •
edited by bedevere-bot

Copy link
Contributor Author

srinivasreddy commented Oct 26, 2018

Should supersede #4641.

@Mariatta Please review the News blurb.

Copy link

mariatta-bot commented Oct 26, 2018

Mariatta was mentioned, but she's out of open source for the rest of September 2018, and parts of October 2018 . Be aware she might not get to this until November 2018.
(I'm a bot)

akuchling approved these changes Mar 21, 2020
View changes
Copy link
Member

akuchling left a comment

This refactoring looks like a pleasant clean-up for the code. I don't think it needs any additions to the test suite, so I don't think any further work is needed.

Copy link
Member

akuchling commented Mar 21, 2020

On looking like the conflict, it looks like someone has already applied basically the same changes from PR-4641. So I'll just close this PR.

akuchling closed this Mar 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

akuchling

Assignees
No one assigned
Labels
awaiting merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants