Skip to content

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

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

Closed
wants to merge 2 commits into from

Conversation

srinivasreddy
Copy link
Contributor

@srinivasreddy srinivasreddy commented Oct 26, 2018

@srinivasreddy
Copy link
Contributor Author

Should supersede #4641.

@Mariatta Please review the News blurb.

@mariatta-bot
Copy link

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)

Copy link
Contributor

@akuchling akuchling left a comment

Choose a reason for hiding this comment

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

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.

@akuchling
Copy link
Contributor

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 akuchling closed this Mar 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants