clippy fixes + comment out CORETEXT flag to make it compile on aarch64

Author: asibahi

build.rs

@@ -10,10 +10,10 @@ fn main() {
     let target = env::var("TARGET").unwrap();
 
     println!("cargo:rerun-if-env-changed=HARFBUZZ_SYS_NO_PKG_CONFIG");
-    if target.contains("wasm32") || env::var_os("HARFBUZZ_SYS_NO_PKG_CONFIG").is_none() {
-        if pkg_config::probe_library("harfbuzz").is_ok() {
-            return;
-        }
+    if (target.contains("wasm32") || env::var_os("HARFBUZZ_SYS_NO_PKG_CONFIG").is_none())
+        && pkg_config::probe_library("harfbuzz").is_ok()
+    {
+        return;
     }
 
     let mut cfg = cc::Build::new();
@@ -27,9 +27,9 @@ fn main() {
         cfg.define("HAVE_PTHREAD", "1");
     }
 
-    if target.contains("apple") {
-        cfg.define("HAVE_CORETEXT", "1");
-    }
+    // if target.contains("apple") {
+    //     cfg.define("HAVE_CORETEXT", "1");
+    // }
 
     if target.contains("windows-gnu") {
         cfg.flag("-Wa,-mbig-obj");

src/blob.rs

@@ -1,6 +1,5 @@
 use std::os::raw::c_void;
 
-use std;
 use std::marker::PhantomData;
 
 use std::fmt;
@@ -89,7 +88,7 @@ impl<'a> Blob<'a> {
         let data = Box::into_raw(boxxed);
 
         extern "C" fn destroy<U>(ptr: *mut c_void) {
-            unsafe { Box::from_raw(ptr as *mut U) };
+            _ = unsafe { Box::from_raw(ptr as *mut U) };
         }
 
         let hb_blob = unsafe {

src/buffer.rs

@@ -159,8 +159,9 @@ impl GlyphInfo {
     }
 }
 
-#[derive(Debug, Copy, Clone, PartialEq, Eq, Hash)]
+#[derive(Debug, Default, Copy, Clone, PartialEq, Eq, Hash)]
 pub enum ClusterLevel {
+    #[default]
     MonotoneGraphemes,
     MonotoneCharacters,
     Characters,
@@ -185,12 +186,6 @@ impl ClusterLevel {
     }
 }
 
-impl Default for ClusterLevel {
-    fn default() -> Self {
-        ClusterLevel::MonotoneGraphemes
-    }
-}
-
 #[derive(Debug)]
 pub(crate) struct GenericBuffer {
     raw: NonNull<hb_buffer_t>,

src/common.rs

@@ -112,7 +112,6 @@ pub enum TagFromStrErr {
     ZeroLengthString,
 }
 
-use std;
 use std::str::FromStr;
 
 impl FromStr for Tag {

src/face.rs

@@ -1,4 +1,3 @@
-use std;
 use std::os::raw::c_void;
 use std::ptr::NonNull;
 
@@ -56,7 +55,7 @@ impl<'a> Face<'a> {
 
     /// Create a face from the bytes of a given slice and an index specifying
     /// which font to read from an OpenType font collection.
-    pub fn from_bytes<'b>(bytes: &'b [u8], index: u32) -> Owned<Face<'b>> {
+    pub fn from_bytes(bytes: &[u8], index: u32) -> Owned<Face<'_>> {
         let blob = Blob::with_bytes(bytes);
         Face::new(blob, index)
     }
@@ -68,7 +67,7 @@ impl<'a> Face<'a> {
         F: 'b + Send + Sync + FnMut(Tag) -> Option<Shared<Blob<'b>>>,
     {
         extern "C" fn destroy_box<U>(ptr: *mut c_void) {
-            unsafe { Box::from_raw(ptr as *mut U) };
+            _ = unsafe { Box::from_raw(ptr as *mut U) };
         }
         extern "C" fn table_func<'b, F>(
             _: *mut hb_face_t,

src/font.rs

@@ -1,4 +1,3 @@
-use std;
 use std::ptr::NonNull;
 
 use std::os::raw::c_void;
@@ -58,7 +57,7 @@ impl FontExtents {
 pub type GlyphExtents = hb_glyph_extents_t;
 
 pub(crate) extern "C" fn destroy_box<U>(ptr: *mut c_void) {
-    unsafe { Box::from_raw(ptr as *mut U) };
+    _ = unsafe { Box::from_raw(ptr as *mut U) };
 }
 
 /// A type representing a single font (i.e. a specific combination of typeface,