[skera] move plan creation out of iterating loop

This will macth harfbuzz and make binary runtime comparison fairer

Author: qxliu76

skera/benches/skera-benchmarks.rs

@@ -13,18 +13,6 @@ fn read_test_font(file_name: &str) -> Vec<u8> {
     std::fs::read(&path).expect("failed to read test font file")
 }
 
-// comment out for now, may be used later
-//fn set_from_ranges<I>(ranges: I) -> IntSet<u32>
-//where
-//    I: IntoIterator<Item = std::ops::RangeInclusive<u32>>,
-//{
-//    let mut set = IntSet::empty();
-//    for range in ranges {
-//        set.insert_range(range);
-//    }
-//    set
-//}
-
 /// Creates a subsetting plan using the same default options as running the src/main.rs with
 /// --unicodes.
 fn create_plan(font: &FontRef, unicodes: &IntSet<u32>) -> Plan {
@@ -44,8 +32,6 @@ fn create_plan(font: &FontRef, unicodes: &IntSet<u32>) -> Plan {
 }
 
 fn benchmark_subset(c: &mut Criterion) {
-    // comment out for now, may be used later
-    // let latin_codepoints = set_from_ranges([0x20..=0x7E, 0xA0..=0xFF, 0x100..=0x24F]);
     c.benchmark_group("subset")
         .bench_function("Amiri-Regular-all", |b| {
             let font_bytes = read_test_font("Amiri-Regular.ttf");

skera/src/main.rs

@@ -233,18 +233,18 @@ fn main() {
     };
 
     let mut output_bytes = Vec::new();
+    let plan = Plan::new(
+        &gids,
+        &unicodes,
+        &font,
+        subset_flags,
+        &drop_tables,
+        &layout_scripts,
+        &layout_features,
+        &name_ids,
+        &name_languages,
+    );
     for _ in 0..args.num_iterations.unwrap_or(1) {
-        let plan = Plan::new(
-            &gids,
-            &unicodes,
-            &font,
-            subset_flags,
-            &drop_tables,
-            &layout_scripts,
-            &layout_features,
-            &name_ids,
-            &name_languages,
-        );
         match subset_font(&font, &plan) {
             Ok(out) => {
                 output_bytes = out;