Cast explicitly to c_long which is the underlying type

servo / font-kit

A cross-platform font loading library written in Rust

Apache License 2.0

680 stars 100 forks source link

Cast explicitly to c_long which is the underlying type #143

Closed richo closed 4 years ago

richo commented 4 years ago

I just copied the type from the underlying struct but if you prefer I'm happy to use the rust types and do some target_pointer_width shenanigans.

richo commented 4 years ago

Or I guess thinking about this more I can just cast to FT_Pos and FT_Fixed and let freetype deal with it. I don't have strong opinions so let me know if you'd prefer I did this some other way!

jdm commented 4 years ago

Casting to the freetype types makes more sense to me.

richo commented 4 years ago

Updated, thankyou! A test fails on my arm machine but I'm not sure if that's a regression or it never worked.

jdm commented 4 years ago

Please run cargo fmt to make CI happy :)