Revert "face.glyph should be used as a pointer instead of a value"

This reverts commit d9bd35c.

Author: Gnimuc

src/rendering.jl

@@ -7,7 +7,7 @@ end
 function renderface(face::FTFont, c::Char, pixelsize::Integer)
     set_pixelsize(face, pixelsize)
     loadchar(face, c)
-    glyph = unsafe_load(face.glyph)
+    glyph = face.glyph
     @assert glyph.format == FreeType.FT_GLYPH_FORMAT_BITMAP
     return glyphbitmap(glyph.bitmap), FontExtent(glyph.metrics)
 end

src/types.jl

@@ -156,6 +156,9 @@ function Base.getproperty(font::FTFont, fieldname::Symbol)
     if field isa Ptr{FT_String}
         field == C_NULL && return ""
         return unsafe_string(field)
+    # Some fields segfault with unsafe_load...Lets find out which another day :D
+    elseif field isa Ptr{FreeType.LibFreeType.FT_GlyphSlotRec}
+        return unsafe_load(field)
     else
         return field
     end
@@ -205,7 +208,7 @@ function internal_get_extent(face::FTFont, char::Char)
     check_error(err, "Could not load char to get extent.")
     # This gives us the font metrics in normalized units (0, 1), with negative
     # numbers interpreted as an offset
-    return FontExtent(unsafe_load(face.glyph).metrics, Float32(face.units_per_EM))
+    return FontExtent(face.glyph.metrics, Float32(face.units_per_EM))
 end
 
 descender(font) = font.descender / font.units_per_EM