Move char_style_indices out of the per-item ShapeOptions

The style indices are text-wide, whereas the options are now per-item.
Fixes the TODO from
linebender#727 (comment).

Author: tomcur

parley/src/shape/mod.rs

@@ -173,7 +173,6 @@ pub(crate) fn shape_text<'a, B: Brush>(
                     font_size: item_style.font_size,
                     features: &style_features[item_style_index as usize],
                     variations: rcx.variations(item_style.font_variations).unwrap_or(&[]),
-                    char_style_indices,
                 },
             })
         })
@@ -184,6 +183,7 @@ pub(crate) fn shape_text<'a, B: Brush>(
     scx.shape_text(
         text,
         analysis,
+        char_style_indices,
         items,
         font_selector,
         &mut layout.data.shaped_text,

parley_engine/src/shape/atom.rs

@@ -858,10 +858,16 @@ mod tests {
                 language: None,
                 features: &[],
                 variations: &[],
-                char_style_indices: &char_style_indices,
             },
         }];
-        shaper.shape_text(text, &analysis, items, SingleFont(font), &mut shaped);
+        shaper.shape_text(
+            text,
+            &analysis,
+            &char_style_indices,
+            items,
+            SingleFont(font),
+            &mut shaped,
+        );
         shaped
     }
 

parley_engine/src/shape/shaped_text.rs

@@ -236,6 +236,7 @@ impl ShapedText {
         item: &Segment,
         options: &ShapeOptions<'_>,
         char_info: &[CharInfo],
+        char_style_indices: &[u16],
         font: &FontInstance,
         glyph_buffer: &harfrust::GlyphBuffer,
         normalized_coords: &[harfrust::NormalizedCoord],
@@ -318,7 +319,7 @@ impl ShapedText {
         for (((byte_offset, ch), info), style_index) in text[range.byte_range.clone()]
             .char_indices()
             .zip(&char_info[range.char_range.clone()])
-            .zip(&options.char_style_indices[range.char_range.clone()])
+            .zip(&char_style_indices[range.char_range.clone()])
         {
             self.characters.push(Character {
                 text_byte_start: (range.byte_range.start + byte_offset) as u32,
@@ -340,7 +341,7 @@ impl ShapedText {
                 scale_factor,
                 glyph_infos.iter(),
                 glyph_positions.iter(),
-                &options.char_style_indices[range.char_range.clone()],
+                &char_style_indices[range.char_range.clone()],
                 &self.characters,
                 characters_start,
             );
@@ -351,7 +352,7 @@ impl ShapedText {
                 scale_factor,
                 glyph_infos.iter().rev(),
                 glyph_positions.iter().rev(),
-                &options.char_style_indices[range.char_range.clone()],
+                &char_style_indices[range.char_range.clone()],
                 &self.characters,
                 characters_start,
             );
@@ -621,10 +622,16 @@ mod tests {
                 language: None,
                 features: &[],
                 variations: &[],
-                char_style_indices: &char_style_indices,
             },
         }];
-        shaper.shape_text(text, &analysis, items, SingleFont(font), &mut shaped);
+        shaper.shape_text(
+            text,
+            &analysis,
+            &char_style_indices,
+            items,
+            SingleFont(font),
+            &mut shaped,
+        );
         shaped
     }
 
@@ -732,7 +739,6 @@ mod tests {
                     language: None,
                     features: &[],
                     variations: &[],
-                    char_style_indices: &char_style_indices,
                 },
             },
             Item {
@@ -742,11 +748,17 @@ mod tests {
                     language: None,
                     features: &[],
                     variations: &[],
-                    char_style_indices: &char_style_indices,
                 },
             },
         ];
-        shaper.shape_text(text, &analysis, items, SingleFont(font), &mut shaped);
+        shaper.shape_text(
+            text,
+            &analysis,
+            &char_style_indices,
+            items,
+            SingleFont(font),
+            &mut shaped,
+        );
 
         let grapheme_starts: Vec<bool> =
             shaped.characters.iter().map(|c| c.grapheme_start).collect();

parley_engine/src/shape/shaper.rs

@@ -34,11 +34,6 @@ pub struct ShapeOptions<'a> {
     pub features: &'a [FontFeature],
     /// The font variations that are constant over an item.
     pub variations: &'a [FontVariation],
-    /// The per-character style indices.
-    // TODO: rename to something like `user_data` (s.t. we don't assume it's a style per se).
-    // TODO: probably move this out of `ShapeOptions`, and supply it as a parameter on
-    // `Shaper::shape_text`.
-    pub char_style_indices: &'a [u16],
 }
 
 /// The font instance to shape an item with.
@@ -92,6 +87,10 @@ impl Shaper {
     /// split on properties like shaping-relevant style changes (e.g., font size) or properties like
     /// language.
     ///
+    /// `char_style_indices` holds per-character style indices; these are copied onto
+    /// [`Character`][super::Character`] and [`ShapedCluster`][super::ShapedCluster]. A shaped
+    /// cluster's style index is that of its logically first constituent character.
+    ///
     /// Characters that don't have a particular script have their script resolved based on
     /// surrounding context (see [`Segment::script`]).
     ///
@@ -106,6 +105,8 @@ impl Shaper {
         &mut self,
         text: &str,
         analysis: &Analysis,
+        // TODO: rename to something like `user_data` (s.t. we don't assume it's a style per se).
+        char_style_indices: &[u16],
         items: impl IntoIterator<Item = Item<'options>>,
         mut select_font: impl FontSelector,
         shaped_text: &mut ShapedText,
@@ -142,6 +143,7 @@ impl Shaper {
                     &item.options,
                     &mut select_font,
                     analysis.char_info(),
+                    char_style_indices,
                     shaped_text,
                 )
                 .is_err()
@@ -182,6 +184,7 @@ fn shape_segment(
     options: &ShapeOptions<'_>,
     select_font: &mut impl FontSelector,
     char_info: &[CharInfo],
+    char_style_indices: &[u16],
     shaped_text: &mut ShapedText,
 ) -> Result<(), ()> {
     select_font.begin_segment(item, options);
@@ -193,7 +196,7 @@ fn shape_segment(
 
     // Only process current item
     let item_char_info = &char_info[char_range.start..char_range.end];
-    let item_char_style_indices = &options.char_style_indices[char_range.start..char_range.end];
+    let item_char_style_indices = &char_style_indices[char_range.start..char_range.end];
 
     if item_text.is_empty() {
         return Ok(()); // No clusters
@@ -378,6 +381,7 @@ fn shape_segment(
             item,
             options,
             char_info,
+            char_style_indices,
             &font,
             &glyph_buffer,
             harf_shaper.coords(),