Skip to content

/ servo

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

Modify font context to use an LRU cache for layout fonts. #3300

Open
glennw opened this issue Sep 12, 2014 · 1 comment
Open

Modify font context to use an LRU cache for layout fonts. #3300

glennw opened this issue Sep 12, 2014 · 1 comment
Labels
A-platform/fonts I-memory-leak

Comments

@glennw
Copy link
Member

@glennw glennw commented Sep 12, 2014

At the moment it holds a strong ref and never releases fonts.
glennw pushed a commit to glennw/servo that referenced this issue Sep 12, 2014
@gw3583
Fix comment and and reference to bug servo#3300
df48e33
@nox
Copy link
Member

@nox nox commented Aug 17, 2016

@glennw @pcwalton Could you explain or even document the code in the font cache thread?

Why does layout_thread uses FontCacheThread directly and not FontContext, even though the documentation for FontContext says it's the public API for layout and paint code?
@emilio emilio mentioned this issue Jan 23, 2017
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-platform/fonts I-memory-leak
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@mbrubeck @nox @glennw
You can’t perform that action at this time.