walkero
/
harfbuzz
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[subset/SingleSubsetFormat1] Use Coverage.intersect_set

This commit is contained in:
Behdad Esfahbod 2022-07-21 11:52:36 -06:00
parent 00dfbbce1c
commit efa388074d
1 changed files with 5 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/OT/Layout/GSUB/SingleSubstFormat1.hh
View File

@ -44,11 +44,7 @@ struct SingleSubstFormat1_3
if (1) if (1)
{ {
/* Walk the coverage and set together, to bail out when the /* Somehow this branch is much faster. Can't understand why. */
* coverage iterator runs over the end of glyphset. Otherwise,
* the coverage may cover many glyphs and we spend a lot of
* time here. Other subtables don't have this problem because
* they zip coverage iterator with some array... */
auto c_iter = hb_iter (this+coverage); auto c_iter = hb_iter (this+coverage);
auto s_iter = hb_iter (c->parent_active_glyphs ()); auto s_iter = hb_iter (c->parent_active_glyphs ());
while (c_iter && s_iter) while (c_iter && s_iter)
@ -69,8 +65,10 @@ struct SingleSubstFormat1_3
} }
else else
{ {
+ hb_iter (this+coverage) hb_set_t intersection;
| hb_filter (c->parent_active_glyphs ()) (this+coverage).intersect_set (c->parent_active_glyphs (), intersection);
+ hb_iter (intersection)
| hb_map ([d, mask] (hb_codepoint_t g) { return (g + d) & mask; }) | hb_map ([d, mask] (hb_codepoint_t g) { return (g + d) & mask; })
| hb_sink (c->output) | hb_sink (c->output)
; ;