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

Audit unsafe code in font handling #64

Closed
brson opened this issue Sep 7, 2012 · 2 comments
Closed

Audit unsafe code in font handling #64

brson opened this issue Sep 7, 2012 · 2 comments
Labels
A-platform/fonts I-refactor I-safety

Comments

@brson
Copy link
Contributor

@brson brson commented Sep 7, 2012

The font code is doing manual memory management that isn't failsafe.
@burg
Copy link

@burg burg commented Oct 25, 2012

In particular, there shouldn't be any unsafe calls within src/servo/text/font.rs (where the glyph-filling code is)
@recrack recrack mentioned this issue Sep 30, 2013
Merged
ChrisParis pushed a commit to ChrisParis/servo that referenced this issue Sep 7, 2014
@zqzhang
Merge pull request servo#64 from w3c/submission/Microsoft/IndexedDB
abc902c 
IndexedDB: Microsoft's tests
ChrisParis pushed a commit to ChrisParis/servo that referenced this issue Sep 7, 2014
Zhiqiang Zhang
Rename idbcursor_advance_index.htm as it was used in servo#64
b76c116
@kmcallister kmcallister changed the title Make high level bindings for azure and cairo fonts Audit unsafe code in font handling May 17, 2015
@kmcallister kmcallister added this to the Dogfooding milestone May 17, 2015
@jdm jdm removed this from the Dogfooding milestone Jul 19, 2016
This was referenced Dec 29, 2016
Closed
Closed
glennw added a commit to glennw/servo that referenced this issue Jan 16, 2017
@glennw
Merge pull request servo#64 from glennw/freelist
2faf90e 
Add a simple dead/live style free-list, and use this for texture cache items.
glennw pushed a commit to glennw/servo that referenced this issue Jan 16, 2017
@bors-servo
Auto merge of servo#64 - pcwalton:scroll-perf-fixes, r=glennw
d86e51a 
Add infrastructure to enable WebRender to skip compositing after scroll events that didn't actually result in motion.

r? @glennw
@zarazek zarazek mentioned this issue Jun 14, 2017
Open
@nox
Copy link
Member

@nox nox commented Sep 29, 2017

There is no unsafe code in any text module anymore.
@nox nox closed this Sep 29, 2017
@jdm jdm mentioned this issue Nov 5, 2019
Closed
@Manishearth Manishearth mentioned this issue Nov 26, 2019
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-refactor I-safety
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants
@jdm @burg @nox @brson @kmcallister
You can’t perform that action at this time.