Make a cosmetic adjustment

bodoni · May 23, 2024 · f8b4922 · f8b4922
1 parent 2080589
commit f8b4922
Showing 1 changed file with 12 additions and 15 deletions.
diff --git a/src/formats/opentype/features/glyphs.rs b/src/formats/opentype/features/glyphs.rs
@@ -114,9 +114,7 @@ impl Glyphs for opentype::tables::glyph_substitution::Type {
                         record.records.iter().map(move |record| {
                             let mut value = Vec::with_capacity(record.glyph_count as usize);
                             value.push(glyph_id.into());
-                            for glyph_id in record.glyph_ids.iter().cloned() {
-                                value.push(glyph_id.into());
-                            }
+                            value.extend(record.glyph_ids.iter().cloned().map(Into::into));
                             (value, vec![record.glyph_id.into()])
                         })
                     },
@@ -128,9 +126,7 @@ impl Glyphs for opentype::tables::glyph_substitution::Type {
                         record.records.iter().map(move |record| {
                             let mut value = Vec::with_capacity(record.glyph_count as usize);
                             value.push(glyph_id.into());
-                            for glyph_id in record.glyph_ids.iter().cloned() {
-                                value.push(glyph_id.into());
-                            }
+                            value.extend(record.glyph_ids.iter().cloned().map(Into::into));
                             (value, Default::default())
                         })
                     },
@@ -176,16 +172,17 @@ impl Glyphs for opentype::tables::glyph_substitution::Type {
                                     + record.glyph_count as usize
                                     + record.forward_glyph_count as usize,
                             );
-                            for glyph_id in record.backward_glyph_ids.iter().rev().cloned() {
-                                value.push(glyph_id.into());
-                            }
+                            value.extend(
+                                record
+                                    .backward_glyph_ids
+                                    .iter()
+                                    .rev()
+                                    .cloned()
+                                    .map(Into::into),
+                            );
                             value.push(glyph_id.into());
-                            for glyph_id in record.glyph_ids.iter().cloned() {
-                                value.push(glyph_id.into());
-                            }
-                            for glyph_id in record.forward_glyph_ids.iter().cloned() {
-                                value.push(glyph_id.into());
-                            }
+                            value.extend(record.glyph_ids.iter().cloned().map(Into::into));
+                            value.extend(record.forward_glyph_ids.iter().cloned().map(Into::into));
                             (value, Default::default())
                         })
                     },