Follow-up fix for encoding data as u32

waywardmonkeys · waywardmonkeys · commit efced13e8e30 · 2025-03-25T22:10:30.000+07:00
The original work was done in PR linebender#817 and as noticed in issue linebender#875, there was still a place where there was a length manipulation that was in terms of `u8` rather than `u32`. Fixes linebender#875.
diff --git a/vello_encoding/src/path.rs b/vello_encoding/src/path.rs
@@ -683,7 +683,7 @@ impl<'a> PathEncoder<'a> {
             self.close();
         }
         if self.state == PathState::MoveTo {
-            let new_len = self.data.len() - 8;
+            let new_len = self.data.len() - 2;
             self.data.truncate(new_len);
         }
         if self.n_encoded_segments != 0 {

Original file line number	Diff line number	Diff line change
`@@ -683,7 +683,7 @@ impl<'a> PathEncoder<'a> {`
`683`	`683`	`self.close();`
`684`	`684`	`}`
`685`	`685`	`if self.state == PathState::MoveTo {`
`686`		`- let new_len = self.data.len() - 8;`
	`686`	`+ let new_len = self.data.len() - 2;`
`687`	`687`	`self.data.truncate(new_len);`
`688`	`688`	`}`
`689`	`689`	`if self.n_encoded_segments != 0 {`