This works, but is very verbose

and still has got a gid.unwrap, which I think is fine due to the preceding gid.is_some(), but I may be wrong.

Author: felipesanches

fontspector-checkapi/src/font.rs

@@ -1,7 +1,7 @@
 use crate::{constants::RIBBI_STYLE_NAMES, filetype::FileTypeConvert, FileType, Testable};
 use read_fonts::{
     tables::cmap::Cmap,
-    tables::gdef::ClassDef,
+    tables::gdef::Gdef,
     tables::os2::SelectionFlags,
     TableProvider,
 };
@@ -73,16 +73,16 @@ impl TestFont {
         Ok(os2.fs_selection())
     }
 
-    pub fn get_gdef_glyph_class_def(&self) -> Result<ClassDef, Box<dyn Error>> {
-        let gdef = self.font().gdef()?;
-        Ok(gdef.glyph_class_def().unwrap()?)
-    }
-
     pub fn get_cmap(&self) -> Result<Cmap, Box<dyn Error>> {
         let cmap = self.font().cmap()?;
         Ok(cmap)
     }
 
+    pub fn get_gdef(&self) -> Result<Gdef, Box<dyn Error>> {
+        let gdef = self.font().gdef()?;
+        Ok(gdef)
+    }
+
     pub fn get_name_entry_strings(&self, name_id: StringId) -> LocalizedStrings {
         self.font().localized_strings(name_id)
     }

fontspector-checkapi/src/status.rs

@@ -60,6 +60,12 @@ impl Status {
             code: StatusCode::Fail,
         }
     }
+    pub fn error(s: &str) -> Self {
+        Self {
+            message: Some(s.to_string()),
+            code: StatusCode::Error,
+        }
+    }
     pub fn skip(s: &str) -> Self {
         Self {
             message: Some(s.to_string()),

profile-universal/src/checks/arabic_spacing_symbols.rs

@@ -23,8 +23,34 @@ const ARABIC_SPACING_SYMBOLS: [u16; 17] = [
 fn arabic_spacing_symbols(t: &Testable) -> StatusList {
     let mut problems: Vec<Status> = vec![];
     let f = TTF.from_testable(t).expect("Not a TTF file");
-    let cmap = f.get_cmap().unwrap();
-    let class_def = f.get_gdef_glyph_class_def().unwrap();
+    let cmap = match f.get_cmap() {
+        Err(_) => {
+            problems.push(Status::error("Font lacks a cmap table"));
+            return return_result(problems);
+        },
+        Ok(c) => c
+    };
+
+    let gdef = match f.get_gdef() {
+        Err(_) => {
+            problems.push(Status::error("Font lacks a gdef table"));
+            return return_result(problems);
+        },
+        Ok(g) => g
+    };
+
+    let class_def = match gdef.glyph_class_def() {
+        None => return return_result(problems),
+        Some(d) => d
+    };
+
+    let class_def = match class_def {
+        Err(e) => {
+            problems.push(Status::error(&format!("Some classDef error: {}", e)));
+            return return_result(problems);
+        },
+        Ok(d) => d
+    };
 
     for codepoint in ARABIC_SPACING_SYMBOLS {
         let gid = cmap.map_codepoint(codepoint);