Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-07-22 23:03:42 +01:00
Code Issues Projects Releases Packages Wiki Activity

Simplify FontTableMethods::with_buffer

Browse source
This commit is contained in:
Matt Brubeck 2016-05-18 10:46:37 -07:00
parent 2d5dc8fa6d
commit 1eab6fbb2e
4 changed files with 9 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

2
components/gfx/font.rs
View file

@ -68,7 +68,7 @@ impl FontTableTagConversions for FontTableTag {
}
pub trait FontTableMethods {
fn with_buffer<F>(&self, F) where F: FnOnce(*const u8, usize);
fn buffer(&self) -> &[u8];
}
#[derive(Clone, Debug, Deserialize, Serialize)]

4
components/gfx/platform/freetype/font.rs
View file

@ -40,8 +40,8 @@ pub struct FontTable {
}
impl FontTableMethods for FontTable {
fn with_buffer<F>(&self, blk: F) where F: FnOnce(*const u8, usize) {
blk(self.buffer.as_ptr(), self.buffer.len())
fn buffer(&self) -> &[u8] {
&self.buffer
}
}

4
components/gfx/platform/macos/font.rs
View file

@ -52,8 +52,8 @@ impl FontTable {
}
impl FontTableMethods for FontTable {
fn with_buffer<F>(&self, blk: F) where F: FnOnce(*const u8, usize) {
blk(self.data.bytes().as_ptr(), self.data.len() as usize);
fn buffer(&self) -> &[u8] {
self.data.bytes()
}
}

10
components/gfx/text/shaping/harfbuzz.rs
View file

@ -529,15 +529,13 @@ extern fn font_table_func(_: *mut hb_face_t,
// this raw pointer back to `destroy_blob_func` which will deallocate the Box.
let font_table_ptr = Box::into_raw(font_table);
let mut blob: *mut hb_blob_t = ptr::null_mut();
(*font_table_ptr).with_buffer(|buf: *const u8, len: usize| {
// HarfBuzz calls `destroy_blob_func` when the buffer is no longer needed.
blob = hb_blob_create(buf as *const c_char,
len as c_uint,
let buf = (*font_table_ptr).buffer();
// HarfBuzz calls `destroy_blob_func` when the buffer is no longer needed.
let blob = hb_blob_create(buf.as_ptr() as *const c_char,
buf.len() as c_uint,
HB_MEMORY_MODE_READONLY,
font_table_ptr as *mut c_void,
Some(destroy_blob_func));
});
assert!(!blob.is_null());
blob