Add a phase that eliminates mixed contour+component glyphs; legal in source but not binary

fontbe/Cargo.toml

@@ -26,9 +26,10 @@ env_logger = "0.9.0"
 
 parking_lot = "0.12.1"
 
-read-fonts = "0.0.5"
-write-fonts = "0.0.5"
-fea-rs = "0.2.0"
+read-fonts = "0.1.0"
+write-fonts = "0.1.0"
+
+fea-rs = "0.3.0"
 smol_str = "0.1.18"
 
 [dev-dependencies]

fontbe/src/orchestration.rs

@@ -3,7 +3,7 @@
 use std::{collections::HashSet, fs, path::Path, sync::Arc};
 
 use fontdrasil::{
-    orchestration::{AccessControlList, Work, MISSING_DATA},
+    orchestration::{access_one, AccessControlList, Work, MISSING_DATA},
     types::GlyphName,
 };
 use fontir::orchestration::{Context as FeContext, WorkId as FeWorkIdentifier};
@@ -74,6 +74,9 @@ pub struct Context {
     // If set additional debug files will be emitted to disk
     pub emit_debug: bool,
 
+    // If set seek to match fontmake (python) behavior even at cost of abandoning optimizations
+    match_legacy: bool,
+
     paths: Arc<Paths>,
 
     // The final, fully populated, read-only FE context
@@ -87,15 +90,29 @@ pub struct Context {
 }
 
 impl Context {
+    fn copy(&self, acl: AccessControlList<AnyWorkId>) -> Context {
+        Context {
+            emit_ir: self.emit_ir,
+            emit_debug: self.emit_debug,
+            match_legacy: self.match_legacy,
+            paths: self.paths.clone(),
+            ir: self.ir.clone(),
+            acl,
+            features: self.features.clone(),
+        }
+    }
+
     pub fn new_root(
         emit_ir: bool,
         emit_debug: bool,
+        match_legacy: bool,
         paths: Paths,
         ir: &fontir::orchestration::Context,
     ) -> Context {
         Context {
             emit_ir,
             emit_debug,
+            match_legacy,
             paths: Arc::from(paths),
             ir: Arc::from(ir.read_only()),
             acl: AccessControlList::read_only(),
@@ -108,14 +125,14 @@ impl Context {
         work_id: WorkId,
         dependencies: Option<HashSet<AnyWorkId>>,
     ) -> Context {
-        Context {
-            emit_ir: self.emit_ir,
-            emit_debug: self.emit_debug,
-            paths: self.paths.clone(),
-            ir: self.ir.clone(),
-            acl: AccessControlList::read_write(dependencies.unwrap_or_default(), work_id.into()),
-            features: self.features.clone(),
-        }
+        self.copy(AccessControlList::read_write(
+            dependencies.unwrap_or_default(),
+            access_one(work_id.into()),
+        ))
+    }
+
+    pub fn read_only(&self) -> Context {
+        self.copy(AccessControlList::read_only())
     }
 }