Use lru_cache where suitable by akx · Pull Request #967 · python-babel/babel

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Merged

akx merged 2 commits intomasterfrom

caches-1

Feb 7, 2023

Merged

Use lru_cache where suitable#967

akx merged 2 commits intomasterfrom

caches-1

Conversation

@akx

Copy link

Member

@akx akx commented

Feb 7, 2023

Fixes #962 (unbounded cache memory leak)

@codecov

Copy link

codecov bot commented

Feb 7, 2023

edited

Loading

Codecov Report

Merging #967 (152a9e3) into master (14e7c00) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #967      +/-   ##
==========================================
- Coverage   90.88%   90.88%   -0.01%     
==========================================
  Files          25       25              
  Lines        4334     4333       -1     
==========================================
- Hits         3939     3938       -1     
  Misses        395      395              
Impacted Files Coverage Δ
babel/dates.py 87.24% <100.00%> (ø)
babel/localedata.py 95.86% <100.00%> (-0.04%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@akx akx merged commit 08af5e2 into master

Feb 7, 2023

@akx akx deleted the caches-1 branch

February 7, 2023 14:30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Memory Leak Detected

1 participant

@akx