Merge pull request #562 from googlefonts/fast_kern

Directly interfere to start kern sooner
googlefonts · Nov 10, 2023 · 0a00caa · 0a00caa
2 parents d87337f + 11c73b3
commit 0a00caa
Showing 1 changed file with 14 additions and 1 deletion.
diff --git a/fontc/src/workload.rs b/fontc/src/workload.rs
@@ -232,11 +232,15 @@ impl<'a> Workload<'a> {
             .queued()
             .run();
 
-        let launchable: Vec<_> = self
+        let mut has_kern = false;
+        let mut launchable: Vec<_> = self
             .jobs_pending
             .iter()
             .filter_map(|(id, job)| {
                 if !job.running && self.can_run(job) {
+                    if matches!(id, AnyWorkId::Fe(FeWorkIdentifier::Kerning)) {
+                        has_kern = true;
+                    }
                     Some(id.clone())
                 } else {
                     None
@@ -245,6 +249,15 @@ impl<'a> Workload<'a> {
             .collect();
         trace!("Launchable: {launchable:?}");
 
+        // https://github.com/googlefonts/fontc/issues/456: try to avoid kern as long pole
+        if has_kern {
+            let kern_idx = launchable
+                .iter()
+                .position(|id| matches!(id, AnyWorkId::Fe(FeWorkIdentifier::Kerning)))
+                .unwrap();
+            launchable.swap(0, kern_idx);
+        }
+
         self.timer.add(timing.complete());
         launchable
     }