Merge pull request #987 from googlefonts/fixup-a-fixup

[normalizer] Fix an old TODO
googlefonts · Oct 3, 2024 · 8644ab9 · 8644ab9
2 parents ac9b6f5 + 3d74574
commit 8644ab9
Showing 1 changed file with 2 additions and 9 deletions.
diff --git a/otl-normalizer/src/variations.rs b/otl-normalizer/src/variations.rs
@@ -1,8 +1,5 @@
 use write_fonts::read::{
-    tables::{
-        layout::VariationIndex,
-        variations::{DeltaSetIndex, ItemVariationStore},
-    },
+    tables::{layout::VariationIndex, variations::ItemVariationStore},
     types::F2Dot14,
     ReadError,
 };
@@ -53,11 +50,7 @@ impl<'a> DeltaComputer<'a> {
         coord: i32,
         idx: VariationIndex,
     ) -> Result<Vec<i32>, ReadError> {
-        // TODO: this deserves an Into?
-        let delta_ix = DeltaSetIndex {
-            outer: idx.delta_set_outer_index(),
-            inner: idx.delta_set_inner_index(),
-        };
+        let delta_ix = idx.into();
         self.locations
             .iter()
             .map(|loc| self.ivs.compute_delta(delta_ix, loc).map(|d| d + coord))